arm64: dts: qcom: sdm845: fix number of pins in 'gpio-ranges'
authorShawn Guo <shawn.guo@linaro.org>
Wed, 3 Mar 2021 03:31:03 +0000 (11:31 +0800)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Fri, 12 Mar 2021 02:22:43 +0000 (20:22 -0600)
The last cell of 'gpio-ranges' should be number of GPIO pins, and in
case of qcom platform it should match msm_pinctrl_soc_data.ngpio rather
than msm_pinctrl_soc_data.ngpio - 1.

This fixes the problem that when the last GPIO pin in the range is
configured with the following call sequence, it always fails with
-EPROBE_DEFER.

    pinctrl_gpio_set_config()
        pinctrl_get_device_gpio_range()
            pinctrl_match_gpio_range()

Fixes: bc2c806293c6 ("arm64: dts: qcom: sdm845: Add gpio-ranges to TLMM node")
Cc: Evan Green <evgreen@chromium.org>
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Link: https://lore.kernel.org/r/20210303033106.549-2-shawn.guo@linaro.org
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
arch/arm64/boot/dts/qcom/sdm845.dtsi

index 8acc731b58a3a4bf44cfe7dcb29a450a01ae38c6..874990522b42c0d94cbbdab7e020648eea564206 100644 (file)
                        #gpio-cells = <2>;
                        interrupt-controller;
                        #interrupt-cells = <2>;
-                       gpio-ranges = <&tlmm 0 0 150>;
+                       gpio-ranges = <&tlmm 0 0 151>;
                        wakeup-parent = <&pdc_intc>;
 
                        cci0_default: cci0-default {