The Linux Kernel Mailing List
 help / color / mirror / Atom feed
* [PATCH v3] bus: ts-nbus: drop unneeded dependency on OF_GPIO
@ 2026-05-06  8:19 Bartosz Golaszewski
  2026-05-06  8:33 ` Andy Shevchenko
  0 siblings, 1 reply; 3+ messages in thread
From: Bartosz Golaszewski @ 2026-05-06  8:19 UTC (permalink / raw)
  To: Alexandre Torgue, Kevin Hilman, Johan Hovold, Shawn Guo,
	Gatien Chevallier, Andy Shevchenko, Laurentiu Mihalcea
  Cc: brgl, linux-kernel, Bartosz Golaszewski

OF_GPIO is selected automatically on all OF systems. Any symbols it
controls also provide stubs and are private to GPIOLIB anyway so there's
really no reason to select it explicitly.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
---
Changes in v3:
- Send the bus patch separately so that it can be picked up for v7.2
- Link to v2: https://patch.msgid.link/20260316-gpio-of-kconfig-v2-0-de2f4b00a0e4@oss.qualcomm.com

Changes in v2:
- Make gpio-watchdog depend on OF && GPIOLIB
- Drop picked up patches
- Link to v1: https://patch.msgid.link/20260304-gpio-of-kconfig-v1-0-d597916e79e7@oss.qualcomm.com

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

diff --git a/drivers/bus/Kconfig b/drivers/bus/Kconfig
index 3181d8aa32a3..e4b1db809187 100644
--- a/drivers/bus/Kconfig
+++ b/drivers/bus/Kconfig
@@ -216,7 +216,7 @@ config TI_SYSC
 config TS_NBUS
 	tristate "Technologic Systems NBUS Driver"
 	depends on SOC_IMX28
-	depends on OF_GPIO && PWM
+	depends on PWM
 	help
 	  Driver for the Technologic Systems NBUS which is used to interface
 	  with the peripherals in the FPGA of the TS-4600 SoM.
-- 
2.47.3


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

* Re: [PATCH v3] bus: ts-nbus: drop unneeded dependency on OF_GPIO
  2026-05-06  8:19 [PATCH v3] bus: ts-nbus: drop unneeded dependency on OF_GPIO Bartosz Golaszewski
@ 2026-05-06  8:33 ` Andy Shevchenko
  2026-05-06 11:59   ` Bartosz Golaszewski
  0 siblings, 1 reply; 3+ messages in thread
From: Andy Shevchenko @ 2026-05-06  8:33 UTC (permalink / raw)
  To: Bartosz Golaszewski
  Cc: Alexandre Torgue, Kevin Hilman, Johan Hovold, Shawn Guo,
	Gatien Chevallier, Laurentiu Mihalcea, brgl, linux-kernel

On Wed, May 06, 2026 at 10:19:59AM +0200, Bartosz Golaszewski wrote:
> OF_GPIO is selected automatically on all OF systems. Any symbols it
> controls also provide stubs and are private to GPIOLIB anyway so there's
> really no reason to select it explicitly.

Is the plan to hide it from the user eventually?

Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

-- 
With Best Regards,
Andy Shevchenko



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

* Re: [PATCH v3] bus: ts-nbus: drop unneeded dependency on OF_GPIO
  2026-05-06  8:33 ` Andy Shevchenko
@ 2026-05-06 11:59   ` Bartosz Golaszewski
  0 siblings, 0 replies; 3+ messages in thread
From: Bartosz Golaszewski @ 2026-05-06 11:59 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: Bartosz Golaszewski, Alexandre Torgue, Kevin Hilman, Johan Hovold,
	Shawn Guo, Gatien Chevallier, Laurentiu Mihalcea, linux-kernel

On Wed, May 6, 2026 at 10:34 AM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> On Wed, May 06, 2026 at 10:19:59AM +0200, Bartosz Golaszewski wrote:
> > OF_GPIO is selected automatically on all OF systems. Any symbols it
> > controls also provide stubs and are private to GPIOLIB anyway so there's
> > really no reason to select it explicitly.
>
> Is the plan to hide it from the user eventually?
>

Yes, that's what I'm trying to do, but I need to remove all these
dependencies tree-wide first.

Bartosz

> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>

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

end of thread, other threads:[~2026-05-06 11:59 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-06  8:19 [PATCH v3] bus: ts-nbus: drop unneeded dependency on OF_GPIO Bartosz Golaszewski
2026-05-06  8:33 ` Andy Shevchenko
2026-05-06 11:59   ` Bartosz Golaszewski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox