NACK/Cmnt: [SRU][Noble][OEM-6.17][PATCH 0/1] Backport upstream ov08x40 camera sensor updates

[Stefan Bader]

On 12/01/2026 08:32, You-Sheng Yang wrote:
> BugLink: https://bugs.launchpad.net/bugs/2116170
> 
> [ Impact ]
> 
> OV08X40, added in v6.2 kernel, has inverted hflip regiter written in
> `ov08x40_set_ctrl_vflip`. This is fixed by v6.18-rc1 commit c7df6f339af94
> ("media: ov08x40: Fix the horizontal flip control").
> 
> [ Test Plan ]
> 
> 1. Install patched kernel from -proposed kernel and reboot into it.
> 
> 2. Find out the device name for ov08x40:
> ```
> $ grep -nr ov08x40 /sys/class/video4linux/*/name
> /sys/class/video4linux/v4l-subdev6/name:1:ov08x40 17-0010
> 
> $ sudo v4l2-ctl -l -d /dev/v4l-subdev6
> 
> User Controls
> 
>           exposure 0x00980911 (int) : min=4 max=4992 step=1 default=4992 value=2470 flags=0x00001000
>    horizontal_flip 0x00980914 (bool) : default=0 value=0 flags=0x00001000
>      vertical_flip 0x00980915 (bool) : default=0 value=0 flags=0x00001000
> ......
> ```
> 
> 3. Set horizontal flip to 1. The original driver won't have any effect on this
> step, but the patched driver should have video output flipped horizontally:
> ```
> $ sudo v4l2-ctl -c horizontal_flip=1 -d /dev/v4l-subdev6
> horizontal_flip: 0
> $ sudo v4l2-ctl -C horizontal_flip -d /dev/v4l-subdev6
> horizontal_flip: 1
> 
> ```
> 
> 4. Use camera app to examine the output. Here an Intel MIPI camera platform with
> ov08x40 camera sensor is used:
> ```
> $ sudo gst-launch-1.0 icamerasrc ! videoconvert ! autovideosink
> ```
> 
> [ Where problems could occur ]
> 
> No. This writes the right value intro device register only.
> 
> [ Other Info ]
> 
> While ov08x40 was added at v6.2, nominate for kernels newer than that.
> 
> Hao Yao (1):
>    media: ov08x40: Fix the horizontal flip control
> 
>   drivers/media/i2c/ov08x40.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 


Rejected for the following reasons:
The target specification is too misleading. Looking at the bug report 
the intention is noble:linux AND noble:linux-oem-6.17. But the subject 
is easily misread as only wanting the latter. We should be clear there 
and use something like "[N][N:linux-oem-6.17]" or at least 
"[N][N:OEM-6.17]".

-Stefan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xE8675DEECBEECEA3.asc
Type: application/pgp-keys
Size: 48643 bytes
Desc: OpenPGP public key
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20260112/ba123b78/attachment-0001.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20260112/ba123b78/attachment-0001.sig>