Re: [PATCH v2 3/8] arm64: dts: allwinner: Pine64-LTS: Add status LED
From: Maxime Ripard <hidden>
Date: 2021-01-14 11:50:08
Also in:
linux-arm-kernel
Hi, On Wed, Jan 13, 2021 at 03:26:25PM +0000, Andre Przywara wrote:
quoted hunk ↗ jump to hunk
The Pine64-LTS board features a blue status LED on pin PL7. Describe it in the DT. Signed-off-by: Andre Przywara <andre.przywara@arm.com> --- .../boot/dts/allwinner/sun50i-a64-pine64-lts.dts | 11 +++++++++++ 1 file changed, 11 insertions(+)diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pine64-lts.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pine64-lts.dts index 302e24be0a31..93d271fdbb6c 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pine64-lts.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pine64-lts.dts@@ -1,10 +1,21 @@ // SPDX-License-Identifier: (GPL-2.0+ OR MIT) // Copyright (c) 2018 ARM Ltd. +#include <dt-bindings/leds/common.h> #include "sun50i-a64-sopine-baseboard.dts" / { model = "Pine64 LTS"; compatible = "pine64,pine64-lts", "allwinner,sun50i-r18", "allwinner,sun50i-a64"; + + led { + compatible = "gpio-leds"; + + status { + function = LED_FUNCTION_STATUS; + color = <LED_COLOR_ID_BLUE>; + gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>; /* PL7 */ + };
Thanks for fixing the node names, but I guess my comment was a bit too vague: it's status that needed to be changed, not the upper level one. I've updated it while applying Maxime
Attachments
- signature.asc [application/pgp-signature] 228 bytes