diff mbox series

[v2] arm64: dts: rockchip: pwm-leds for Orange Pi 5

Message ID 20241024012726.1730-2-honyuenkwun@gmail.com (mailing list archive)
State New
Headers show
Series [v2] arm64: dts: rockchip: pwm-leds for Orange Pi 5 | expand

Commit Message

Jimmy Hon Oct. 24, 2024, 1:27 a.m. UTC 
The green LED RUN is driven by PWM0_LED that can be controlled by PWM0_M2

Fixes: b6bc755d806e ("arm64: dts: rockchip: Add Orange Pi 5")
Signed-off-by: Jimmy Hon <honyuenkwun@gmail.com>
---
v2: add subsystem prefix to header

Based off of v6.12-armsoc/dtsfixes in the same spirit as the clean up of
enable-active-low
This change will also avoid the DTB warning coming from using the id "leds-gpio"
This commit implements Ondřej Jirman's suggestion from the first patch series. [1]

[1] https://patchwork.kernel.org/project/linux-rockchip/patch/a1eca379d151c3f91f4cd4e1751ba389096c4f13.1692102057.git.efectn@6tel.net/#25472495

 .../boot/dts/rockchip/rk3588s-orangepi-5.dts  | 26 +++++++++----------
 1 file changed, 13 insertions(+), 13 deletions(-)

Comments

Heiko Stuebner Oct. 24, 2024, 8:30 a.m. UTC | #1 
On Wed, 23 Oct 2024 20:27:27 -0500, Jimmy Hon wrote:
> The green LED RUN is driven by PWM0_LED that can be controlled by PWM0_M2
> 
> 

Applied, thanks!

[1/1] arm64: dts: rockchip: pwm-leds for Orange Pi 5
      commit: 012d72a00b535694093d2d0935764b6a009d1ffd

Best regards,
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-orangepi-5.dts b/arch/arm64/boot/dts/rockchip/rk3588s-orangepi-5.dts
index 6b77be643249..c78eb298a017 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-orangepi-5.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-orangepi-5.dts
@@ -36,15 +36,15 @@  button-recovery {
 		};
 	};
 
-	leds {
-		compatible = "gpio-leds";
-		pinctrl-names = "default";
-		pinctrl-0 = <&leds_gpio>;
+	pwm-leds {
+		compatible = "pwm-leds";
 
-		led-1 {
-			gpios = <&gpio1 RK_PA2 GPIO_ACTIVE_HIGH>;
-			label = "status_led";
+		led {
+			color = <LED_COLOR_ID_GREEN>;
+			function = LED_FUNCTION_STATUS;
 			linux,default-trigger = "heartbeat";
+			max-brightness = <255>;
+			pwms = <&pwm0 0 25000 0>;
 		};
 	};
 
@@ -297,12 +297,6 @@  &pcie2x1l2 {
 };
 
 &pinctrl {
-	gpio-func {
-		leds_gpio: leds-gpio {
-			rockchip,pins = <0 RK_PA2 RK_FUNC_GPIO &pcfg_pull_none>;
-		};
-	};
-
 	hym8563 {
 		hym8563_int: hym8563-int {
 			rockchip,pins = <0 RK_PB0 RK_FUNC_GPIO &pcfg_pull_none>;
@@ -320,6 +314,12 @@  typec5v_pwren: typec5v-pwren {
 	};
 };
 
+&pwm0 {
+	pinctrl-0 = <&pwm0m2_pins>;
+	pinctrl-names = "default";
+	status = "okay";
+};
+
 &saradc {
 	vref-supply = <&avcc_1v8_s0>;
 	status = "okay";