[PATCH v3 0/2] media: i2c: vd55g1: preserve stream errors and always attempt stop
From: Jizhou Tong
Date: Mon Oct 12 2026 - 22:55:00 EST
This series fixes two error paths in the VD55G1 driver. Stream-start
currently replaces the original error with -EINVAL. On stream-off, a
failed exposure read prevents the subsequent stop command from being
issued because the CCI error accumulator remains set.
Preserve the stream-start error and handle the exposure read separately
from the stop sequence. The first patch builds on 5931eed35cb6, which
fixed the earlier false-success return.
Changes in v3:
- Reference the documented CCI error-accumulator behavior in the
stream-off comment.
- No functional changes from v2.
Changes in v2:
- Drop both Fixes tags as requested by Benjamin Mugnier.
- No source changes from v1.
v2: https://lore.kernel.org/all/010601a1251f83a6-b487de82-23f8-461b-99c2-dbf401de2158-000000@xxxxxxxxxxxxxxxxxxxxxxxxxxxx/
Based on Linux 7.3-rc5. Rechecked against mainline 24a19b953e2a and
media-next d7e7bca71698 on 12 October 2026; both issues remain.
Validation:
- ARM64 W=1 driver-object build.
- ASan/UBSan tests of the actual functions: twelve start failure points
with three error codes, and eight stream-off error combinations.
- Media CI for v2 passed builds, static analysis, ABI checks, checkpatch
and virtual-device tests.
- Hardware use on a Surface Laptop 7 with downstream VD55G0 support;
normal IR capture and face authentication have been working.
Jizhou Tong (2):
media: i2c: vd55g1: preserve the stream-start error
media: i2c: vd55g1: stop streaming after an exposure read error
drivers/media/i2c/vd55g1.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
base-commit: 72d3fcf802c45d00b300f25b848a93c3a2bd7c7e
--
2.53.0