serial: sa1100: add note about modem control signals
authorRussell King <rmk+kernel@armlinux.org.uk>
Fri, 31 May 2019 15:21:28 +0000 (16:21 +0100)
committerRussell King <rmk+kernel@armlinux.org.uk>
Tue, 4 Jun 2019 11:56:33 +0000 (12:56 +0100)
As suggested by Uwe, add a note indicating that the modem control
signals do not support interrupts, which precludes the driver from
using mctrl_gpio_init().

Suggested-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
drivers/tty/serial/sa1100.c

index 97bdfec..8e61812 100644 (file)
@@ -860,6 +860,10 @@ static int sa1100_serial_resume(struct platform_device *dev)
 static int sa1100_serial_add_one_port(struct sa1100_port *sport, struct platform_device *dev)
 {
        sport->port.dev = &dev->dev;
+
+       // mctrl_gpio_init() requires that the GPIO driver supports interrupts,
+       // but we need to support GPIO drivers for hardware that has no such
+       // interrupts.  Use mctrl_gpio_init_noauto() instead.
        sport->gpios = mctrl_gpio_init_noauto(sport->port.dev, 0);
        if (IS_ERR(sport->gpios)) {
                int err = PTR_ERR(sport->gpios);