> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: Thursday, July 4, 2024 2:36 PM
> To: Agarwal, Utsav <Utsav.Agarwal@xxxxxxxxxx>; Hennerich, Michael
> <Michael.Hennerich@xxxxxxxxxx>; Dmitry Torokhov
> <dmitry.torokhov@xxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof
> Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>; Sa,
> Nuno <Nuno.Sa@xxxxxxxxxx>
> Cc: linux-input@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Artamonovs, Arturs
> <Arturs.Artamonovs@xxxxxxxxxx>; Bimpikas, Vasileios
> <Vasileios.Bimpikas@xxxxxxxxxx>; Gaskell, Oliver
> <Oliver.Gaskell@xxxxxxxxxx>
> Subject: Re: [PATCH v7 3/3] dt-bindings: input: Update dtbinding for adp5588
>
> [External]
>
> On 04/07/2024 15:03, Utsav Agarwal via B4 Relay wrote:
> > From: Utsav Agarwal <utsav.agarwal@xxxxxxxxxx>
>
> Subject: everything is an update, be more specific.
>
> >
> > Updating dt bindings for adp5588. Since the device can now function in a
> > purely gpio mode, the following keypad specific properties are now made
> > optional:
> > - interrupts
> > - keypad,num-rows
> > - keypad,num-columns
> > - linux,keymap
> >
> > However the above properties are required to be specified when
> > configuring the device as a keypad, dependencies have been added
> > such that specifying either one would require the remaining as well.
> >
> > Note that interrupts are optional, but required when the device has been
> > configured in keypad mode.
> >
> > Signed-off-by: Utsav Agarwal <utsav.agarwal@xxxxxxxxxx>
> > ---
> > .../devicetree/bindings/input/adi,adp5588.yaml | 46
> +++++++++++++++++++---
> > 1 file changed, 41 insertions(+), 5 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/input/adi,adp5588.yaml
> b/Documentation/devicetree/bindings/input/adi,adp5588.yaml
> > index 26ea66834ae2..481c37595ebb 100644
> > --- a/Documentation/devicetree/bindings/input/adi,adp5588.yaml
> > +++ b/Documentation/devicetree/bindings/input/adi,adp5588.yaml
> > @@ -49,7 +49,10 @@ properties:
> > interrupt-controller:
> > description:
> > This property applies if either keypad,num-rows lower than 8 or
> > - keypad,num-columns lower than 10.
> > + keypad,num-columns lower than 10. This property is optional if
> > + keypad,num-rows or keypad,num-columns are not specified since the
> > + device then acts as gpio only, during which interrupts may or may
> > + not be utilized.
> >
> > '#interrupt-cells':
> > const: 2
> > @@ -65,13 +68,28 @@ properties:
> > minItems: 1
> > maxItems: 2
> >
> > +
> > +dependencies:
> > + keypad,num-rows:
> > + - linux,keymap
> > + - keypad,num-columns
> > + keypad,num-columns:
> > + - linux,keymap
> > + - keypad,num-rows
> > + linux,keymap:
> > + - keypad,num-rows
> > + - keypad,num-columns
> > +
> > +if:
> > + required:
> > + - linux,keymap
> > +then:
> > + required:
> > + - interrupts
> > +
> > required:
> > - compatible
> > - reg
> > - - interrupts
> > - - keypad,num-rows
> > - - keypad,num-columns
> > - - linux,keymap
> >
> > unevaluatedProperties: false
> >
> > @@ -108,4 +126,22 @@ examples:
> > >;
> > };
> > };
> > +
> > + - |
> > + #include <dt-bindings/interrupt-controller/irq.h>
> > + #include <dt-bindings/input/input.h>
> > + #include <dt-bindings/gpio/gpio.h>
> > +
> > + i2c {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + gpio@34 {
> > + compatible = "adi,adp5588";
> > + reg = <0x34>;
>
> Still messed indentation.
>
> Didn't you get already such feedback?
>
>
> Best regards,
> Krzysztof
I did, I somehow missed the extra spaces. My apologies for that.
Thanks,
Utsav
