[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZTvlrJK8nwvFbUhU@kekkonen.localdomain>
Date: Fri, 27 Oct 2023 16:30:36 +0000
From: Sakari Ailus <sakari.ailus@...ux.intel.com>
To: Sebastian Reichel <sre@...nel.org>
Cc: Jacopo Mondi <jacopo.mondi@...asonboard.com>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>,
linux-media@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/4] GC0308 Camera Sensor
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
Powered by blists - more mailing lists