On Fri, Oct 06, 2023 at 07:04:30PM +0300, Dmitry Rokosov wrote: > Property 'awinic,hwen-gpio' is optional, it can be used by the board > developer to connect AW200XX LED controller with appropriate poweron > GPIO pad. > > Signed-off-by: Dmitry Rokosov <ddrokosov@xxxxxxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/leds/awinic,aw200xx.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/leds/awinic,aw200xx.yaml b/Documentation/devicetree/bindings/leds/awinic,aw200xx.yaml > index 73b81f7a7258..e3ad11fc7a84 100644 > --- a/Documentation/devicetree/bindings/leds/awinic,aw200xx.yaml > +++ b/Documentation/devicetree/bindings/leds/awinic,aw200xx.yaml > @@ -41,6 +41,9 @@ properties: > description: > Leds matrix size > > + awinic,hwen-gpio: > + maxItems: 1 We have standard 'enable-gpios' or 'powerdown-gpios'. Those don't work here? Note that *-gpio is deprecated in favor of *-gpios. > + > patternProperties: > "^led@[0-9a-f]+$": > type: object > @@ -90,12 +93,15 @@ additionalProperties: false > > examples: > - | > + #include <dt-bindings/gpio/gpio.h> > #include <dt-bindings/leds/common.h> > > i2c { > #address-cells = <1>; > #size-cells = <0>; > > + awinic,hwen-gpio = <&gpio 3 GPIO_ACTIVE_HIGH>; > + > led-controller@3a { > compatible = "awinic,aw20036"; > reg = <0x3a>; > -- > 2.36.0 >