[PATCH 0/2] media: i2c: vd55g1: Two v4l2-compliance fixes for controls

[Fernando Rimoli]

Two control fixes for failures v4l2-compliance reports on the sensor
subdev.

1/2: In manual exposure mode the driver marks the auto exposure lock and
bias controls grabbed. Outside streaming that fails the QUERYCTRL and
QUERY_EXT_CTRL tests, and because v4l2-compliance switches the exposure
mode while it walks the controls, the S_CTRL tests also fail with -EBUSY
when it starts from auto mode. The controls do not apply in manual mode,
which is what the inactive flag is for.

2/2: Control events, as most sensor drivers have.

How these were tested, since it is not the obvious way: I do not have a
VD55G1. I have the VD55G0 of a Surface Pro 9 (Intel IPU6), which runs an
out-of-tree driver derived from vd55g1 through Peter Marshall's VD55G0
support. In the code these patches touch, the exposure control handling
and the subdev ops, it is identical to vd55g1 in media next. I applied
the same two changes there. With v4l2-compliance 1.26.1 on the subdev:

  before:      41/44 in both exposure modes
  after 1/2:   QUERYCTRL, QUERY_EXT_CTRL and S_CTRL pass, controls show
               "inactive" rather than "grabbed" in manual mode, and the
               bias can be set there
  after both:  44/44 in both exposure modes

Getting from 43 to 44 also needed a fix to that driver's own flash
duty-cycle cap, which vd55g1 does not have (it is not upstream), so I
mention it only so the numbers add up. Streaming is unchanged, 150 of
150 frames at 57.60 fps before and after.

Both patches build without warnings at W=1 against v7.3-rc1, the same as
the unpatched file, and apply to media next as of d7e7bca71698. The file
there has not changed since 4a069a763e8a.

Fernando Rimoli (2):
  media: i2c: vd55g1: Mark the AE lock and bias controls inactive in
    manual mode
  media: i2c: vd55g1: Support control events

 drivers/media/i2c/vd55g1.c | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

base-commit: d7e7bca7169873b46e857c025c10f3336c1258d8
-- 
2.43.0