linux-kernel - Re: [PATCH V0 1/2] dt-bindings: iio: adc: Add support for ump518 pmic adc

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]

Message-ID: <20230828153930.73d879ba@jic23-huawei>
Date:   Mon, 28 Aug 2023 15:40:10 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     Conor Dooley <conor@...nel.org>
Cc:     Mingjin Yang <mingjin.yang@...soc.com>,
        Lars-Peter Clausen <lars@...afoo.de>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>, magicyangmingjin@...il.com,
        Ling_Ling.Xu@...soc.com, Jinfeng.Lin1@...soc.com,
        Yangbin.Li@...soc.com, Jiansheng.Wu@...soc.com,
        Orson Zhai <orsonzhai@...il.com>,
        Baolin Wang <baolin.wang@...ux.alibaba.com>,
        Chunyan Zhang <zhang.lyra@...il.com>,
        linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH V0 1/2] dt-bindings: iio: adc: Add support for ump518
 pmic adc

On Wed, 16 Aug 2023 15:44:55 +0100
Conor Dooley <conor@...nel.org> wrote:

> On Wed, Aug 16, 2023 at 04:02:24PM +0800, Mingjin Yang wrote:
> > Add ump518 pmic adc description and sample in dt-bindings.
ADC

Description?  I'm seeing the compatible only which doesn't seem like
a description to me.



> > 
> > Signed-off-by: Mingjin Yang <mingjin.yang@...soc.com>
> > ---
> >  Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml | 2 ++  
> 
> Acked-by: Conor Dooley <conor.dooley@...rochip.com>
> 
> Thanks,
> Conor.
> 
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml b/Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml
> > index 8181cf9a8e07..f28d188d83da 100644
> > --- a/Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml
> > +++ b/Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml
> > @@ -21,6 +21,7 @@ properties:
> >        - sprd,sc2730-adc
> >        - sprd,sc2731-adc
> >        - sprd,ump9620-adc
> > +      - sprd,ump518-adc

alpha-numeric ordering probably worth retaining.
So I'd move this and the case below up one line.

Thanks,

Jonathan

> >  
> >    reg:
> >      maxItems: 1
> > @@ -46,6 +47,7 @@ allOf:
> >              contains:
> >                enum:
> >                  - sprd,ump9620-adc
> > +                - sprd,ump518-adc
> >      then:
> >        properties:
> >          nvmem-cells:
> > -- 
> > 2.17.1
> >