staging: media: max96712: drop unneeded dependency on OF_GPIO

[PATCH v3] staging: media: max96712: drop unneeded dependency on OF_GPIO

Posted by Bartosz Golaszewski 1 month, 1 week ago

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 staging 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/staging/media/max96712/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/staging/media/max96712/Kconfig b/drivers/staging/media/max96712/Kconfig
index 117fadf81bd0..93a2d583e90d 100644
--- a/drivers/staging/media/max96712/Kconfig
+++ b/drivers/staging/media/max96712/Kconfig
@@ -2,7 +2,6 @@
 config VIDEO_MAX96712
 	tristate "Maxim MAX96712 Quad GMSL2 Deserializer support"
 	depends on I2C
-	depends on OF_GPIO
 	depends on VIDEO_DEV
 	select V4L2_FWNODE
 	select VIDEO_V4L2_SUBDEV_API
-- 
2.47.3

Re: [PATCH v3] staging: media: max96712: drop unneeded dependency on OF_GPIO

Posted by Bartosz Golaszewski 5 days, 6 hours ago

On Wed, 06 May 2026 10:22:11 +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.
> 
> 

Applied, thanks!

[1/1] staging: media: max96712: drop unneeded dependency on OF_GPIO
      https://git.kernel.org/brgl/c/b8840c4f640e07562f15e5fc477ddaf1b2efb241

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>

Re: [PATCH v3] staging: media: max96712: drop unneeded dependency on OF_GPIO

Posted by Bartosz Golaszewski 3 weeks, 2 days ago

On Wed, May 6, 2026 at 10:22 AM Bartosz Golaszewski
<bartosz.golaszewski@oss.qualcomm.com> 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.
>
> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> ---
> Changes in v3:
> - Send the staging 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/staging/media/max96712/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/staging/media/max96712/Kconfig b/drivers/staging/media/max96712/Kconfig
> index 117fadf81bd0..93a2d583e90d 100644
> --- a/drivers/staging/media/max96712/Kconfig
> +++ b/drivers/staging/media/max96712/Kconfig
> @@ -2,7 +2,6 @@
>  config VIDEO_MAX96712
>         tristate "Maxim MAX96712 Quad GMSL2 Deserializer support"
>         depends on I2C
> -       depends on OF_GPIO
>         depends on VIDEO_DEV
>         select V4L2_FWNODE
>         select VIDEO_V4L2_SUBDEV_API
> --
> 2.47.3
>

Gentle ping.

Bart

Re: [PATCH v3] staging: media: max96712: drop unneeded dependency on OF_GPIO

Posted by Bartosz Golaszewski 1 week, 5 days ago

On Thu, May 21, 2026 at 2:37 PM Bartosz Golaszewski <brgl@kernel.org> wrote:
>
> On Wed, May 6, 2026 at 10:22 AM Bartosz Golaszewski
> <bartosz.golaszewski@oss.qualcomm.com> 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.
> >
> > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>
> > ---
> > Changes in v3:
> > - Send the staging 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/staging/media/max96712/Kconfig | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/staging/media/max96712/Kconfig b/drivers/staging/media/max96712/Kconfig
> > index 117fadf81bd0..93a2d583e90d 100644
> > --- a/drivers/staging/media/max96712/Kconfig
> > +++ b/drivers/staging/media/max96712/Kconfig
> > @@ -2,7 +2,6 @@
> >  config VIDEO_MAX96712
> >         tristate "Maxim MAX96712 Quad GMSL2 Deserializer support"
> >         depends on I2C
> > -       depends on OF_GPIO
> >         depends on VIDEO_DEV
> >         select V4L2_FWNODE
> >         select VIDEO_V4L2_SUBDEV_API
> > --
> > 2.47.3
> >
>
> Gentle ping.
>
> Bart

Are there any objections to me queuing this through the GPIO tree for v7.2?

Bartosz