lkml.org 
[lkml]   [2008]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
Subjectvme_scc.c breakage (was: Re: linux-next: Tree for November 21)
	Hi Alan,

drivers/char/vme_scc.c no longer compiles in linux-next:
http://kisskb.ellerman.id.au/kisskb/buildresult/56964/

I get a slightly different error message, though:

| drivers/char/vme_scc.c: In function 'scc_carrier_raised':
| drivers/char/vme_scc.c:634: error: expected identifier or '(' before 'volatile'
| drivers/char/vme_scc.c:634: error: expected ')' before '(' token
| drivers/char/vme_scc.c:634: error: expected statement before ')' token
| drivers/char/vme_scc.c:635: error: 'struct tty_port' has no member named 'channel'

The problem seems to be introduced by commit
e21405cbbaa859c68119b260c0fe43afdb86a385 ("tty-port-extract-cd"). The relevant
part is below:

| diff --git a/drivers/char/vme_scc.c b/drivers/char/vme_scc.c
| index 1718b3c..d4e1534 100644
| --- a/drivers/char/vme_scc.c
| +++ b/drivers/char/vme_scc.c
| @@ -624,9 +629,9 @@ static void scc_enable_rx_interrupts(void *ptr)
| }
|
|
| -static int scc_get_CD(void *ptr)
| +static int scc_carrier_raised(struct tty_port *port)
| {
| - 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'.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds


\
 
 \ /
  Last update: 2008-11-22 16:11    [W:0.808 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site