Re: vme_scc.c breakage (was: Re: linux-next: Tree for November 21)

From: Alan Cox
Date: Sat Nov 22 2008 - 11:48:03 EST


> | - struct scc_port *port = ptr;
> | + struct scc_port *scc = container_of(port, struct scc_port, gs.port);
> | unsigned channel = port->channel;
> |
> | return !!(scc_last_status_reg[channel] & SR_DCD);
>
> Issues:
> 1. When Atari support is enabled, `scc' is defined in
> include/asm-m68k/atari.h as:
>
> # define scc ((*(volatile struct SCC*)SCC_BAS))
>
> However, renaming the variable `scc' doesn't help, as
>
> 2. the variable `scc' is not used in scc_carrier_raised(),
>
> 3. `port->channel' on line 635 still fails as `port' is not longer `struct
> scc_port', but a 'struct tty_port', which has no member named 'channel'.

Will take a look at this, but gs is going to away so at some point
someone with hardware is going to have to admit to owning this driver or
just mark it BROKEN..

Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/