linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] gpio: fold the Kconfig prompt into the option type for GPIO CDEV
@ 2024-10-14 11:07 Bartosz Golaszewski
  2024-10-15  8:02 ` Linus Walleij
  2024-10-17  6:57 ` Bartosz Golaszewski
  0 siblings, 2 replies; 3+ messages in thread
From: Bartosz Golaszewski @ 2024-10-14 11:07 UTC (permalink / raw)
  To: Linus Walleij, Kent Gibson; +Cc: linux-gpio, linux-kernel, Bartosz Golaszewski

From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

The 'if !EXPERT' bit will work fine if it follows the 'bool "foobar"'
declaration. No need to have separate entries for it.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
---
 drivers/gpio/Kconfig | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index bfa6b5a2c537..7f59e861de5b 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -70,8 +70,7 @@ config GPIO_SYSFS
 	  ioctl() operations instead.
 
 config GPIO_CDEV
-	bool
-	prompt "Character device (/dev/gpiochipN) support" if EXPERT
+	bool "Character device (/dev/gpiochipN) support" if EXPERT
 	default y
 	help
 	  Say Y here to add the character device /dev/gpiochipN interface
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] gpio: fold the Kconfig prompt into the option type for GPIO CDEV
  2024-10-14 11:07 [PATCH] gpio: fold the Kconfig prompt into the option type for GPIO CDEV Bartosz Golaszewski
@ 2024-10-15  8:02 ` Linus Walleij
  2024-10-17  6:57 ` Bartosz Golaszewski
  1 sibling, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2024-10-15  8:02 UTC (permalink / raw)
  To: Bartosz Golaszewski
  Cc: Kent Gibson, linux-gpio, linux-kernel, Bartosz Golaszewski

On Mon, Oct 14, 2024 at 1:07 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:

> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
>
> The 'if !EXPERT' bit will work fine if it follows the 'bool "foobar"'

Nit: if EXPERT

No negation.

> declaration. No need to have separate entries for it.
>
> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

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

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] gpio: fold the Kconfig prompt into the option type for GPIO CDEV
  2024-10-14 11:07 [PATCH] gpio: fold the Kconfig prompt into the option type for GPIO CDEV Bartosz Golaszewski
  2024-10-15  8:02 ` Linus Walleij
@ 2024-10-17  6:57 ` Bartosz Golaszewski
  1 sibling, 0 replies; 3+ messages in thread
From: Bartosz Golaszewski @ 2024-10-17  6:57 UTC (permalink / raw)
  To: Linus Walleij, Kent Gibson, Bartosz Golaszewski
  Cc: Bartosz Golaszewski, linux-gpio, linux-kernel

From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>


On Mon, 14 Oct 2024 13:07:07 +0200, Bartosz Golaszewski wrote:
> The 'if !EXPERT' bit will work fine if it follows the 'bool "foobar"'
> declaration. No need to have separate entries for it.
> 
> 

Applied, thanks!

[1/1] gpio: fold the Kconfig prompt into the option type for GPIO CDEV
      commit: 3dcf0643fa5cd65e667057cfa11d6e54ac36f2ff

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-10-17  6:57 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-14 11:07 [PATCH] gpio: fold the Kconfig prompt into the option type for GPIO CDEV Bartosz Golaszewski
2024-10-15  8:02 ` Linus Walleij
2024-10-17  6:57 ` Bartosz Golaszewski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).