arm64: dts: qcom: sm8350: fix number of pins in 'gpio-ranges'
authorShawn Guo <shawn.guo@linaro.org>
Wed, 3 Mar 2021 03:31:06 +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: b7e8f433a673 ("arm64: dts: qcom: Add basic devicetree support for SM8350 SoC")
Cc: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Link: https://lore.kernel.org/r/20210303033106.549-5-shawn.guo@linaro.org
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
arch/arm64/boot/dts/qcom/sm8350.dtsi

index 2c02f451379b9d9efa566125e6c6a3721e1e4650..11401ea0c16c10fb9ca0cf12f845b6091b3aa554 100644 (file)
                        #gpio-cells = <2>;
                        interrupt-controller;
                        #interrupt-cells = <2>;
-                       gpio-ranges = <&tlmm 0 0 203>;
+                       gpio-ranges = <&tlmm 0 0 204>;
 
                        qup_uart3_default_state: qup-uart3-default-state {
                                rx {