On 15/08/2025 11:09, Bartosz Golaszewski wrote:
From: Bartosz Golaszewski <[email protected]>
The strict flag in struct pinmux_ops disallows the usage of the same pin
as a GPIO and for another function. Without it, a rouge user-space
process with enough privileges (or even a buggy driver) can request a
used pin as GPIO and drive it, potentially confusing devices or even
crashing the system. Set it globally for all pinctrl-msm users.
Reviewed-by: Konrad Dybcio <[email protected]>
Signed-off-by: Bartosz Golaszewski <[email protected]>
---
drivers/pinctrl/qcom/pinctrl-msm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/pinctrl/qcom/pinctrl-msm.c
b/drivers/pinctrl/qcom/pinctrl-msm.c
index
a5f69464827119dfe2a7781b558094b283fca215..1751d838ce95d6138c824b90098f74891dec7656
100644
--- a/drivers/pinctrl/qcom/pinctrl-msm.c
+++ b/drivers/pinctrl/qcom/pinctrl-msm.c
@@ -268,6 +268,7 @@ static const struct pinmux_ops msm_pinmux_ops = {
.function_is_gpio = pinmux_generic_function_is_gpio,
.gpio_request_enable = msm_pinmux_request_gpio,
.set_mux = msm_pinmux_set_mux,
+ .strict = true,
};
static int msm_config_reg(struct msm_pinctrl *pctrl,
I tested the patchset on CI, and I got the following errors:
db410c:
https://git.codelinaro.org/linaro/qcomlt/ci/staging/cdba-tester/-/jobs/253304#L537
[ 4.056081] msm8916-pinctrl 1000000.pinctrl: pin GPIO_38 already requested
by 1000000.pinctrl:550; cannot claim for 7864900.mmc
[ 4.061402] msm8916-pinctrl 1000000.pinctrl: error -EINVAL: pin-38
(7864900.mmc)
[ 4.077245] msm8916-pinctrl 1000000.pinctrl: error -EINVAL: could not
request pin 38 (GPIO_38) from group gpio38 on device 1000000.pinctrl
[ 4.088430] sdhci_msm 7864900.mmc: Error applying setting, reverse things
back
[ 4.100751] msm8916-pinctrl 1000000.pinctrl: pin GPIO_38 already requested
by 1000000.pinctrl:550; cannot claim for 7864900.mmc
[ 4.107960] msm8916-pinctrl 1000000.pinctrl: error -EINVAL: pin-38
(7864900.mmc)
[ 4.107970] msm8916-pinctrl 1000000.pinctrl: error -EINVAL: could not
request pin 38 (GPIO_38) from group gpio38 on device 1000000.pinctrl
[ 4.126971] sdhci_msm 7864900.mmc: Error applying setting, reverse things
back
[ 4.139203] sdhci_msm 7864900.mmc: failed to activate pinctrl state sleep
rb3gen2:
https://git.codelinaro.org/linaro/qcomlt/ci/staging/cdba-tester/-/jobs/253305#L1120
[ 4.155880] sc7280-pinctrl f100000.pinctrl: pin GPIO_91 already requested by
f100000.pinctrl:638; cannot claim for 8804000.mmc
[ 4.167636] sc7280-pinctrl f100000.pinctrl: error -EINVAL: pin-91
(8804000.mmc)
[ 4.179817] sc7280-pinctrl f100000.pinctrl: error -EINVAL: could not request
pin 91 (GPIO_91) from group gpio91 on device f100000.pinctrl
[ 4.196265] sdhci_msm 8804000.mmc: Error applying setting, reverse things
back
[ 4.196293] sc7280-pinctrl f100000.pinctrl: pin GPIO_91 already requested by
f100000.pinctrl:638; cannot claim for 8804000.mmc
[ 4.203735] sc7280-pinctrl f100000.pinctrl: error -EINVAL: pin-91
(8804000.mmc)
[ 4.203743] sc7280-pinctrl f100000.pinctrl: error -EINVAL: could not request
pin 91 (GPIO_91) from group gpio91 on device f100000.pinctrl
[ 4.235716] sdhci_msm 8804000.mmc: Error applying setting, reverse things
back
[ 4.235722] sdhci_msm 8804000.mmc: failed to activate pinctrl state sleep
x1-qcp:
https://git.codelinaro.org/linaro/qcomlt/ci/staging/cdba-tester/-/jobs/253310#L1776
[ 5.881549] mmc0: SDHCI controller on 8804000.mmc [8804000.mmc] using ADMA
64-bit
[ 5.889494] x1e80100-tlmm f100000.pinctrl: pin GPIO_71 already requested by
f100000.pinctrl:665; cannot claim for 8804000.mmc
[ 5.901103] x1e80100-tlmm f100000.pinctrl: error -EINVAL: pin-71
(8804000.mmc)
[ 5.913184] x1e80100-tlmm f100000.pinctrl: error -EINVAL: could not request
pin 71 (GPIO_71) from group gpio71 on device f100000.pinctrl
[ 5.913186] sdhci_msm 8804000.mmc: Error applying setting, reverse things
back
[ 5.913188] x1e80100-tlmm f100000.pinctrl: pin GPIO_71 already requested by
f100000.pinctrl:665; cannot claim for 8804000.mmc
[ 5.929538] x1e80100-tlmm f100000.pinctrl: error -EINVAL: pin-71
(8804000.mmc)
[ 5.948557] x1e80100-tlmm f100000.pinctrl: error -EINVAL: could not request
pin 71 (GPIO_71) from group gpio71 on device f100000.pinctrl
[ 5.955983] sdhci_msm 8804000.mmc: Error applying setting, reverse things
back
[ 5.975984] sdhci_msm 8804000.mmc: failed to activate pinctrl state sleep
Neil
