On Wed, Mar 12, 2008 at 02:30:06AM +0100, Matteo Croce wrote:
> Ugly but we need it
why ? I'm running AR7 uarts just like all other 16550.
> #if defined (CONFIG_SERIAL_8250_AU1X00)
> @@ -2455,7 +2462,11 @@ static void serial8250_console_putchar(struct uart_port *port, int ch)
> {
> struct uart_8250_port *up = (struct uart_8250_port *)port;
>
> +#ifdef CONFIG_AR7
> + wait_for_xmitr(up, BOTH_EMPTY);
> +#else
> wait_for_xmitr(up, UART_LSR_THRE);
> +#endif
> serial_out(up, UART_TX, ch);
> }
this doesn't make sense. Why don't you check for the port type and
decide, if you need to use the AR7 way or the normal way. This will
for example break 16550 uarts connected via PCI on UR8 devices.
> diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h
> index 289942f..869b6df 100644
> --- a/include/linux/serial_core.h
> +++ b/include/linux/serial_core.h
> @@ -40,6 +40,7 @@
> #define PORT_NS16550A 14
> #define PORT_XSCALE 15
> #define PORT_RM9000 16 /* PMC-Sierra RM9xxx internal UART */
> +#define PORT_AR7 16
this doesn't look correct.
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessary a
good idea. [ RFC1925, 2.3 ]
