Linux Media Controller development
 help / color / mirror / Atom feed
* [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6
@ 2026-09-22  6:40 Sakari Ailus
  2026-09-23 12:21 ` Antti Laakso
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Sakari Ailus @ 2026-09-22  6:40 UTC (permalink / raw)
  To: linux-media
  Cc: Thorsten Leemhuis, Junjie Cao, Miguel Vadillo,
	Mauro Carvalho Chehab, Manuel Knitza, Antti Laakso, Mehdi Djait,
	regressions

The cvs driver is only usable with the Intel ipu6 driver and is commonly
required for the ipu6 driver to function. Make the default of the cvs
driver the same as what was selected for the ipu6 driver.

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
 drivers/media/i2c/cvs/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/media/i2c/cvs/Kconfig b/drivers/media/i2c/cvs/Kconfig
index 4309d20dd726..f482f1ccc38a 100644
--- a/drivers/media/i2c/cvs/Kconfig
+++ b/drivers/media/i2c/cvs/Kconfig
@@ -4,6 +4,7 @@ config VIDEO_INTEL_CVS
 	tristate "Intel CVS CSI-2 bridge support"
 	depends on I2C && ACPI && VIDEO_DEV
 	depends on IPU_BRIDGE || !IPU_BRIDGE
+	default VIDEO_INTEL_IPU6
 	select MEDIA_CONTROLLER
 	select VIDEO_V4L2_SUBDEV_API
 	select V4L2_FWNODE

base-commit: 508a3559448684c4c0110a0c1614deaface399c4
-- 
2.47.3


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

* Re: [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6
  2026-09-22  6:40 [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6 Sakari Ailus
@ 2026-09-23 12:21 ` Antti Laakso
  2026-09-23 12:44 ` Mehdi Djait
  2026-09-23 18:10 ` Vadillo, Miguel
  2 siblings, 0 replies; 6+ messages in thread
From: Antti Laakso @ 2026-09-23 12:21 UTC (permalink / raw)
  To: Sakari Ailus
  Cc: linux-media, Thorsten Leemhuis, Junjie Cao, Miguel Vadillo,
	Mauro Carvalho Chehab, Manuel Knitza, Mehdi Djait, regressions

On Tue, Sep 22, 2026 at 09:40:17AM +0300, Sakari Ailus wrote:
> The cvs driver is only usable with the Intel ipu6 driver and is commonly
> required for the ipu6 driver to function. Make the default of the cvs
> driver the same as what was selected for the ipu6 driver.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
>  drivers/media/i2c/cvs/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/media/i2c/cvs/Kconfig b/drivers/media/i2c/cvs/Kconfig
> index 4309d20dd726..f482f1ccc38a 100644
> --- a/drivers/media/i2c/cvs/Kconfig
> +++ b/drivers/media/i2c/cvs/Kconfig
> @@ -4,6 +4,7 @@ config VIDEO_INTEL_CVS
>  	tristate "Intel CVS CSI-2 bridge support"
>  	depends on I2C && ACPI && VIDEO_DEV
>  	depends on IPU_BRIDGE || !IPU_BRIDGE
> +	default VIDEO_INTEL_IPU6
>  	select MEDIA_CONTROLLER
>  	select VIDEO_V4L2_SUBDEV_API
>  	select V4L2_FWNODE
> 
> base-commit: 508a3559448684c4c0110a0c1614deaface399c4
> -- 
> 2.47.3
> 

Reviewed-by: Antti Laakso <antti.laakso@linux.intel.com>

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

* Re: [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6
  2026-09-22  6:40 [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6 Sakari Ailus
  2026-09-23 12:21 ` Antti Laakso
@ 2026-09-23 12:44 ` Mehdi Djait
  2026-09-23 21:04   ` Sakari Ailus
  2026-09-23 18:10 ` Vadillo, Miguel
  2 siblings, 1 reply; 6+ messages in thread
From: Mehdi Djait @ 2026-09-23 12:44 UTC (permalink / raw)
  To: Sakari Ailus
  Cc: linux-media, Thorsten Leemhuis, Junjie Cao, Miguel Vadillo,
	Mauro Carvalho Chehab, Manuel Knitza, Antti Laakso, regressions

Hello Sakari,

Thank you for the patch!

On Tue, Sep 22, 2026 at 09:40:17AM +0300, Sakari Ailus wrote:
> The cvs driver is only usable with the Intel ipu6 driver and is commonly

I don't know about this statement. Do you mean the ipu6 with the ipu7
support merged ?

I write this because I use the cvs with the ipu7 driver from staging on
one my dell laptops. No ipu6 on that system.

> required for the ipu6 driver to function. Make the default of the cvs
> driver the same as what was selected for the ipu6 driver.

--
Kind Regards,
Mehdi Djait

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

* Re: [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6
  2026-09-22  6:40 [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6 Sakari Ailus
  2026-09-23 12:21 ` Antti Laakso
  2026-09-23 12:44 ` Mehdi Djait
@ 2026-09-23 18:10 ` Vadillo, Miguel
  2026-09-23 21:06   ` Sakari Ailus
  2 siblings, 1 reply; 6+ messages in thread
From: Vadillo, Miguel @ 2026-09-23 18:10 UTC (permalink / raw)
  To: Sakari Ailus, linux-media
  Cc: Thorsten Leemhuis, Junjie Cao, Mauro Carvalho Chehab,
	Manuel Knitza, Antti Laakso, Mehdi Djait, regressions

Hi Sakari,

On 9/21/26 11:40 PM, Sakari Ailus wrote:
> The cvs driver is only usable with the Intel ipu6 driver and is commonly
> required for the ipu6 driver to function. Make the default of the cvs
> driver the same as what was selected for the ipu6 driver.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---
>   drivers/media/i2c/cvs/Kconfig | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/media/i2c/cvs/Kconfig b/drivers/media/i2c/cvs/Kconfig
> index 4309d20dd726..f482f1ccc38a 100644
> --- a/drivers/media/i2c/cvs/Kconfig
> +++ b/drivers/media/i2c/cvs/Kconfig
> @@ -4,6 +4,7 @@ config VIDEO_INTEL_CVS
>   	tristate "Intel CVS CSI-2 bridge support"
>   	depends on I2C && ACPI && VIDEO_DEV
>   	depends on IPU_BRIDGE || !IPU_BRIDGE
> +	default VIDEO_INTEL_IPU6

Would this be affected once the IPU6 and 7 are somehow merged?

--
regards,
Miguel>   	select MEDIA_CONTROLLER
>   	select VIDEO_V4L2_SUBDEV_API
>   	select V4L2_FWNODE
> 
> base-commit: 508a3559448684c4c0110a0c1614deaface399c4


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

* Re: [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6
  2026-09-23 12:44 ` Mehdi Djait
@ 2026-09-23 21:04   ` Sakari Ailus
  0 siblings, 0 replies; 6+ messages in thread
From: Sakari Ailus @ 2026-09-23 21:04 UTC (permalink / raw)
  To: Mehdi Djait
  Cc: linux-media, Thorsten Leemhuis, Junjie Cao, Miguel Vadillo,
	Mauro Carvalho Chehab, Manuel Knitza, Antti Laakso, regressions

Hi Mehdi,

On Wed, Sep 23, 2026 at 02:44:47PM +0200, Mehdi Djait wrote:
> Hello Sakari,
> 
> Thank you for the patch!
> 
> On Tue, Sep 22, 2026 at 09:40:17AM +0300, Sakari Ailus wrote:
> > The cvs driver is only usable with the Intel ipu6 driver and is commonly
> 
> I don't know about this statement. Do you mean the ipu6 with the ipu7
> support merged ?
> 
> I write this because I use the cvs with the ipu7 driver from staging on
> one my dell laptops. No ipu6 on that system.

Right, this will hold true once the ipu7 staging driver will be gone.

I can add a note about this in v2.

-- 
Regards,

Sakari Ailus

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

* Re: [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6
  2026-09-23 18:10 ` Vadillo, Miguel
@ 2026-09-23 21:06   ` Sakari Ailus
  0 siblings, 0 replies; 6+ messages in thread
From: Sakari Ailus @ 2026-09-23 21:06 UTC (permalink / raw)
  To: Vadillo, Miguel
  Cc: linux-media, Thorsten Leemhuis, Junjie Cao, Mauro Carvalho Chehab,
	Manuel Knitza, Antti Laakso, Mehdi Djait, regressions

Hi Miguel,

On Wed, Sep 23, 2026 at 11:10:55AM -0700, Vadillo, Miguel wrote:
> Hi Sakari,
> 
> On 9/21/26 11:40 PM, Sakari Ailus wrote:
> > The cvs driver is only usable with the Intel ipu6 driver and is commonly
> > required for the ipu6 driver to function. Make the default of the cvs
> > driver the same as what was selected for the ipu6 driver.
> > 
> > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> > ---
> >   drivers/media/i2c/cvs/Kconfig | 1 +
> >   1 file changed, 1 insertion(+)
> > 
> > diff --git a/drivers/media/i2c/cvs/Kconfig b/drivers/media/i2c/cvs/Kconfig
> > index 4309d20dd726..f482f1ccc38a 100644
> > --- a/drivers/media/i2c/cvs/Kconfig
> > +++ b/drivers/media/i2c/cvs/Kconfig
> > @@ -4,6 +4,7 @@ config VIDEO_INTEL_CVS
> >   	tristate "Intel CVS CSI-2 bridge support"
> >   	depends on I2C && ACPI && VIDEO_DEV
> >   	depends on IPU_BRIDGE || !IPU_BRIDGE
> > +	default VIDEO_INTEL_IPU6
> 
> Would this be affected once the IPU6 and 7 are somehow merged?

It's unrelated. Mehdi asked about the ipu7 driver, too, and I think I'll
send v2 that takes that into account in the commit message.

-- 
Regards,

Sakari Ailus

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

end of thread, other threads:[~2026-09-23 21:06 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-22  6:40 [PATCH 1/1] media: cvs: Default selection to VIDEO_INTEL_IPU6 Sakari Ailus
2026-09-23 12:21 ` Antti Laakso
2026-09-23 12:44 ` Mehdi Djait
2026-09-23 21:04   ` Sakari Ailus
2026-09-23 18:10 ` Vadillo, Miguel
2026-09-23 21:06   ` Sakari Ailus

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