Re: [PATCH v2 2/3] media: cvs: Use ipu_bridge_get_ipu6()

[Antti Laakso <antti.laakso@linux.intel.com>]

On Wed, Sep 09, 2026 at 11:19:51AM +0300, Sakari Ailus wrote:
> Instead of going through the PCI device ID table, use
> ipu_bridge_get_ipu6() to obtain a pointer to the IPU PCI device.
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> ---

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

>  drivers/media/i2c/cvs/core.c | 7 ++-----
>  1 file changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/media/i2c/cvs/core.c b/drivers/media/i2c/cvs/core.c
> index d4a3b9c3bab1..98cadd58a075 100644
> --- a/drivers/media/i2c/cvs/core.c
> +++ b/drivers/media/i2c/cvs/core.c
> @@ -21,7 +21,6 @@
>  #include <linux/workqueue.h>
>  
>  #include <media/ipu-bridge.h>
> -#include <media/ipu6-pci-table.h>
>  
>  #include "icvs.h"
>  
> @@ -656,14 +655,12 @@ static int cvs_configure_dev_caps(struct icvs *ctx)
>   */
>  static int cvs_core_probe(struct device *dev, struct i2c_client *i2c)
>  {
> -	struct pci_dev *ipu = NULL;
> +	struct pci_dev *ipu;
>  	struct icvs *ctx;
>  	int ret;
>  
>  	/* Locate IPU device */
> -	for (unsigned int i = 0; !ipu && ipu6_pci_tbl[i].vendor; i++)
> -		ipu = pci_get_device(ipu6_pci_tbl[i].vendor,
> -				     ipu6_pci_tbl[i].device, NULL);
> +	ipu = ipu_bridge_get_ipu6();
>  	for (unsigned int i = 0; !ipu && icvs_ipu7_tbl[i].vendor; i++)
>  		ipu = pci_get_device(icvs_ipu7_tbl[i].vendor,
>  				     icvs_ipu7_tbl[i].device, NULL);
> -- 
> 2.47.3
> 
>