linux-kernel - Re: [PATCH 2/2] iio: light: Add support for the AMS TCS3430 color sensor

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Z0yjphBfTalBSId7@collins>
Date: Sun, 1 Dec 2024 18:57:58 +0100
From: Paul Kocialkowski <paulk@...-base.io>
To: Jonathan Cameron <jic23@...nel.org>
Cc: linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, Lars-Peter Clausen <lars@...afoo.de>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>
Subject: Re: [PATCH 2/2] iio: light: Add support for the AMS TCS3430 color
 sensor

Hi,

Le Sun 01 Dec 24, 11:58, Jonathan Cameron a écrit :
> On Sat, 30 Nov 2024 18:42:39 +0100
> Paul Kocialkowski <paulk@...-base.io> wrote:
> 
> > The AMS TCS3430 is a XYZ tristimulus color sensor, with an additional
> > infrared channel.
> > 
> > This driver implements support for all channels, with configurable
> > integration time and gain. Both direct reading and triggered-buffer
> > modes are supported.
> > 
> > The device's interrupt can be used to sample all channels at the end
> > of conversion and is optional.
> > 
> > A thereshold mechanism is available in hardware but only applies to
> > the Z channel. As a result it is of very low interest and was not
> > implemented.
> > 
> > Signed-off-by: Paul Kocialkowski <paulk@...-base.io>
> 
> Hi Paul
> 
> As at least some of the feedback I gave on your other driver applies
> here as well, I'm not going to review this version separately.
> 
> I'd generally advise against sending multiple similar drivers for review
> at the same time as it is less efficient.  Sad truth is that reviewer
> time is generally a bigger bottleneck for kernel code than the time
> of driver authors.  Any time you can put in to help with that will generally
> also accelerate the rate at which we get to your code!
> 
> Absolutely fine to overlap your submissions though. When the first
> driver is getting close to being merged, feel free to send the second
> one to get review started.

That makes sense yes! Let's focus on the first driver for now and I'll apply
corresponding changes to this one when the first one is ready.

Thanks!

Paul

-- 
Paul Kocialkowski,

Independent contractor - sys-base - https://www.sys-base.io/
Free software developer - https://www.paulk.fr/

Expert in multimedia, graphics and embedded hardware support with Linux.

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)