Commit 62209e80 authored Nov 04, 2021 by Bjorn Andersson Committed by Linus Walleij Nov 16, 2021

pinctrl: qcom: sm8350: Correct UFS and SDC offsets



The downstream TLMM binding covers a group of TLMM-related hardware
blocks, but the upstream binding only captures the particular block
related to controlling the TLMM pins from an OS. In the translation of
the driver from downstream, the offset of 0x100000 was lost for the UFS
and SDC pingroups.

Fixes: d5d348a3 ("pinctrl: qcom: Add SM8350 pinctrl driver")
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Reviewed-by: Vinod Koul <vkoul@kernel.org>
Reviewed-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
Link: https://lore.kernel.org/r/20211104170835.1993686-1-bjorn.andersson@linaro.org


Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

parent 293083f8

drivers/pinctrl/qcom/pinctrl-sm8350.c

+4 −4

Original line number	Diff line number	Diff line
		@@ -1597,10 +1597,10 @@ static const struct msm_pingroup sm8350_groups[] = {
		[200] = PINGROUP(200, qdss_gpio, _, _, _, _, _, _, _, _),
		[201] = PINGROUP(201, _, _, _, _, _, _, _, _, _),
		[202] = PINGROUP(202, _, _, _, _, _, _, _, _, _),
		[203] = UFS_RESET(ufs_reset, 0x1d8000),
		[204] = SDC_PINGROUP(sdc2_clk, 0x1cf000, 14, 6),
		[205] = SDC_PINGROUP(sdc2_cmd, 0x1cf000, 11, 3),
		[206] = SDC_PINGROUP(sdc2_data, 0x1cf000, 9, 0),
		[203] = UFS_RESET(ufs_reset, 0xd8000),
		[204] = SDC_PINGROUP(sdc2_clk, 0xcf000, 14, 6),
		[205] = SDC_PINGROUP(sdc2_cmd, 0xcf000, 11, 3),
		[206] = SDC_PINGROUP(sdc2_data, 0xcf000, 9, 0),
		};

		static const struct msm_gpio_wakeirq_map sm8350_pdc_map[] = {