]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
pinctrl: airoha: Fix return value in pinconf callbacks
authorLorenzo Bianconi <lorenzo@kernel.org>
Fri, 22 Aug 2025 12:14:18 +0000 (14:14 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 4 Sep 2025 14:55:29 +0000 (16:55 +0200)
[ Upstream commit 563fcd6475931c5c8c652a4dd548256314cc87ed ]

Pinctrl stack requires ENOTSUPP error code if the parameter is not
supported by the pinctrl driver. Fix the returned error code in pinconf
callbacks if the operation is not supported.

Fixes: 1c8ace2d0725 ("pinctrl: airoha: Add support for EN7581 SoC")
Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Link: https://lore.kernel.org/20250822-airoha-pinconf-err-val-fix-v1-1-87b4f264ced2@kernel.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/pinctrl/mediatek/pinctrl-airoha.c

index b97b28ebb37a6ec092360f8ea404dd67e6c43eac..3fa5131d81e525079ac75f32d6b6e46247a5f19f 100644 (file)
@@ -2696,7 +2696,7 @@ static int airoha_pinconf_get(struct pinctrl_dev *pctrl_dev,
                arg = 1;
                break;
        default:
-               return -EOPNOTSUPP;
+               return -ENOTSUPP;
        }
 
        *config = pinconf_to_config_packed(param, arg);
@@ -2788,7 +2788,7 @@ static int airoha_pinconf_set(struct pinctrl_dev *pctrl_dev,
                        break;
                }
                default:
-                       return -EOPNOTSUPP;
+                       return -ENOTSUPP;
                }
        }
 
@@ -2805,10 +2805,10 @@ static int airoha_pinconf_group_get(struct pinctrl_dev *pctrl_dev,
                if (airoha_pinconf_get(pctrl_dev,
                                       airoha_pinctrl_groups[group].pins[i],
                                       config))
-                       return -EOPNOTSUPP;
+                       return -ENOTSUPP;
 
                if (i && cur_config != *config)
-                       return -EOPNOTSUPP;
+                       return -ENOTSUPP;
 
                cur_config = *config;
        }