Re: [PATCH 10/14] watchdog: convert the Kconfig dependency on OF_GPIO to OF
Guenter Roeck <[email protected]> Fri, 6 Mar 2026 15:33:36 -0800
| Newsgroups | org.infradead.lists.linux-snps-arc,dev.linux.lists.linux-staging,org.infradead.lists.linux-amlogic,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-gpio,org.kernel.vger.linux-hwmon,org.kernel.vger.linux-kernel,org.kernel.vger.linux-leds,org.kernel.vger.linux-media,org.kernel.vger.linux-pm,org.kernel.vger.linux-watchdog,org.kernel.vger.netdev,org.ozlabs.lists.linuxppc-dev |
|---|---|
| Message-ID | <[email protected]> |
On Wed, Mar 04, 2026 at 10:02:31AM +0100, Bartosz Golaszewski wrote:
> OF_GPIO is selected automatically on all OF systems. Any symbols it
Actually, that isn't really true.
if GPIOLIB
...
config OF_GPIO
def_bool y
depends on OF
depends on HAS_IOMEM
OF_GPIO also depends on GPIOLIB and HAS_IOMEM. I don't know why it needs
to depend on HAS_IOMEM, but the GPIOLIB dependency is real.
Guenter
> controls also provide stubs so there's really no reason to select it
> explicitly. We could simply remove the dependency but in order to avoid
> a new symbol popping up for everyone in make config - just convert it to
> requiring CONFIG_OF.
>
> Signed-off-by: Bartosz Golaszewski <[email protected]>
> Reviewed-by: Guenter Roeck <[email protected]>
> ---
> drivers/watchdog/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index dc78729ba2a5d6e035ed3cbe5c2b631d11b76b20..ef200339a22a6f9c51a46c9c0b8466add74313e2 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -250,7 +250,7 @@ config DA9062_WATCHDOG
>
> config GPIO_WATCHDOG
> tristate "Watchdog device controlled through GPIO-line"
> - depends on OF_GPIO
> + depends on OF
> select WATCHDOG_CORE
> help
> If you say yes here you get support for watchdog device
_______________________________________________
linux-snps-arc mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-snps-arc