[<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