On Wed, Aug 31, 2011 at 21:24, Jiri Slaby <jslaby@xxxxxxx> wrote: > The code is dead at least since 2002. So remove it to not distort git > grep output (about port.tty usage). > > Remove the whole do_softirq tasklet as it's noop now. > > Signed-off-by: Jiri Slaby <jslaby@xxxxxxx> > Cc: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > Cc: Alan Cox <alan@xxxxxxxxxxxxxxx> > --- > Cc: linux-m68k@xxxxxxxxxxxxxxxxxxxx This is a uClinux driver. > drivers/tty/serial/68328serial.c | 35 ----------------------------------- > drivers/tty/serial/68328serial.h | 1 - > 2 files changed, 0 insertions(+), 36 deletions(-) > > diff --git a/drivers/tty/serial/68328serial.c b/drivers/tty/serial/68328serial.c > index e0a7754..f549231 100644 > --- a/drivers/tty/serial/68328serial.c > +++ b/drivers/tty/serial/68328serial.c > @@ -235,22 +235,6 @@ static void batten_down_hatches(void) > > static void status_handle(struct m68k_serial *info, unsigned short status) > { > -#if 0 > - if(status & DCD) { > - if((info->port.tty->termios->c_cflag & CRTSCTS) && > - ((info->curregs[3] & AUTO_ENAB)==0)) { > - info->curregs[3] |= AUTO_ENAB; > - info->pendregs[3] |= AUTO_ENAB; > - write_zsreg(info->m68k_channel, 3, info->curregs[3]); > - } > - } else { > - if((info->curregs[3] & AUTO_ENAB)) { > - info->curregs[3] &= ~AUTO_ENAB; > - info->pendregs[3] &= ~AUTO_ENAB; > - write_zsreg(info->m68k_channel, 3, info->curregs[3]); > - } > - } > -#endif > /* If this is console input and this is a > * 'break asserted' status change interrupt > * see if we can drop into the debugger > @@ -340,9 +324,6 @@ static void transmit_chars(struct m68k_serial *info) > info->xmit_tail = info->xmit_tail & (SERIAL_XMIT_SIZE-1); > info->xmit_cnt--; > > - if (info->xmit_cnt < WAKEUP_CHARS) > - schedule_work(&info->tqueue); > - > if(info->xmit_cnt <= 0) { > /* All done for now... TX ints off */ > uart->ustcnt &= ~USTCNT_TX_INTR_MASK; > @@ -378,21 +359,6 @@ irqreturn_t rs_interrupt(int irq, void *dev_id) > return IRQ_HANDLED; > } > > -static void do_softint(struct work_struct *work) > -{ > - struct m68k_serial *info = container_of(work, struct m68k_serial, tqueue); > - struct tty_struct *tty; > - > - tty = info->tty; > - if (!tty) > - return; > -#if 0 > - if (clear_bit(RS_EVENT_WRITE_WAKEUP, &info->event)) { > - tty_wakeup(tty); > - } > -#endif > -} > - > static int startup(struct m68k_serial * info) > { > m68328_uart *uart = &uart_addr[info->line]; > @@ -1324,7 +1290,6 @@ rs68328_init(void) > info->event = 0; > info->count = 0; > info->blocked_open = 0; > - INIT_WORK(&info->tqueue, do_softint); > init_waitqueue_head(&info->open_wait); > init_waitqueue_head(&info->close_wait); > info->line = i; > diff --git a/drivers/tty/serial/68328serial.h b/drivers/tty/serial/68328serial.h > index 8c9c3c0..3d2faab 100644 > --- a/drivers/tty/serial/68328serial.h > +++ b/drivers/tty/serial/68328serial.h > @@ -158,7 +158,6 @@ struct m68k_serial { > int xmit_head; > int xmit_tail; > int xmit_cnt; > - struct work_struct tqueue; > wait_queue_head_t open_wait; > wait_queue_head_t close_wait; > }; > -- > 1.7.6.1 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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 -- To unsubscribe from this list: send the line "unsubscribe linux-serial" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html