All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO
@ 2019-04-13  8:35 Chris Packham
  2019-04-13  8:35 ` [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO Chris Packham
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Chris Packham @ 2019-04-13  8:35 UTC (permalink / raw)
  To: u-boot

CONFIG_GPIO does not exist. sysreset_gpio.c uses the DM gpio APIs so the
correct option to select is DM_GPIO.

Reported-by: Robert P. J. Day <rpjday@crashcourse.ca>
Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 drivers/sysreset/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/sysreset/Kconfig b/drivers/sysreset/Kconfig
index 8ce3e2e20761..73d141184953 100644
--- a/drivers/sysreset/Kconfig
+++ b/drivers/sysreset/Kconfig
@@ -17,7 +17,7 @@ if SYSRESET
 
 config SYSRESET_GPIO
 	bool "Enable support for GPIO reset driver"
-	select GPIO
+	select DM_GPIO
 	help
 	  Reset support via GPIO pin connected reset logic. This is used for
 	  example on Microblaze where reset logic can be controlled via GPIO
-- 
2.21.0

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

* [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO
  2019-04-13  8:35 [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO Chris Packham
@ 2019-04-13  8:35 ` Chris Packham
  2019-04-15 16:24   ` Simon Glass
  2019-05-10 15:10   ` Tom Rini
  2019-04-15 16:24 ` [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO Simon Glass
  2019-05-10 15:10 ` Tom Rini
  2 siblings, 2 replies; 6+ messages in thread
From: Chris Packham @ 2019-04-13  8:35 UTC (permalink / raw)
  To: u-boot

CONFIG_GPIO does not exist. There is one hit for it in tegra_gpio.c but
it is a variable name.

Signed-off-by: Chris Packham <judge.packham@gmail.com>
---

 scripts/config_whitelist.txt | 1 -
 1 file changed, 1 deletion(-)

diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index fa98efc24c0b..48622b12bb64 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -713,7 +713,6 @@ CONFIG_GLOBAL_TIMER
 CONFIG_GMII
 CONFIG_GOOD_SESH4
 CONFIG_GPCNTRL
-CONFIG_GPIO
 CONFIG_GPIO_ENABLE_SPI_FLASH
 CONFIG_GPIO_LED_INVERTED_TABLE
 CONFIG_GPIO_LED_STUBS
-- 
2.21.0

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

* [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO
  2019-04-13  8:35 [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO Chris Packham
  2019-04-13  8:35 ` [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO Chris Packham
@ 2019-04-15 16:24 ` Simon Glass
  2019-05-10 15:10 ` Tom Rini
  2 siblings, 0 replies; 6+ messages in thread
From: Simon Glass @ 2019-04-15 16:24 UTC (permalink / raw)
  To: u-boot

On Sat, 13 Apr 2019 at 02:35, Chris Packham <judge.packham@gmail.com> wrote:
>
> CONFIG_GPIO does not exist. sysreset_gpio.c uses the DM gpio APIs so the
> correct option to select is DM_GPIO.
>
> Reported-by: Robert P. J. Day <rpjday@crashcourse.ca>
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
>
>  drivers/sysreset/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

Reviewed-by: Simon Glass <sjg@chromium.org>

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

* [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO
  2019-04-13  8:35 ` [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO Chris Packham
@ 2019-04-15 16:24   ` Simon Glass
  2019-05-10 15:10   ` Tom Rini
  1 sibling, 0 replies; 6+ messages in thread
From: Simon Glass @ 2019-04-15 16:24 UTC (permalink / raw)
  To: u-boot

On Sat, 13 Apr 2019 at 02:35, Chris Packham <judge.packham@gmail.com> wrote:
>
> CONFIG_GPIO does not exist. There is one hit for it in tegra_gpio.c but
> it is a variable name.
>
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> ---
>
>  scripts/config_whitelist.txt | 1 -
>  1 file changed, 1 deletion(-)

Reviewed-by: Simon Glass <sjg@chromium.org>

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

* [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO
  2019-04-13  8:35 [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO Chris Packham
  2019-04-13  8:35 ` [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO Chris Packham
  2019-04-15 16:24 ` [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO Simon Glass
@ 2019-05-10 15:10 ` Tom Rini
  2 siblings, 0 replies; 6+ messages in thread
From: Tom Rini @ 2019-05-10 15:10 UTC (permalink / raw)
  To: u-boot

On Sat, Apr 13, 2019 at 08:35:23PM +1200, Chris Packham wrote:

> CONFIG_GPIO does not exist. sysreset_gpio.c uses the DM gpio APIs so the
> correct option to select is DM_GPIO.
> 
> Reported-by: Robert P. J. Day <rpjday@crashcourse.ca>
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190510/2decb6d2/attachment.sig>

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

* [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO
  2019-04-13  8:35 ` [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO Chris Packham
  2019-04-15 16:24   ` Simon Glass
@ 2019-05-10 15:10   ` Tom Rini
  1 sibling, 0 replies; 6+ messages in thread
From: Tom Rini @ 2019-05-10 15:10 UTC (permalink / raw)
  To: u-boot

On Sat, Apr 13, 2019 at 08:35:24PM +1200, Chris Packham wrote:

> CONFIG_GPIO does not exist. There is one hit for it in tegra_gpio.c but
> it is a variable name.
> 
> Signed-off-by: Chris Packham <judge.packham@gmail.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190510/cfd02bc9/attachment.sig>

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

end of thread, other threads:[~2019-05-10 15:10 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-13  8:35 [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO Chris Packham
2019-04-13  8:35 ` [U-Boot] [PATCH 2/2] Remove whitelist entry for CONFIG_GPIO Chris Packham
2019-04-15 16:24   ` Simon Glass
2019-05-10 15:10   ` Tom Rini
2019-04-15 16:24 ` [U-Boot] [PATCH 1/2] sysreset: select DM_GPIO instead of GPIO Simon Glass
2019-05-10 15:10 ` Tom Rini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.