From: Thomas Schlichter the patch "serial-03-fixups.patch" introduced following compile error: CC [M] drivers/char/riscom8.o drivers/char/riscom8.c: In function `rc_tiocmget': drivers/char/riscom8.c:1323: error: `result' undeclared (first use in this= =20 function) drivers/char/riscom8.c:1323: error: (Each undeclared identifier is reported= =20 only once drivers/char/riscom8.c:1323: error: for each function it appears in.) drivers/char/riscom8.c: In function `rc_tiocmset': drivers/char/riscom8.c:1336: Warnung: unused variable `arg' drivers/char/riscom8.c: At top level: drivers/char/riscom8.c:1669: error: `rx_tiocmset' undeclared here (not in a= =20 function) drivers/char/riscom8.c:1669: error: initializer element is not constant drivers/char/riscom8.c:1669: error: (near initialization for=20 `riscom_ops.tiocmset') drivers/char/riscom8.c:1334: Warnung: `rc_tiocmset' defined but not used --- 25-akpm/drivers/char/riscom8.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff -puN drivers/char/riscom8.c~serial-03-fixups-fix-2 drivers/char/riscom8.c --- 25/drivers/char/riscom8.c~serial-03-fixups-fix-2 Wed Jan 14 13:00:33 2004 +++ 25-akpm/drivers/char/riscom8.c Wed Jan 14 13:00:33 2004 @@ -1311,6 +1311,7 @@ static int rc_tiocmget(struct tty_struct struct riscom_port *port = (struct riscom_port *)tty->driver_data; struct riscom_board * bp; unsigned char status; + unsigned int result; unsigned long flags; if (rc_paranoia_check(port, tty->name, __FUNCTION__)) @@ -1322,18 +1323,18 @@ static int rc_tiocmget(struct tty_struct status = rc_in(bp, CD180_MSVR); result = rc_in(bp, RC_RI) & (1u << port_No(port)) ? 0 : TIOCM_RNG; restore_flags(flags); - return ((status & MSVR_RTS) ? TIOCM_RTS : 0) + result |= ((status & MSVR_RTS) ? TIOCM_RTS : 0) | ((status & MSVR_DTR) ? TIOCM_DTR : 0) | ((status & MSVR_CD) ? TIOCM_CAR : 0) | ((status & MSVR_DSR) ? TIOCM_DSR : 0) | ((status & MSVR_CTS) ? TIOCM_CTS : 0); + return result; } static int rc_tiocmset(struct tty_struct *tty, struct file *file, unsigned int set, unsigned int clear) { struct riscom_port *port = (struct riscom_port *)tty->driver_data; - unsigned int arg; unsigned long flags; struct riscom_board *bp; @@ -1666,7 +1667,7 @@ static struct tty_operations riscom_ops .start = rc_start, .hangup = rc_hangup, .tiocmget = rc_tiocmget, - .tiocmset = rx_tiocmset, + .tiocmset = rc_tiocmset, }; static inline int rc_init_drivers(void) _