Skip to contentSkip to conversation

pinctrl: qcom: Fix msm8953 Kconfig entry to depend on, not select PINCTRL_MSM

PATCH Applied John Stultz 4 participants 4 messages Nov 10, 2020 – Dec 29, 2020

Subsystems (5)
Arm Qualcomm mailing list GPIO Pin control Pin controller - Qualcomm +1 more subsystems
lore
Share & tools
lore

John Stultz

author
#0 lore raw

PATCH v2 1/1 pinctrl: qcom: Fix msm8953 Kconfig entry to depend on, not select PINCTRL_MSM

One fixup following my patch commit be117ca32261 ("pinctrl:
qcom: Kconfig: Rework PINCTRL_MSM to be a depenency rather then a selected config") being queued in LinusW's tree, as a new config entry was added for the msm8953 that also needs the change.

Applies to LinusW's pinctrl devel tree.

Cc Andy Gross <[email protected]>
Cc Prasad Sodagudi <[email protected]>
Cc Vladimir Lypak <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Linus Walleij <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by John Stultz <[email protected]>
Change-Id I9e8e83b1ea57aff338074be9174fce53cef29eff

---
v2:
* Fix flipped numbers in the soc name, pointed out by Jeffrey Hugo

---
 drivers/pinctrl/qcom/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/pinctrl/qcom/Kconfig +1 -1
index 8bdf878fe970c..cf56e029cd9c7 100644
@@ -115,7 +115,7 @@ config PINCTRL_MSM8916
 config PINCTRL_MSM8953
 	tristate "Qualcomm 8953 pin controller driver"
 	depends on GPIOLIB && OF
-	select PINCTRL_MSM
+	depends on PINCTRL_MSM
 	help
 	  This is the pinctrl, pinmux, pinconf and gpiolib driver for the
 	  Qualcomm TLMM block found on the Qualcomm MSM8953 platform.
Signature · 1 line
2.17.1
Reviewed-by on v2 1/1
#1 lore raw
Quoted 8 lines · John Stultz

One fixup following my patch commit be117ca32261 ("pinctrl:
qcom: Kconfig: Rework PINCTRL_MSM to be a depenency rather then a selected config") being queued in LinusW's tree, as a new config entry was added for the msm8953 that also needs the change.

Applies to LinusW's pinctrl devel tree.

Quoted 9 lines
Cc Andy Gross <[email protected]>
Cc Prasad Sodagudi <[email protected]>
Cc Vladimir Lypak <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Linus Walleij <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by John Stultz <[email protected]>
Change-Id I9e8e83b1ea57aff338074be9174fce53cef29eff

Linus, please drop the change-id as you apply this.

Reviewed-by Bjorn Andersson <[email protected]>

Regards,
Bjorn

Quoted 24 lines

---
v2:
* Fix flipped numbers in the soc name, pointed out by Jeffrey Hugo

---
 drivers/pinctrl/qcom/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/pinctrl/qcom/Kconfig +1 -1
index 8bdf878fe970c..cf56e029cd9c7 100644
@@ -115,7 +115,7 @@ config PINCTRL_MSM8916
 config PINCTRL_MSM8953
 	tristate "Qualcomm 8953 pin controller driver"
 	depends on GPIOLIB && OF
-	select PINCTRL_MSM
+	depends on PINCTRL_MSM
 	help
 	  This is the pinctrl, pinmux, pinconf and gpiolib driver for the
 	  Qualcomm TLMM block found on the Qualcomm MSM8953 platform.
Signature · 1 line
2.17.1
#2 lore raw
In reply to John Stultz
Quoted 17 lines · John Stultz

One fixup following my patch commit be117ca32261 ("pinctrl:
qcom: Kconfig: Rework PINCTRL_MSM to be a depenency rather then a selected config") being queued in LinusW's tree, as a new config entry was added for the msm8953 that also needs the change.

Applies to LinusW's pinctrl devel tree.

Cc Andy Gross <[email protected]>
Cc Prasad Sodagudi <[email protected]>
Cc Vladimir Lypak <[email protected]>
Cc Bjorn Andersson <[email protected]>
Cc Linus Walleij <[email protected]>
Cc [email protected]
Cc [email protected]
Signed-off-by John Stultz <[email protected]>
Change-Id I9e8e83b1ea57aff338074be9174fce53cef29eff
Quoted 4 lines

---
v2:
* Fix flipped numbers in the soc name, pointed out by Jeffrey Hugo

Patch applied, for some reason this got lost in my inbox.

Yours,
Linus Walleij

CI Applied
#3 lore raw
In reply to John Stultz
This patch was applied to qcom/linux.git (refs/heads/for-next):

Hello:

This patch was applied to qcom/linux.git (refs/heads/for-next):

Quoted 9 lines · On Tue, 10 Nov 2020 21:56:19 +0000 you wrote:

One fixup following my patch commit be117ca32261 ("pinctrl:
qcom: Kconfig: Rework PINCTRL_MSM to be a depenency rather then a selected config") being queued in LinusW's tree, as a new config entry was added for the msm8953 that also needs the change.

Applies to LinusW's pinctrl devel tree.

[...]

Here is the summary with links:
- [v2] pinctrl: qcom: Fix msm8953 Kconfig entry to depend on, not select PINCTRL_MSM

    https://git.kernel.org/qcom/c/2a984219b534

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html