Re: [PATCH v3 0/4] GC0308 Camera Sensor

From: Sakari Ailus
Date: Fri Oct 27 2023 - 12:32:34 EST


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