| Message ID | 20250409-livetree-fixup-v1-6-76dfea80b07f@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Qualcomm: cleanup OF_LIVE fixup and fix RB1/2 | expand |
On 09/04/2025 19:17, Caleb Connolly wrote: > There are 134 pins not 133, oops! This fixes the sdcard on the RB1 as > the pins now all get configured correctly. > > Fixes: 0ecb8cfcb930 ("pinctrl: qcom: add qcm2290 pinctrl driver") > Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> > --- > drivers/pinctrl/qcom/pinctrl-qcm2290.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pinctrl/qcom/pinctrl-qcm2290.c b/drivers/pinctrl/qcom/pinctrl-qcm2290.c > index 0c2222ce663e6d584d229e7521f88fedf8aa19da..84f76b63b93ad78182524661dba561672feb4c85 100644 > --- a/drivers/pinctrl/qcom/pinctrl-qcm2290.c > +++ b/drivers/pinctrl/qcom/pinctrl-qcm2290.c > @@ -44,9 +44,9 @@ static int qcm2290_get_function_mux(__maybe_unused unsigned int pin, unsigned in > } > > struct msm_pinctrl_data qcm2290_data = { > .pin_data = { > - .pin_count = 133, > + .pin_count = 134, > .special_pins_start = 127, > }, > .functions_count = ARRAY_SIZE(msm_pinctrl_functions), > .get_function_name = qcm2290_get_function_name, > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
On Wed, Apr 09, 2025 at 07:17:29PM +0200, Caleb Connolly wrote: > There are 134 pins not 133, oops! This fixes the sdcard on the RB1 as > the pins now all get configured correctly. > > Fixes: 0ecb8cfcb930 ("pinctrl: qcom: add qcm2290 pinctrl driver") > Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> > --- > drivers/pinctrl/qcom/pinctrl-qcm2290.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Nice catch! Reviewed-by: Sumit Garg <sumit.garg@oss.qualcomm.com> -Sumit > diff --git a/drivers/pinctrl/qcom/pinctrl-qcm2290.c b/drivers/pinctrl/qcom/pinctrl-qcm2290.c > index 0c2222ce663e6d584d229e7521f88fedf8aa19da..84f76b63b93ad78182524661dba561672feb4c85 100644 > --- a/drivers/pinctrl/qcom/pinctrl-qcm2290.c > +++ b/drivers/pinctrl/qcom/pinctrl-qcm2290.c > @@ -44,9 +44,9 @@ static int qcm2290_get_function_mux(__maybe_unused unsigned int pin, unsigned in > } > > struct msm_pinctrl_data qcm2290_data = { > .pin_data = { > - .pin_count = 133, > + .pin_count = 134, > .special_pins_start = 127, > }, > .functions_count = ARRAY_SIZE(msm_pinctrl_functions), > .get_function_name = qcm2290_get_function_name, > > -- > 2.49.0 >
diff --git a/drivers/pinctrl/qcom/pinctrl-qcm2290.c b/drivers/pinctrl/qcom/pinctrl-qcm2290.c index 0c2222ce663e6d584d229e7521f88fedf8aa19da..84f76b63b93ad78182524661dba561672feb4c85 100644 --- a/drivers/pinctrl/qcom/pinctrl-qcm2290.c +++ b/drivers/pinctrl/qcom/pinctrl-qcm2290.c @@ -44,9 +44,9 @@ static int qcm2290_get_function_mux(__maybe_unused unsigned int pin, unsigned in } struct msm_pinctrl_data qcm2290_data = { .pin_data = { - .pin_count = 133, + .pin_count = 134, .special_pins_start = 127, }, .functions_count = ARRAY_SIZE(msm_pinctrl_functions), .get_function_name = qcm2290_get_function_name,
There are 134 pins not 133, oops! This fixes the sdcard on the RB1 as the pins now all get configured correctly. Fixes: 0ecb8cfcb930 ("pinctrl: qcom: add qcm2290 pinctrl driver") Signed-off-by: Caleb Connolly <caleb.connolly@linaro.org> --- drivers/pinctrl/qcom/pinctrl-qcm2290.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)