Re: [PATCH 0/3] media: i2c: ov5693: Fix mode changes, flips and autosuspend

From: Chris Simpson

Date: Tue Oct 27 2026 - 17:09:02 EST


Tested on a Surface Pro 9 (Intel Alder Lake-U, i7-1255U, IPU6, front
OV5693) in a 7.3-based kernel, with the series applied to an ov5693.c
that matches the base-commit (d7e7bca71698) apart from the pad ops'
client_info argument, against the same file unpatched, using raw
captures from the ISYS video node:

- Patch 1: with the sensor held powered between streams, OUTPUT_SIZE
(read over I2C during each stream) matched the requested size in
32 of 32 streams requesting 2592x1944 or 1296x972, and every
stream delivered image data. Unpatched, it stayed at 2592x1944 in all
32 and the binned streams gave no image.
- Patch 2: raw 2592x1944 frames are BGGR at all four hflip/vflip
settings. Unpatched, the order is GBRG with exactly one flip on.
- Patch 3: with autosuspend_delay_ms at 3000, the sensor suspended
3.4-3.7 s after stream stop (the delay plus the 25% timer slack),
after both 1 s and 5 s streams. Unpatched, 1-2 ms after a 5 s stream.

The test scripts, the analysis of their output and this summary were
prepared with an AI coding assistant (Claude); I ran the tests on my
Surface Pro 9.

Tested-by: Chris Simpson <chris@xxxxxxxx>