Jimi Xenidis
2006-Mar-28 15:13 UTC
[XenPPC] [patch] all UART parameters should be unsigned.
A signed baud results in a huge timeout if baud == BAUD_AUTO which is -1. There is no reason for any of uart members to be signed. Signed-off-by: Jimi Xenidis <jimix@watson.ibm.com> --- diff -r 7cdae181971d xen/drivers/char/ns16550.c --- a/xen/drivers/char/ns16550.c Mon Mar 27 16:52:31 2006 -0500 +++ b/xen/drivers/char/ns16550.c Tue Mar 28 10:11:57 2006 -0500 @@ -27,7 +27,7 @@ string_param("com2", opt_com2); string_param("com2", opt_com2); static struct ns16550 { - int baud, data_bits, parity, stop_bits, irq; + unsigned int baud, data_bits, parity, stop_bits, irq; unsigned long io_base; /* I/O port or memory-mapped I/O address. */ char *remapped_io_base; /* Remapped virtual address of mmap I/ O. */ /* UART with IRQ line: interrupt-driven I/O. */ _______________________________________________ Xen-ppc-devel mailing list Xen-ppc-devel@lists.xensource.com http://lists.xensource.com/xen-ppc-devel