linux-kernel - Re: [PATCH] dt-bindings: iio: adc: add a7779 doc

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: Windows password security audit tool. GUI, reports in PDF.

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

Date: Thu, 23 May 2024 18:03:52 +0100
From: Jonathan Cameron <Jonathan.Cameron@...wei.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
CC: ranechita <ramona.nechita@...log.com>, <linux-iio@...r.kernel.org>,
	Jonathan Cameron <jic23@...nel.org>, Lars-Peter Clausen <lars@...afoo.de>,
	Michael Hennerich <Michael.Hennerich@...log.com>, Rob Herring
	<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
	<conor+dt@...nel.org>, <linux-kernel@...r.kernel.org>,
	<devicetree@...r.kernel.org>
Subject: Re: [PATCH] dt-bindings: iio: adc: add a7779 doc

On Wed, 22 May 2024 15:33:56 +0200
Krzysztof Kozlowski <krzk@...nel.org> wrote:
Typo in the patch title.. dt-bindings: iio: adc: add ad7779

> On 22/05/2024 14:01, ranechita wrote:
> > Add dt bindings for adc ad7779.

Good to provide a very brief intro here to the device.
> > 
> > Signed-off-by: ranechita <ramona.nechita@...log.com>  
> 
> Are you sure that's the name or identity you want to use? Looks like
> misconfigured git...
> 
> > ---
> >  .../ABI/testing/sysfs-bus-iio-adc-ad777x      | 23 +++++
> >  .../bindings/iio/adc/adi,ad7779.yaml          | 93 +++++++++++++++++++  
> 
> Where is any user of it?
> ...
> 
> > +
> > +  clock-names:
> > +    items:
> > +      - const: adc-clk  
> 
> Drop names, it repeats what's obvious.
> 
> 
> Best regards,
> Krzysztof
> 
>