RE: [PATCH] arm64: dts: imx8mn-ddr4-evk: Enable GPIO LED
Hi, Pavel > On Mon 2019-09-02 16:45:38, Anson Huang wrote: > > i.MX8MN DDR4 EVK board has a GPIO LED to indicate status, add support > > for it. > > LED maintainers want to be on the cc list... The get_maintainer.pl does NOT show the LED maintainers...But I have added it in V2. > > > @@ -15,6 +15,18 @@ > > stdout-path = &uart2; > > }; > > > > + leds { > > + compatible = "gpio-leds"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_gpio_led>; > > + > > + status { > > + label = "status"; > > + gpios = <&gpio3 16 GPIO_ACTIVE_HIGH>; > > + default-state = "on"; > > + }; > > And we should really standardize labels for user-controlled status LEDs. > Mentioning color would be nice, for a start. OK, I change the label to "yellow:status" in V2, please help review, thanks. Anson
Re: [PATCH] arm64: dts: imx8mn-ddr4-evk: Enable GPIO LED
On Mon 2019-09-02 16:45:38, Anson Huang wrote: > i.MX8MN DDR4 EVK board has a GPIO LED to indicate status, > add support for it. LED maintainers want to be on the cc list... > @@ -15,6 +15,18 @@ > stdout-path = &uart2; > }; > > + leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_gpio_led>; > + > + status { > + label = "status"; > + gpios = <&gpio3 16 GPIO_ACTIVE_HIGH>; > + default-state = "on"; > + }; And we should really standardize labels for user-controlled status LEDs. Mentioning color would be nice, for a start. Best regards, Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html