Re: [PATCH v2 1/2] dt-bindings: iio: light: isl76682: Document ISL76682

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, 19 Nov 2023 19:08:45 +0100
Marek Vasut <marex@xxxxxxx> wrote:

> On 11/19/23 16:07, Jonathan Cameron wrote:
> > On Sat, 18 Nov 2023 01:06:09 +0100
> > Marek Vasut <marex@xxxxxxx> wrote:
> >   
> >> The ISL76682 is very basic ALS which only supports ALS or IR mode
> >> in four ranges, 1k/4k/16k/64k LUX. There is no IRQ support or any
> >> other fancy functionality. Document it as trivial device.
> >>
> >> Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> >> Signed-off-by: Marek Vasut <marex@xxxxxxx>  
> > 
> > I'm always a bit in 2 minds about trivial-devices as it doesn't
> > include a basic power supply.  In this case vdd which I'd prefer
> > to see documented even on such a simple device as this.
> > Not everyone is going to leave it powered up all the time!
> > 
> > Still I'm not that fussed. Maybe it's worth allowing vdd-supply in
> > trivial-devices.yaml?  
> 
> The hardware I have is powered all the time, so I'd be adding completely 
> untested code which is no less useless for me (at least right now). Lets 
> postpone this until there is a need for it, can we ?

It's untested but trivial code (one call and an error code check) but fair enough.
Previous experience says that we'll soon see it added... Its probably the most common
follow up patch for a new driver.

> 
> I addressed all the other comments in 2/2.
> 





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux