From f96418d810e0a91228498d968560379064b6accf Mon Sep 17 00:00:00 2001 From: Shilpa Suresh Date: Wed, 16 Oct 2024 17:33:26 +0530 Subject: [PATCH] ARM: dts: msm: Enable tri-led node for Ravelin Add compatible string to led DT node for Ravelin. This was removed from the bulk DT porting for Ravelin on qcom-6.6 device-tree branch. Also, the pwm driver probe fails when the "qcom,num-lpg-channels" value is higher than the number of configured channels in the dtsi for the pwm node. This is because the driver uses the lpg_sdam_base for every channel during probe and fails if not configured correctly in the dtsi. So, change the num-lpg-channels to 3 since only 3 channels are configured and used. Change-Id: I9dbe8a2bf97316fad69ff6a5f9db42a19b839901 Signed-off-by: Shilpa Suresh --- qcom/pmi632.dtsi | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/qcom/pmi632.dtsi b/qcom/pmi632.dtsi index ab5b1805..1d470d30 100644 --- a/qcom/pmi632.dtsi +++ b/qcom/pmi632.dtsi @@ -199,7 +199,7 @@ reg = <0xb300>; reg-names = "lpg-base"; #pwm-cells = <2>; - qcom,num-lpg-channels = <5>; + qcom,num-lpg-channels = <3>; nvmem-names = "ppg_sdam"; nvmem = <&pmi632_sdam7>; qcom,pbs-client = <&pmi632_pbs_client3>; @@ -235,6 +235,7 @@ }; pmi632_rgb: qcom,leds@d000 { + compatible = "qcom,tri-led"; reg = <0xd000>; red { label = "red";