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

Hans de Goede
2026-04-17T18:56:42Z

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@gmail.com>

Thanks, patch looks good to me:

Reviewed-by: Hans de Goede <johannes.goede@oss.qualcomm.com>

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 */