Commit f91eafcb authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Linus Walleij

pinctrl: aw9523: Use correct error code for not supported functionality

The pin control subsystem internally uses ENOTSUPP for the not supported
functionality. The checkpatch is false positive about this error code.
Signed-off-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Message-ID: <20240329105634.712457-3-andy.shevchenko@gmail.com>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent e5e8a580
...@@ -6,6 +6,7 @@ ...@@ -6,6 +6,7 @@
*/ */
#include <linux/bitfield.h> #include <linux/bitfield.h>
#include <linux/errno.h>
#include <linux/gpio/consumer.h> #include <linux/gpio/consumer.h>
#include <linux/gpio/driver.h> #include <linux/gpio/driver.h>
#include <linux/i2c.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) ...@@ -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); reg = AW9523_REG_OUT_STATE(pin);
break; break;
default: default:
return -EOPNOTSUPP; return -ENOTSUPP;
} }
*r = reg; *r = reg;
...@@ -290,7 +291,7 @@ static int aw9523_pconf_get(struct pinctrl_dev *pctldev, unsigned int pin, ...@@ -290,7 +291,7 @@ static int aw9523_pconf_get(struct pinctrl_dev *pctldev, unsigned int pin,
val = FIELD_GET(AW9523_GCR_GPOMD_MASK, val); val = FIELD_GET(AW9523_GCR_GPOMD_MASK, val);
break; break;
default: default:
return -EOPNOTSUPP; return -ENOTSUPP;
} }
if (val < 1) if (val < 1)
return -EINVAL; return -EINVAL;
...@@ -344,7 +345,7 @@ static int aw9523_pconf_set(struct pinctrl_dev *pctldev, unsigned int pin, ...@@ -344,7 +345,7 @@ static int aw9523_pconf_set(struct pinctrl_dev *pctldev, unsigned int pin,
case PIN_CONFIG_DRIVE_OPEN_DRAIN: case PIN_CONFIG_DRIVE_OPEN_DRAIN:
/* Open-Drain is supported only on port 0 */ /* Open-Drain is supported only on port 0 */
if (pin >= AW9523_PINS_PER_PORT) { if (pin >= AW9523_PINS_PER_PORT) {
rc = -EOPNOTSUPP; rc = -ENOTSUPP;
goto end; goto end;
} }
mask = AW9523_GCR_GPOMD_MASK; mask = AW9523_GCR_GPOMD_MASK;
...@@ -361,7 +362,7 @@ static int aw9523_pconf_set(struct pinctrl_dev *pctldev, unsigned int pin, ...@@ -361,7 +362,7 @@ static int aw9523_pconf_set(struct pinctrl_dev *pctldev, unsigned int pin,
val = AW9523_GCR_GPOMD_MASK; val = AW9523_GCR_GPOMD_MASK;
break; break;
default: default:
rc = -EOPNOTSUPP; rc = -ENOTSUPP;
goto end; goto end;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment