pinctrl: aw9523: Use correct error code for not supported functionality
authorAndy Shevchenko <andy.shevchenko@gmail.com>
Fri, 29 Mar 2024 10:55:16 +0000 (12:55 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Thu, 4 Apr 2024 11:16:46 +0000 (13:16 +0200)
The pin control subsystem internally uses ENOTSUPP for the not supported
functionality. The checkpatch is false positive about this error code.

Signed-off-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Message-ID: <20240329105634.712457-3-andy.shevchenko@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-aw9523.c

index 66629af0b88b4bbf877f61bde83882fada97b16d..65d523697b7311751188586041767987bfcdbefb 100644 (file)
@@ -6,6 +6,7 @@
  */
 
 #include <linux/bitfield.h>
+#include <linux/errno.h>
 #include <linux/gpio/consumer.h>
 #include <linux/gpio/driver.h>
 #include <linux/i2c.h>
@@ -239,7 +240,7 @@ static int aw9523_pcfg_param_to_reg(enum pin_config_param pcp, int pin, u8 *r)
                reg = AW9523_REG_OUT_STATE(pin);
                break;
        default:
-               return -EOPNOTSUPP;
+               return -ENOTSUPP;
        }
        *r = reg;
 
@@ -290,7 +291,7 @@ static int aw9523_pconf_get(struct pinctrl_dev *pctldev, unsigned int pin,
                        val = FIELD_GET(AW9523_GCR_GPOMD_MASK, val);
                break;
        default:
-               return -EOPNOTSUPP;
+               return -ENOTSUPP;
        }
        if (val < 1)
                return -EINVAL;
@@ -344,7 +345,7 @@ static int aw9523_pconf_set(struct pinctrl_dev *pctldev, unsigned int pin,
                case PIN_CONFIG_DRIVE_OPEN_DRAIN:
                        /* Open-Drain is supported only on port 0 */
                        if (pin >= AW9523_PINS_PER_PORT) {
-                               rc = -EOPNOTSUPP;
+                               rc = -ENOTSUPP;
                                goto end;
                        }
                        mask = AW9523_GCR_GPOMD_MASK;
@@ -361,7 +362,7 @@ static int aw9523_pconf_set(struct pinctrl_dev *pctldev, unsigned int pin,
                        val = AW9523_GCR_GPOMD_MASK;
                        break;
                default:
-                       rc = -EOPNOTSUPP;
+                       rc = -ENOTSUPP;
                        goto end;
                }