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

[Dan Scally <dan.scally@ideasonboard.com>]

Hi sakari

On 09/09/2026 09:19, 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: Daniel Scally <dan.scally@ideasonboard.com>
>   drivers/media/pci/intel/ivsc/mei_csi.c | 7 +------
>   1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/drivers/media/pci/intel/ivsc/mei_csi.c b/drivers/media/pci/intel/ivsc/mei_csi.c
> index c2917e156345..d4256415a829 100644
> --- a/drivers/media/pci/intel/ivsc/mei_csi.c
> +++ b/drivers/media/pci/intel/ivsc/mei_csi.c
> @@ -27,7 +27,6 @@
>   #include <linux/workqueue.h>
>   
>   #include <media/ipu-bridge.h>
> -#include <media/ipu6-pci-table.h>
>   #include <media/v4l2-async.h>
>   #include <media/v4l2-ctrls.h>
>   #include <media/v4l2-fwnode.h>
> @@ -644,13 +643,9 @@ static int mei_csi_probe(struct mei_cl_device *cldev,
>   	struct device *dev = &cldev->dev;
>   	struct pci_dev *ipu;
>   	struct mei_csi *csi;
> -	unsigned int i;
>   	int ret;
>   
> -	for (i = 0, ipu = NULL; !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();
>   	if (!ipu)
>   		return -ENODEV;
>