Subject: [PATCH] input: misc: Kconfig: use select instead of depends on KEYBOARD_GPIO

From: Enrico Weigelt <[email protected]>

Make selection of the driver a bit easier by replacing depends on
KEYBOARD_GPIO by select, so we don't need to enable KEYBOARD_GPIO
before being able to enable INPUT_SOC_BUTTON_ARRAY.

Signed-off-by: Enrico Weigelt, metux IT consult <[email protected]>
---
drivers/input/misc/Kconfig | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
index 7d9ae394e597..99d263177d8d 100644
--- a/drivers/input/misc/Kconfig
+++ b/drivers/input/misc/Kconfig
@@ -813,7 +813,8 @@ config INPUT_IDEAPAD_SLIDEBAR

config INPUT_SOC_BUTTON_ARRAY
tristate "Windows-compatible SoC Button Array"
- depends on KEYBOARD_GPIO && ACPI
+ depends on ACPI
+ select KEYBOARD_GPIO
help
Say Y here if you have a SoC-based tablet that originally runs
Windows 8 or a Microsoft Surface Book 2, Pro 5, Laptop 1 or later.
--
2.11.0


2019-11-22 20:01:45

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH] input: misc: Kconfig: use select instead of depends on KEYBOARD_GPIO

On Thu, Nov 21, 2019 at 02:20:46PM +0100, Enrico Weigelt, metux IT consult wrote:
> From: Enrico Weigelt <[email protected]>
>
> Make selection of the driver a bit easier by replacing depends on
> KEYBOARD_GPIO by select, so we don't need to enable KEYBOARD_GPIO
> before being able to enable INPUT_SOC_BUTTON_ARRAY.
>
> Signed-off-by: Enrico Weigelt, metux IT consult <[email protected]>
> ---
> drivers/input/misc/Kconfig | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig
> index 7d9ae394e597..99d263177d8d 100644
> --- a/drivers/input/misc/Kconfig
> +++ b/drivers/input/misc/Kconfig
> @@ -813,7 +813,8 @@ config INPUT_IDEAPAD_SLIDEBAR
>
> config INPUT_SOC_BUTTON_ARRAY
> tristate "Windows-compatible SoC Button Array"
> - depends on KEYBOARD_GPIO && ACPI
> + depends on ACPI
> + select KEYBOARD_GPIO

KEYBOARD_GPIO depends on GPIOLIB and "select" does not propagate
dependencies, so you may still end up with invalid configs...

Thanks.

--
Dmitry

Subject: Re: [PATCH] input: misc: Kconfig: use select instead of depends on KEYBOARD_GPIO

On 22.11.19 20:58, Dmitry Torokhov wrote:

> KEYBOARD_GPIO depends on GPIOLIB and "select" does not propagate
> dependencies, so you may still end up with invalid configs...

Sure that they don't propagate ? In this case, I've got yet
another todo ... :o


--mtx

--
Dringender Hinweis: aufgrund existenzieller Bedrohung durch "Emotet"
sollten Sie *niemals* MS-Office-Dokumente via E-Mail annehmen/öffenen,
selbst wenn diese von vermeintlich vertrauenswürdigen Absendern zu
stammen scheinen. Andernfalls droht Totalschaden.
---
Hinweis: unverschlüsselte E-Mails können leicht abgehört und manipuliert
werden ! Für eine vertrauliche Kommunikation senden Sie bitte ihren
GPG/PGP-Schlüssel zu.
---
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
[email protected] -- +49-151-27565287

2019-12-02 20:25:07

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: [PATCH] input: misc: Kconfig: use select instead of depends on KEYBOARD_GPIO

On Thu, Nov 28, 2019 at 02:03:40PM +0100, Enrico Weigelt, metux IT consult wrote:
> On 22.11.19 20:58, Dmitry Torokhov wrote:
>
> > KEYBOARD_GPIO depends on GPIOLIB and "select" does not propagate
> > dependencies, so you may still end up with invalid configs...
>
> Sure that they don't propagate ? In this case, I've got yet
> another todo ... :o

Yes, I am sure.

--
Dmitry