Commit ace7dd96 authored by Jiri Slaby's avatar Jiri Slaby Committed by Linus Torvalds

Char: mxser, various cleanups

- remove unused macro
- some whitespace cleanup
- useless debug prints removal
Signed-off-by: default avatarJiri Slaby <jirislaby@gmail.com>
Acked-by: default avatarAlan Cox <alan@redhat.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 1df00924
...@@ -48,7 +48,6 @@ ...@@ -48,7 +48,6 @@
#define MXSER_VERSION "2.0.4" /* 1.12 */ #define MXSER_VERSION "2.0.4" /* 1.12 */
#define MXSERMAJOR 174 #define MXSERMAJOR 174
#define MXSERCUMAJOR 175
#define MXSER_BOARDS 4 /* Max. boards */ #define MXSER_BOARDS 4 /* Max. boards */
#define MXSER_PORTS_PER_BOARD 8 /* Max. ports per board */ #define MXSER_PORTS_PER_BOARD 8 /* Max. ports per board */
...@@ -191,7 +190,6 @@ struct mxser_log { ...@@ -191,7 +190,6 @@ struct mxser_log {
unsigned long txcnt[MXSER_PORTS]; unsigned long txcnt[MXSER_PORTS];
}; };
struct mxser_mon { struct mxser_mon {
unsigned long rxcnt; unsigned long rxcnt;
unsigned long txcnt; unsigned long txcnt;
...@@ -1305,13 +1303,9 @@ static void mxser_flush_chars(struct tty_struct *tty) ...@@ -1305,13 +1303,9 @@ static void mxser_flush_chars(struct tty_struct *tty)
struct mxser_port *info = tty->driver_data; struct mxser_port *info = tty->driver_data;
unsigned long flags; unsigned long flags;
if (info->xmit_cnt <= 0 || if (info->xmit_cnt <= 0 || tty->stopped || !info->port.xmit_buf ||
tty->stopped || (tty->hw_stopped && info->type != PORT_16550A &&
!info->port.xmit_buf || !info->board->chip_flag))
(tty->hw_stopped &&
(info->type != PORT_16550A) &&
(!info->board->chip_flag)
))
return; return;
spin_lock_irqsave(&info->slock, flags); spin_lock_irqsave(&info->slock, flags);
...@@ -1329,9 +1323,7 @@ static int mxser_write_room(struct tty_struct *tty) ...@@ -1329,9 +1323,7 @@ static int mxser_write_room(struct tty_struct *tty)
int ret; int ret;
ret = SERIAL_XMIT_SIZE - info->xmit_cnt - 1; ret = SERIAL_XMIT_SIZE - info->xmit_cnt - 1;
if (ret < 0) return ret < 0 ? 0 : ret;
ret = 0;
return ret;
} }
static int mxser_chars_in_buffer(struct tty_struct *tty) static int mxser_chars_in_buffer(struct tty_struct *tty)
...@@ -2762,8 +2754,6 @@ static int __init mxser_module_init(void) ...@@ -2762,8 +2754,6 @@ static int __init mxser_module_init(void)
unsigned int b, i, m; unsigned int b, i, m;
int retval; int retval;
pr_debug("Loading module mxser ...\n");
mxvar_sdriver = alloc_tty_driver(MXSER_PORTS + 1); mxvar_sdriver = alloc_tty_driver(MXSER_PORTS + 1);
if (!mxvar_sdriver) if (!mxvar_sdriver)
return -ENOMEM; return -ENOMEM;
...@@ -2829,8 +2819,6 @@ static int __init mxser_module_init(void) ...@@ -2829,8 +2819,6 @@ static int __init mxser_module_init(void)
} /* else: we have some ISA cards under control */ } /* else: we have some ISA cards under control */
} }
pr_debug("Done.\n");
return 0; return 0;
err_unr: err_unr:
tty_unregister_driver(mxvar_sdriver); tty_unregister_driver(mxvar_sdriver);
...@@ -2843,8 +2831,6 @@ static void __exit mxser_module_exit(void) ...@@ -2843,8 +2831,6 @@ static void __exit mxser_module_exit(void)
{ {
unsigned int i, j; unsigned int i, j;
pr_debug("Unloading module mxser ...\n");
pci_unregister_driver(&mxser_driver); pci_unregister_driver(&mxser_driver);
for (i = 0; i < MXSER_BOARDS; i++) /* ISA remains */ for (i = 0; i < MXSER_BOARDS; i++) /* ISA remains */
...@@ -2858,8 +2844,6 @@ static void __exit mxser_module_exit(void) ...@@ -2858,8 +2844,6 @@ static void __exit mxser_module_exit(void)
for (i = 0; i < MXSER_BOARDS; i++) for (i = 0; i < MXSER_BOARDS; i++)
if (mxser_boards[i].info != NULL) if (mxser_boards[i].info != NULL)
mxser_release_res(&mxser_boards[i], NULL, 1); mxser_release_res(&mxser_boards[i], NULL, 1);
pr_debug("Done.\n");
} }
module_init(mxser_module_init); module_init(mxser_module_init);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment