Am Freitag, 1. März 2019, 23:58:13 CET schrieb Leonidas P. Papadakos: > Assign the LED's to heartbeat and sdcard io, as in other RK boards. > > https://github.com/armbian/build/commit/f1affad5c7be62d6e93832af3556c7609edd0858 > > Signed-off-by: Leonidas P. Papadakos <papadakospan@xxxxxxxxx> > --- > .../arm64/boot/dts/rockchip/rk3328-roc-cc.dts | 27 +++++++++++++++++++ > 1 file changed, 27 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3328-roc-cc.dts b/arch/arm64/boot/dts/rockchip/rk3328-roc-cc.dts > index 33c44e857..a881d4c8d 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3328-roc-cc.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3328-roc-cc.dts > @@ -81,6 +81,33 @@ > regulator-always-on; > regulator-boot-on; > }; > + > + leds { > + compatible = "gpio-leds"; > + power { > + label = "firefly:blue:power"; > + linux,default-trigger = "heartbeat"; > + gpios = <&rk805 1 GPIO_ACTIVE_LOW>; > + default-state = "on"; > + mode = <0x23>; > + }; > + > + user { > + label = "firefly:yellow:user"; > + linux,default-trigger = "mmc1"; > + gpios = <&rk805 0 GPIO_ACTIVE_LOW>; > + default-state = "off"; > + mode = <0x05>; > + }; > + > + ir { > + status = "disabled"; please explain that. Is this an IR transmitting LED? And why are you adding a disabled property? Similar to how the kernel should not contain commented out code having a disabled node in the actual board file is strange. Heiko > + /* gpios = <&gpio2 GPIO_C2 GPIO_ACTIVE_HIGH>; */ > + linux,default-trigger = "ir"; > + default-state = "off"; > + mode = <0x00>; > + }; > + }; > }; > > &cpu0 { > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/linux-rockchip
