On Tue, Oct 30, 2007 at 09:40:15PM +0100, Giuseppe Sacco wrote:
> this is a patch that make ttyS0 and ttyS1 work on my SGI O2. I don't know if
> it is enough good for a general use since I am also changing code
> drivers/serial/serial_core.c. Probably the best solution would be to use
> mapbase instead of membase in arch/mips/sgi-ip32/ip32-platform.c.
> diff --git a/arch/mips/sgi-ip32/ip32-platform.c
> b/arch/mips/sgi-ip32/ip32-platform.c
> index 7309e48..77febd6 100644
> --- a/arch/mips/sgi-ip32/ip32-platform.c
> +++ b/arch/mips/sgi-ip32/ip32-platform.c
> @@ -42,7 +42,7 @@ static struct platform_device uart8250_device = {
> static int __init uart8250_init(void)
> {
> uart8250_data[0].membase = (void __iomem *) &mace->isa.serial1;
> - uart8250_data[1].membase = (void __iomem *) &mace->isa.serial1;
> + uart8250_data[1].membase = (void __iomem *) &mace->isa.serial2;
The s/isa.serial1/isa.serial2/ part looks reasonable.
> diff --git a/drivers/serial/serial_core.c b/drivers/serial/serial_core.c
> index 3bb5d24..7caa877 100644
> --- a/drivers/serial/serial_core.c
> +++ b/drivers/serial/serial_core.c
> @@ -2455,6 +2455,8 @@ int uart_match_port(struct uart_port *port1, struct
> uart_port *port2)
> case UPIO_AU:
> case UPIO_TSI:
> case UPIO_DWAPB:
> + if (port1->mapbase==0 && port2->mapbase==0)
> + return (port1->membase == port2->membase);
This hack is only needed because ->mapbase is not initialized.
Ralf
|