Thread (5 messages) 5 messages, 4 authors, 2019-01-22

Re: [PATCH v1] pinctrl: mediatek: improve Kconfig dependencies

From: Sean Wang <sean.wang@kernel.org>
Date: 2018-12-13 04:21:35
Also in: linux-devicetree, linux-gpio, linux-mediatek, lkml

It looks good to me. Thanks for bringing the improved version soon.

Acked-by: Sean Wang <sean.wang@kernel.org>

On Wed, Dec 12, 2018 at 6:28 PM Ryder Lee [off-list ref] wrote:
quoted hunk ↗ jump to hunk
Remove prompts to make all pinctrl cores to non-visible symbols and
make sure the target SoCs would be coupled with the corresponding
cores.

Signed-off-by: Ryder Lee <ryder.lee@mediatek.com>
---
Changes since v1:
- Revise commit message
- Change all pinctrl cores to non-visible symbols
- Drop EINT_MTK in PINCTRL_MTK_MOORE
---
 drivers/pinctrl/mediatek/Kconfig | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/pinctrl/mediatek/Kconfig b/drivers/pinctrl/mediatek/Kconfig
index d8cb584..1817786 100644
--- a/drivers/pinctrl/mediatek/Kconfig
+++ b/drivers/pinctrl/mediatek/Kconfig
@@ -17,7 +17,7 @@ config PINCTRL_MTK
        select OF_GPIO

 config PINCTRL_MTK_MOORE
-       bool "MediaTek Moore Core that implements generic binding"
+       bool
        depends on OF
        select GENERIC_PINCONF
        select GENERIC_PINCTRL_GROUPS
@@ -26,7 +26,7 @@ config PINCTRL_MTK_MOORE
        select OF_GPIO

 config PINCTRL_MTK_PARIS
-       bool "MediaTek Paris Core that implements vendor binding"
+       bool
        depends on OF
        select PINMUX
        select GENERIC_PINCONF
@@ -45,14 +45,14 @@ config PINCTRL_MT2701
 config PINCTRL_MT7623
        bool "Mediatek MT7623 pin control with generic binding"
        depends on MACH_MT7623 || COMPILE_TEST
-       depends on PINCTRL_MTK_MOORE
-       default y
+       default MACH_MT7623
+       select PINCTRL_MTK_MOORE

 config PINCTRL_MT7629
        bool "Mediatek MT7629 pin control"
        depends on MACH_MT7629 || COMPILE_TEST
-       depends on PINCTRL_MTK_MOORE
-       default y
+       default MACH_MT7629
+       select PINCTRL_MTK_MOORE

 config PINCTRL_MT8135
        bool "Mediatek MT8135 pin control"
@@ -93,8 +93,8 @@ config PINCTRL_MT6797
 config PINCTRL_MT7622
        bool "MediaTek MT7622 pin control"
        depends on ARM64 || COMPILE_TEST
-       depends on PINCTRL_MTK_MOORE
-       default y
+       default ARM64 && ARCH_MEDIATEK
+       select PINCTRL_MTK_MOORE

 config PINCTRL_MT8173
        bool "Mediatek MT8173 pin control"
--
1.9.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help