Hi Sebastian, On Fri, Oct 27, 2023 at 03:12:00AM +0200, Sebastian Reichel wrote: > Hi, > > I did the following tests done by me on an i.MX6ULL based system [0]: > > * v4l2-compliance -u /dev/v4l-subdev1 > - v4l2-compliance 1.24.1, 32 bits, 32-bit time_t > (from Debian testing) > - Total for device /dev/v4l-subdev1: 44, Succeeded: 44 > * Using gstreamer + v4l2-ctl > - Tried 640x480, 320x240, 160x120 YUYV8_2X8 formats > - Tested effect of all exposed user controls > * checkpatch does not report any driver issues > * dt_binding_check does not report anything > > Note, that there is another patchset adding GC2145, which > adds the same vendor prefix. I just included it for completeness, > since it's needed to avoid checkpatch and dt_binding_check > warnings. Thanks for the update. There's a spelling error I noticed which I'll fix while applying but otherwise I'm looking forward to have Jacopo's comments on the patch. It seems good to me. -- Kind regards, Sakari Ailus