serial: mctrl_gpio: Fix passing zero to 'ERR_PTR' warning
authorYueHaibing <yuehaibing@huawei.com>
Sat, 31 Oct 2020 03:05:30 +0000 (11:05 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Nov 2020 16:12:05 +0000 (17:12 +0100)
drivers/tty/serial/serial_mctrl_gpio.c:214
 mctrl_gpio_init() warn: passing zero to 'ERR_PTR'

gpiod_to_irq() never return 0, so remove the useless test
and make code more clear.

Reviewed-by: Jiri Slaby <jirislaby@kernel.org>
Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Link: https://lore.kernel.org/r/20201031030530.1304-1-yuehaibing@huawei.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/serial_mctrl_gpio.c

index fb47812..c41d891 100644 (file)
@@ -207,7 +207,7 @@ struct mctrl_gpios *mctrl_gpio_init(struct uart_port *port, unsigned int idx)
                        continue;
 
                ret = gpiod_to_irq(gpios->gpio[i]);
-               if (ret <= 0) {
+               if (ret < 0) {
                        dev_err(port->dev,
                                "failed to find corresponding irq for %s (idx=%d, err=%d)\n",
                                mctrl_gpios_desc[i].name, idx, ret);