serial_pnx8xxx.h just contains structure and register definitions for
this driver - wouldn't it make more sense for it to be in drivers/serial
along side this driver?
> +I guess it's include/asm-mips/arch-pnx8550/uart.h, but it is not
> +#include <asm/io.h>
> +#include <asm/irq.h>
> +
> +#include <uart.h>
What is uart.h? It isn't in this patch, neither is it part of mainline.
Still not using read_status_mask here, as detailed in my previous review.
status &= sport->port.read_status_mask;
is what's missing.
> + /*
> + * Disable all interrupts, port and break condition.
> + */
> + serial_out(sport, PNX8XXX_IEN, 0);
This comment's not correct - where is the break condition disabled?
I thought it might be in the next serial_out() but it seems to be
missing from there as well?
> + if (termios->c_iflag & IGNBRK) {
> + sport->port.ignore_status_mask |=
> + ISTAT_TO_SM(PNX8XXX_UART_INT_BREAK);
> + /*
> + * If we're ignoring parity and break indicators,
> + * ignore overruns too (for real raw support).
> + */
> + if (termios->c_iflag & IGNPAR)
> + sport->port.ignore_status_mask |=
> + ISTAT_TO_SM(PNX8XXX_UART_INT_RXOVRN);
> + }
How about CREAD support?