diff mbox series

[V0,1/2] dt-bindings: iio: adc: Add support for ump518 pmic adc

Message ID 20230816080225.21482-2-mingjin.yang@unisoc.com
State Not Applicable, archived
Headers show
Series Add support for UMP serise pmic adc | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Mingjin Yang Aug. 16, 2023, 8:02 a.m. UTC
Add ump518 pmic adc description and sample in dt-bindings.

Signed-off-by: Mingjin Yang <mingjin.yang@unisoc.com>
---
 Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml | 2 ++
 1 file changed, 2 insertions(+)

Comments

Conor Dooley Aug. 16, 2023, 2:44 p.m. UTC | #1
On Wed, Aug 16, 2023 at 04:02:24PM +0800, Mingjin Yang wrote:
> Add ump518 pmic adc description and sample in dt-bindings.
> 
> Signed-off-by: Mingjin Yang <mingjin.yang@unisoc.com>
> ---
>  Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml | 2 ++

Acked-by: Conor Dooley <conor.dooley@microchip.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
>  
>    reg:
>      maxItems: 1
> @@ -46,6 +47,7 @@ allOf:
>              contains:
>                enum:
>                  - sprd,ump9620-adc
> +                - sprd,ump518-adc
>      then:
>        properties:
>          nvmem-cells:
> -- 
> 2.17.1
>
Jonathan Cameron Aug. 28, 2023, 2:40 p.m. UTC | #2
On Wed, 16 Aug 2023 15:44:55 +0100
Conor Dooley <conor@kernel.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@unisoc.com>
> > ---
> >  Documentation/devicetree/bindings/iio/adc/sprd,sc2720-adc.yaml | 2 ++  
> 
> Acked-by: Conor Dooley <conor.dooley@microchip.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
> >
diff mbox series

Patch

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