| Message ID | 20201110153831.5336-1-tmn505@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: mcbin-singleshot: add heartbeat LED | expand |
Hi, > With board revision 1.3, SolidRun moved the power LED to the middle of > the board. In old place of power LED a GPIO controllable heartbeat LED > was added. This commit only touches Single Shot variant, since only this > variant is all revision 1.3. > > Reported-by: Alexandra Alth <alexandra@alth.de> > Signed-off-by: Tomasz Maciej Nowak <tmn505@gmail.com> Applied on mvebu/dt64 Thanks, Gregory > --- > .../marvell/armada-8040-mcbin-singleshot.dts | 22 +++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts > index 2e6832d02a59..411d20064271 100644 > --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts > +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts > @@ -5,6 +5,8 @@ > * Device Tree file for MACCHIATOBin Armada 8040 community board platform > */ > > +#include <dt-bindings/leds/common.h> > + > #include "armada-8040-mcbin.dtsi" > > / { > @@ -12,6 +14,19 @@ / { > compatible = "marvell,armada8040-mcbin-singleshot", > "marvell,armada8040-mcbin", "marvell,armada8040", > "marvell,armada-ap806-quad", "marvell,armada-ap806"; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-0 = <&cp0_led18_pins>; > + pinctrl-names = "default"; > + > + led18 { > + gpios = <&cp0_gpio2 1 GPIO_ACTIVE_LOW>; > + function = LED_FUNCTION_HEARTBEAT; > + color = <LED_COLOR_ID_GREEN>; > + linux,default-trigger = "heartbeat"; > + }; > + }; > }; > > &cp0_eth0 { > @@ -27,3 +42,10 @@ &cp1_eth0 { > managed = "in-band-status"; > sfp = <&sfp_eth1>; > }; > + > +&cp0_pinctrl { > + cp0_led18_pins: led18-pins { > + marvell,pins = "mpp33"; > + marvell,function = "gpio"; > + }; > +}; > -- > 2.29.2 >
diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts index 2e6832d02a59..411d20064271 100644 --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts @@ -5,6 +5,8 @@ * Device Tree file for MACCHIATOBin Armada 8040 community board platform */ +#include <dt-bindings/leds/common.h> + #include "armada-8040-mcbin.dtsi" / { @@ -12,6 +14,19 @@ / { compatible = "marvell,armada8040-mcbin-singleshot", "marvell,armada8040-mcbin", "marvell,armada8040", "marvell,armada-ap806-quad", "marvell,armada-ap806"; + + leds { + compatible = "gpio-leds"; + pinctrl-0 = <&cp0_led18_pins>; + pinctrl-names = "default"; + + led18 { + gpios = <&cp0_gpio2 1 GPIO_ACTIVE_LOW>; + function = LED_FUNCTION_HEARTBEAT; + color = <LED_COLOR_ID_GREEN>; + linux,default-trigger = "heartbeat"; + }; + }; }; &cp0_eth0 { @@ -27,3 +42,10 @@ &cp1_eth0 { managed = "in-band-status"; sfp = <&sfp_eth1>; }; + +&cp0_pinctrl { + cp0_led18_pins: led18-pins { + marvell,pins = "mpp33"; + marvell,function = "gpio"; + }; +};
With board revision 1.3, SolidRun moved the power LED to the middle of the board. In old place of power LED a GPIO controllable heartbeat LED was added. This commit only touches Single Shot variant, since only this variant is all revision 1.3. Reported-by: Alexandra Alth <alexandra@alth.de> Signed-off-by: Tomasz Maciej Nowak <tmn505@gmail.com> --- .../marvell/armada-8040-mcbin-singleshot.dts | 22 +++++++++++++++++++ 1 file changed, 22 insertions(+)