linux-kernel - Re: [PATCH v3 1/3] dt-bindings: iio: potentiometer: Add the Renesas X9250 potentiometers

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] [day] [month] [year] [list]

Date:   Mon, 1 May 2023 16:39:20 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     Herve Codina <herve.codina@...tlin.com>
Cc:     Lars-Peter Clausen <lars@...afoo.de>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Christophe Leroy <christophe.leroy@...roup.eu>,
        Thomas Petazzoni <thomas.petazzoni@...tlin.com>
Subject: Re: [PATCH v3 1/3] dt-bindings: iio: potentiometer: Add the Renesas
 X9250 potentiometers

On Mon, 24 Apr 2023 09:03:18 +0200
Herve Codina <herve.codina@...tlin.com> wrote:

> Hi Jonathan, Krzysztof,
> 
> On Sat, 22 Apr 2023 17:18:07 +0100
> Jonathan Cameron <jic23@...nel.org> wrote:
> 
> > On Fri, 21 Apr 2023 10:52:43 +0200
> > Herve Codina <herve.codina@...tlin.com> wrote:
> >   
> > > The Renesas X9250 is a quad digitally controlled potentiometers.
> > > 
> > > Signed-off-by: Herve Codina <herve.codina@...tlin.com>    
> > 
> > Hi Herve,
> > 
> > Historically we've been a bit lax in IIO bindings in always making
> > sure the per supplies are included.  As a result we frequently get
> > them added later and it just makes things messier than they should
> > be.
> > 
> > So please add vcc-supply from the start.  V+ and V- are a little trickier.
> > I was expecting datasheet to say they should be symmetric about 0 but it
> > doesn't. So they could be two independent supplies.
> > 
> > Also make it required as my current understanding is that we should
> > do that for supplies that are definitely present even if we could
> > rely on the fallback to regulator stubs if they aren't supplied.
> > So add the 3 supplies to required as well.  
> 
> Yes, I will add the following supplies in the next iteration:
>  - 'vcc-supply' for VCC
>  - 'avp-supply' for the analog V+
>  - 'avn-supply' for the analog V-
> 
> and add them in the required list of properties.
> 
> Are the names correct for these power supplies (avp and avn) ?

I think so.  I'm not totally sure on how DT maintainers think we should deal
with a two voltage level reference though.  Perhaps add some description to
make it very clear what is going on and we'll see what review comments we get!

Jonathan