summaryrefslogtreecommitdiffstats
path: root/drivers/tty
diff options
context:
space:
mode:
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>2012-06-13 00:28:24 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2012-06-12 15:56:54 -0700
commit8856a7d6b7c39eece126f23c6cdbd11ff2218d6f (patch)
tree90287af8761178fac5bb1734477b4a7b3fc5698c /drivers/tty
parent9800ee6f50a3b94181d4df57542b9379e331decb (diff)
downloadop-kernel-dev-8856a7d6b7c39eece126f23c6cdbd11ff2218d6f.zip
op-kernel-dev-8856a7d6b7c39eece126f23c6cdbd11ff2218d6f.tar.gz
serial: sh-sci: Make probe fail for ports that exceed the maximum count
The driver supports a maximum number of ports configurable at compile time. Make sure the probe() method fails when registering a port that exceeds the maximum instead of returning success without registering the port. This fixes a crash at system suspend time, when the driver tried to suspend a non-registered port using the UART core. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty')
-rw-r--r--drivers/tty/serial/sh-sci.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c
index 27df2ad..1bd9163 100644
--- a/drivers/tty/serial/sh-sci.c
+++ b/drivers/tty/serial/sh-sci.c
@@ -2391,7 +2391,7 @@ static int __devinit sci_probe_single(struct platform_device *dev,
index+1, SCI_NPORTS);
dev_notice(&dev->dev, "Consider bumping "
"CONFIG_SERIAL_SH_SCI_NR_UARTS!\n");
- return 0;
+ return -EINVAL;
}
ret = sci_init_single(dev, sciport, index, p);
OpenPOWER on IntegriCloud