Re: [PATCH 14/46] dt-bindings:iio:light:avago,apds9960: txt to yaml conversion

[Jonathan Cameron <jic23@kernel.org>]

On Sat, 31 Oct 2020 14:35:56 -0700
Matt Ranostay <matt.ranostay@konsulko.com> wrote:

> On Sat, Oct 31, 2020 at 2:33 PM Matt Ranostay
> <matt.ranostay@konsulko.com> wrote:
> >
> > On Sat, Oct 31, 2020 at 11:51 AM Jonathan Cameron <jic23@kernel.org> wrote:  
> > >
> > > From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > >
> > > Very simple binding that we could move into trivial-devices.yaml
> > > with a small loss of documentation.
> > >
> > > Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
> > > Cc: Matt Ranostay <mranostay@gmail.com>  
> >
> > Acked-by: Matt Ranostay <matt.ranostay@konsulko.com>  
> 
> Actually still Acked-by but could you change my email below to my Konsulko one?
Will do.

> 
> Thanks,
> 
> Matt
> 
> > > ---
> > >  .../bindings/iio/light/apds9960.txt           | 21 ---------
> > >  .../bindings/iio/light/avago,apds9960.yaml    | 44 +++++++++++++++++++
> > >  2 files changed, 44 insertions(+), 21 deletions(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/iio/light/apds9960.txt b/Documentation/devicetree/bindings/iio/light/apds9960.txt
> > > deleted file mode 100644
> > > index c53ddb81c4aa..000000000000
> > > --- a/Documentation/devicetree/bindings/iio/light/apds9960.txt
> > > +++ /dev/null
> > > @@ -1,21 +0,0 @@
> > > -* Avago APDS9960 gesture/RGB/ALS/proximity sensor
> > > -
> > > -https://www.avagotech.com/docs/AV02-4191EN
> > > -
> > > -Required properties:
> > > -
> > > -  - compatible: must be "avago,apds9960"
> > > -  - reg: the I2c address of the sensor
> > > -  - interrupts : the sole interrupt generated by the device
> > > -
> > > -  Refer to interrupt-controller/interrupts.txt for generic interrupt client
> > > -  node bindings.
> > > -
> > > -Example:
> > > -
> > > -apds9960@39 {
> > > -       compatible = "avago,apds9960";
> > > -       reg = <0x39>;
> > > -       interrupt-parent = <&gpio1>;
> > > -       interrupts = <16 1>;
> > > -};
> > > diff --git a/Documentation/devicetree/bindings/iio/light/avago,apds9960.yaml b/Documentation/devicetree/bindings/iio/light/avago,apds9960.yaml
> > > new file mode 100644
> > > index 000000000000..eae8c7327c0f
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/iio/light/avago,apds9960.yaml
> > > @@ -0,0 +1,44 @@
> > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> > > +%YAML 1.2
> > > +---
> > > +$id: http://devicetree.org/schemas/iio/light/avago,apds9960.yaml#
> > > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > > +
> > > +title: Avago APDS9960 gesture/RGB/ALS/proximity sensor
> > > +
> > > +maintainers:
> > > +  - Matt Ranostay <mranostay@gmail.com>
> > > +
> > > +description: |
> > > +  Datasheet at https://www.avagotech.com/docs/AV02-4191EN
> > > +
> > > +properties:
> > > +  compatible:
> > > +    const: avago,apds9960
> > > +
> > > +  reg:
> > > +    maxItems: 1
> > > +
> > > +  interrupts:
> > > +    maxItems: 1
> > > +
> > > +additionalProperties: false
> > > +
> > > +required:
> > > +  - compatible
> > > +  - reg
> > > +
> > > +examples:
> > > +  - |
> > > +    i2c {
> > > +        #address-cells = <1>;
> > > +        #size-cells = <0>;
> > > +
> > > +        light-sensor@39 {
> > > +            compatible = "avago,apds9960";
> > > +            reg = <0x39>;
> > > +            interrupt-parent = <&gpio1>;
> > > +            interrupts = <16 1>;
> > > +        };
> > > +    };
> > > +...
> > > --
> > > 2.28.0
> > >