ACK: Re: [PATCH 0/1][SRU][U][R][Q] Support Samsung S5K3J1 sensor for Intel MIPI camera

[Paolo Pisati]

On Sat, Oct 18, 2025 at 12:21:43AM +0800, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/2121852
> 
> [ Impact ]
> 
> Failed to probe Samsung S5K3J1 sensor for Intel MIPI camera.
> 
> This takes ipu6-drivers/questing >= 0~git202506270118.9bff7368-0ubuntu1.
> 
> [ Test Plan ]
> 
> 1. install kernel from -proposed pocket. For Resolute and Questing:
>    ```
>    $ sudo apt install --no-install-recommends --yes \
>       linux-generic \
>       linux-modules-ipu6-generic \
>       linux-modules-ipu7-generic \
>       linux-modules-usbio-generic \
>       linux-modules-vision-generic
>    ```
>    For linux-oem-6.14, linux-oem-6.17 in Noble:
>    ```
>    $ sudo apt install --no-install-recommends --yes \
>       linux-oem-24.04c \
>       linux-modules-ipu6-oem-24.04c \
>       linux-modules-ipu7-oem-24.04c \
>       linux-modules-usbio-oem-24.04c \
>       linux-modules-vision-oem-24.04c \
>       linux-oem-24.04d \
>       linux-modules-ipu6-oem-24.04d \
>       linux-modules-ipu7-oem-24.04d \
>       linux-modules-usbio-oem-24.04d \
>       linux-modules-vision-oem-24.04d
>    ```
> 
> 2. Reboot to installed kernel and check dmesg:
>    ```
>    intel_ipu7_isys.isys intel_ipu7.isys.40: bind s5k3j1 3-0010 nlanes is 4 port is 3
>    s5k3j1 i2c-INT346D:00: Instantiated dw9714 VCM
>    ```
> 
> 3. Check gstreamer icamerasrc element:
>    ```
>    $ sudo gst-inspect-1.0 icamerasrc
>    ...
>    Element Properties:
>      ...
>      device-name : The input devices name queried from HAL
>                         flags: readable, writable
>                         Enum "GstCamerasrcDeviceName" Default: 0, "ov08x40-uf"
>                            (0): ov08x40-uf - ov08x40 as sensor.
>                            (1): s5k3j1-wf - s5k3j1 as sensor.
>    ```
> 
> 4. Use https://webcamtests.com/ to test webcam.
> 
> [ Where problems could occur ]
> 
> This allows ipu-bridge to probe/link the camera sensor. No side effect.
> 
> [ Other Info ]
> 
> Nominate for oem-6.14 and oem-6.17 in Noble, and the future migration path
> Questing and Resolute.
> 
> Jimmy Su (1):
>   UBUNTU: SAUCE: media: ipu-bridge: Support s5k3j1 sensor
> 
>  drivers/media/pci/intel/ipu-bridge.c | 2 ++
>  1 file changed, 2 insertions(+)

Acked-by: Paolo Pisati <paolo.pisati at canonical.com>
-- 
bye,
p.