aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/wan
diff options
context:
space:
mode:
authorJiri Slaby <jslaby@suse.cz>2010-11-24 18:27:55 -0500
committerGreg Kroah-Hartman <gregkh@suse.de>2010-11-29 17:51:54 -0500
commit6a20bd45fda271bd57e3ba41b3de3009797f4b5e (patch)
treed0699ec589106045e082b8fd7035860b616d1cbd /drivers/net/wan
parent7f90cfc505d613f4faf096e0d84ffe99208057d9 (diff)
NET: wan/x25, fix ldisc->open retval
We should never return positive values from ldisc->open, tty layer doesn't (and never did) expect that. If we do that, weird things like warnings in tty_ldisc_close happen. Not sure if dev->base_addr is used somehow now. Signed-off-by: Jiri Slaby <jslaby@suse.cz> Cc: Alan Cox <alan@linux.intel.com> Cc: Andrew Hendry <andrew.hendry@gmail.com> Cc: linux-x25@vger.kernel.org Tested-by: Sergey Lapin <slapin@ossfans.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/net/wan')
-rw-r--r--drivers/net/wan/x25_asy.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wan/x25_asy.c b/drivers/net/wan/x25_asy.c
index d81ad8397885..66cda25aaa26 100644
--- a/drivers/net/wan/x25_asy.c
+++ b/drivers/net/wan/x25_asy.c
@@ -582,7 +582,7 @@ static int x25_asy_open_tty(struct tty_struct *tty)
582 if (err) 582 if (err)
583 return err; 583 return err;
584 /* Done. We have linked the TTY line to a channel. */ 584 /* Done. We have linked the TTY line to a channel. */
585 return sl->dev->base_addr; 585 return 0;
586} 586}
587 587
588 588