Re: [PATCH v3 4/5] media: ipu-bridge: add sensor configuration for OV8858 (INT3477)

[Hans de Goede]

Hi,

On 17-Apr-26 18:32, Thierry Chatard wrote:
> The Omnivision OV8858 is used as the rear camera in several Intel
> IPU3-based devices (e.g. Dell Latitude 5285 2-in-1). Its ACPI HID is
> INT3477. Add a sensor configuration entry with a link frequency of
> 360 MHz to allow ipu_bridge to create the firmware node for this sensor.
> 
> Signed-off-by: Thierry Chatard <tchatard@xxxxxxxxx>

Thanks, patch looks good to me:

Reviewed-by: Hans de Goede <johannes.goede@xxxxxxxxxxxxxxxx>

Regards,

Hans



> ---
>  drivers/media/pci/intel/ipu-bridge.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/media/pci/intel/ipu-bridge.c b/drivers/media/pci/intel/ipu-bridge.c
> index fc6608e33..f51749d0f 100644
> --- a/drivers/media/pci/intel/ipu-bridge.c
> +++ b/drivers/media/pci/intel/ipu-bridge.c
> @@ -63,6 +63,8 @@ static const struct ipu_sensor_config ipu_supported_sensors[] = {
>  	IPU_SENSOR_CONFIG("INT33F0", 1, 384000000),
>  	/* Omnivision OV2740 */
>  	IPU_SENSOR_CONFIG("INT3474", 1, 180000000),
> +	/* Omnivision OV8858 */
> +	IPU_SENSOR_CONFIG("INT3477", 1, 360000000),
>  	/* Omnivision OV5670 */
>  	IPU_SENSOR_CONFIG("INT3479", 1, 422400000),
>  	/* Omnivision OV8865 */