Messages in this thread | ![/](/images/icornerl.gif) | | Date | Mon, 22 May 2017 19:48:20 -0500 | From | Rob Herring <> | Subject | Re: [PATCH 12/12] arm64: dts: hikey960: add LED nodes |
| |
On Wed, May 17, 2017 at 04:37:45PM +0800, Guodong Xu wrote: > HiKey960 has four user LEDs, and two special purpose LEDs: WiFi and BT > respectively. > > All of them are implemented as GPIO. > > Signed-off-by: Guodong Xu <guodong.xu@linaro.org> > --- > arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts | 48 +++++++++++++++++++++++ > 1 file changed, 48 insertions(+) > > diff --git a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts > index 6de86c0..4839885 100644 > --- a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts > +++ b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts > @@ -48,6 +48,54 @@ > linux,code = <KEY_POWER>; > }; > }; > + > + leds {
This too should move out of the soc node.
> + compatible = "gpio-leds"; > + status = "disabled"; > + user_led1 { > + label = "user_led1"; > + /* gpio_150_user_led1 */ > + gpios = <&gpio18 6 0>; > + linux,default-trigger = "heartbeat"; > + }; > + > + user_led2 { > + label = "user_led2"; > + /* gpio_151_user_led2 */ > + gpios = <&gpio18 7 0>; > + linux,default-trigger = "mmc0"; > + }; > + > + user_led3 { > + label = "user_led3"; > + /* gpio_189_user_led3 */ > + gpios = <&gpio23 5 0>; > + default-state = "off"; > + }; > + > + user_led4 { > + label = "user_led4"; > + /* gpio_190_user_led4 */ > + gpios = <&gpio23 6 0>; > + linux,default-trigger = "cpu0"; > + }; > + > + wlan_active_led { > + label = "wifi_active"; > + /* gpio_205_wifi_active */ > + gpios = <&gpio25 5 0>; > + linux,default-trigger = "phy0tx"; > + default-state = "off"; > + }; > + > + bt_active_led { > + label = "bt_active"; > + gpios = <&gpio25 7 0>; > + /* gpio_207_user_led1 */ > + linux,default-trigger = "hci0rx"; > + default-state = "off"; > + }; > + }; > }; > }; > > -- > 2.10.2 >
| ![\](/images/icornerr.gif) |