[PATCH v2 08/18] arm64: dts: rockchip: add hdd leds to Qnap-TS433

[Kever Yang]

On 2024/10/14 03:24, Heiko Stuebner wrote:
> Add the 4 gpio-controlled LEDs to the Qnap-TS433.
>
> They are meant for individual disk activitivy, but I haven't found a
> way for how to connect them to their individual sata slot yet.
>
> Tested-by: Uwe Kleine-König <ukleinek at debian.org>
> Signed-off-by: Heiko Stuebner <heiko at sntech.de>
> Link: https://lore.kernel.org/r/20240723195538.1133436-9-heiko@sntech.de
>
> [ upstream commit: ea91aabf18bcad6f5eceae6848ea6570ea61f126 ]
>
> (cherry picked from commit 5a11b1bb40ac7b39e04077c045c3e3409fa352e2)
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
>   .../src/arm64/rockchip/rk3568-qnap-ts433.dts  | 59 +++++++++++++++++++
>   1 file changed, 59 insertions(+)
>
> diff --git a/dts/upstream/src/arm64/rockchip/rk3568-qnap-ts433.dts b/dts/upstream/src/arm64/rockchip/rk3568-qnap-ts433.dts
> index 8be36250aa1..abeb00add42 100644
> --- a/dts/upstream/src/arm64/rockchip/rk3568-qnap-ts433.dts
> +++ b/dts/upstream/src/arm64/rockchip/rk3568-qnap-ts433.dts
> @@ -6,6 +6,7 @@
>   
>   /dts-v1/;
>   
> +#include <dt-bindings/leds/common.h>
>   #include <dt-bindings/gpio/gpio.h>
>   #include "rk3568.dtsi"
>   
> @@ -23,6 +24,46 @@
>   		stdout-path = "serial2:115200n8";
>   	};
>   
> +	leds {
> +		compatible = "gpio-leds";
> +
> +		led-0 {
> +			color = <LED_COLOR_ID_GREEN>;
> +			function = LED_FUNCTION_DISK;
> +			gpios = <&gpio1 RK_PD5 GPIO_ACTIVE_LOW>;
> +			linux,default-trigger = "disk-activity";
> +			pinctrl-names = "default";
> +			pinctrl-0 = <&hdd1_led_pin>;
> +		};
> +
> +		led-1 {
> +			color = <LED_COLOR_ID_GREEN>;
> +			function = LED_FUNCTION_DISK;
> +			gpios = <&gpio1 RK_PD6 GPIO_ACTIVE_LOW>;
> +			linux,default-trigger = "disk-activity";
> +			pinctrl-names = "default";
> +			pinctrl-0 = <&hdd2_led_pin>;
> +		};
> +
> +		led-2 {
> +			color = <LED_COLOR_ID_GREEN>;
> +			function = LED_FUNCTION_DISK;
> +			gpios = <&gpio1 RK_PD7 GPIO_ACTIVE_LOW>;
> +			linux,default-trigger = "disk-activity";
> +			pinctrl-names = "default";
> +			pinctrl-0 = <&hdd3_led_pin>;
> +		};
> +
> +		led-3 {
> +			color = <LED_COLOR_ID_GREEN>;
> +			function = LED_FUNCTION_DISK;
> +			gpios = <&gpio2 RK_PA0 GPIO_ACTIVE_LOW>;
> +			linux,default-trigger = "disk-activity";
> +			pinctrl-names = "default";
> +			pinctrl-0 = <&hdd4_led_pin>;
> +		};
> +	};
> +
>   	dc_12v: regulator-dc-12v {
>   		compatible = "regulator-fixed";
>   		regulator-name = "dc_12v";
> @@ -161,6 +202,24 @@
>   };
>   
>   &pinctrl {
> +	leds {
> +		hdd1_led_pin: hdd1-led-pin {
> +			rockchip,pins = <1 RK_PD5 RK_FUNC_GPIO &pcfg_pull_up>;
> +		};
> +
> +		hdd2_led_pin: hdd2-led-pin {
> +			rockchip,pins = <1 RK_PD6 RK_FUNC_GPIO &pcfg_pull_up>;
> +		};
> +
> +		hdd3_led_pin: hdd3-led-pin {
> +			rockchip,pins = <1 RK_PD7 RK_FUNC_GPIO &pcfg_pull_up>;
> +		};
> +
> +		hdd4_led_pin: hdd4_led-pin {
> +			rockchip,pins = <2 RK_PA0 RK_FUNC_GPIO &pcfg_pull_up>;
> +		};
> +	};
> +
>   	usb {
>   		vcc5v0_host_en: vcc5v0-host-en {
>   			rockchip,pins = <0 RK_PA6 RK_FUNC_GPIO &pcfg_pull_none>;