2021-09-18 12:12:29

by Cai,Huoqing

[permalink] [raw]
Subject: [PATCH v2] pinctrl: intel: Kconfig: Add config menu to intel pinctrl

Adding a config menu to hold many intel pinctrl drivers
helps to make the display more concise.

Signed-off-by: Cai Huoqing <[email protected]>
---
v1->v2: Revert to "select PINCTRL_INTE" for PINCTRL_TIGERLAKE config.

drivers/pinctrl/intel/Kconfig | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/pinctrl/intel/Kconfig b/drivers/pinctrl/intel/Kconfig
index fb1495bd77c4..64f8b37fd705 100644
--- a/drivers/pinctrl/intel/Kconfig
+++ b/drivers/pinctrl/intel/Kconfig
@@ -1,7 +1,7 @@
# SPDX-License-Identifier: GPL-2.0
# Intel pin control drivers
-
-if (X86 || COMPILE_TEST)
+menu "Intel pinctrl drivers"
+ depends on X86 || COMPILE_TEST

config PINCTRL_BAYTRAIL
bool "Intel Baytrail GPIO pin control"
@@ -168,4 +168,5 @@ config PINCTRL_TIGERLAKE
This pinctrl driver provides an interface that allows configuring
of Intel Tiger Lake PCH pins and using them as GPIOs.

-endif
+endmenu
+
--
2.25.1


2021-09-18 12:13:42

by Andy Shevchenko

[permalink] [raw]
Subject: Re: [PATCH v2] pinctrl: intel: Kconfig: Add config menu to intel pinctrl

On Sat, Sep 18, 2021 at 6:17 AM Cai Huoqing <[email protected]> wrote:
>
> Adding a config menu to hold many intel pinctrl drivers

configuration
Intel
pin control

> helps to make the display more concise.

After above addressed,
Acked-by: Andy Shevchenko <[email protected]>

> Signed-off-by: Cai Huoqing <[email protected]>
> ---
> v1->v2: Revert to "select PINCTRL_INTE" for PINCTRL_TIGERLAKE config.
>
> drivers/pinctrl/intel/Kconfig | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/pinctrl/intel/Kconfig b/drivers/pinctrl/intel/Kconfig
> index fb1495bd77c4..64f8b37fd705 100644
> --- a/drivers/pinctrl/intel/Kconfig
> +++ b/drivers/pinctrl/intel/Kconfig
> @@ -1,7 +1,7 @@
> # SPDX-License-Identifier: GPL-2.0
> # Intel pin control drivers
> -
> -if (X86 || COMPILE_TEST)
> +menu "Intel pinctrl drivers"
> + depends on X86 || COMPILE_TEST
>
> config PINCTRL_BAYTRAIL
> bool "Intel Baytrail GPIO pin control"
> @@ -168,4 +168,5 @@ config PINCTRL_TIGERLAKE
> This pinctrl driver provides an interface that allows configuring
> of Intel Tiger Lake PCH pins and using them as GPIOs.
>
> -endif
> +endmenu
> +
> --
> 2.25.1
>


--
With Best Regards,
Andy Shevchenko