summaryrefslogtreecommitdiffstats
path: root/drivers/char/mxser.c
diff options
context:
space:
mode:
authorPeter Botha <peterb@goldcircle.co.za>2009-06-10 02:16:32 +0200
committerLinus Torvalds <torvalds@linux-foundation.org>2009-06-10 03:09:03 +0200
commit96050dfb25966612008dcea7d342e91fa01e993c (patch)
tree787b6337a9ef0ad018181218e9d36c883998c090 /drivers/char/mxser.c
parentjbd: fix race in buffer processing in commit code (diff)
downloadlinux-96050dfb25966612008dcea7d342e91fa01e993c.tar.xz
linux-96050dfb25966612008dcea7d342e91fa01e993c.zip
char: mxser, fix ISA board lookup
There's a bug in the mxser kernel module that still appears in the 2.6.29.4 kernel. mxser_get_ISA_conf takes a ioaddress as its first argument, by passing the not of the ioaddr, you're effectively passing 0 which means it won't be able to talk to an ISA card. I have tested this, and removing the ! fixes the problem. Cc: "Peter Botha" <peterb@goldcircle.co.za> Signed-off-by: Jiri Slaby <jirislaby@gmail.com> Acked-by: Alan Cox <alan@lxorguk.ukuu.org.uk> Cc: <stable@kernel.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to '')
-rw-r--r--drivers/char/mxser.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/mxser.c b/drivers/char/mxser.c
index a420e8d437dd..13f8871e5b21 100644
--- a/drivers/char/mxser.c
+++ b/drivers/char/mxser.c
@@ -2711,7 +2711,7 @@ static int __init mxser_module_init(void)
continue;
brd = &mxser_boards[m];
- retval = mxser_get_ISA_conf(!ioaddr[b], brd);
+ retval = mxser_get_ISA_conf(ioaddr[b], brd);
if (retval <= 0) {
brd->info = NULL;
continue;