2022-05-09 07:22:10

by Luca Weiss

Subject: [PATCH] ARM: dts: qcom: msm8974-FP2: Add notification LED

FP2 has a RGB LED connected to the TRILED and hence channels 7, 6 and
5 of the LPG. Add a node describing this.

Signed-off-by: Luca Weiss <[email protected]>
---
This patch depends on the PM8941 LPG patch:
https://lore.kernel.org/linux-arm-msm/[email protected]/

.../dts/qcom-msm8974pro-fairphone-fp2.dts | 30 +++++++++++++++++++
1 file changed, 30 insertions(+)

diff --git a/arch/arm/boot/dts/qcom-msm8974pro-fairphone-fp2.dts b/arch/arm/boot/dts/qcom-msm8974pro-fairphone-fp2.dts
index 0700a0008caa..0fad82fc9e03 100644
--- a/arch/arm/boot/dts/qcom-msm8974pro-fairphone-fp2.dts
+++ b/arch/arm/boot/dts/qcom-msm8974pro-fairphone-fp2.dts
@@ -3,6 +3,7 @@
#include "qcom-pm8841.dtsi"
#include "qcom-pm8941.dtsi"
#include <dt-bindings/input/input.h>
+#include <dt-bindings/leds/common.h>
#include <dt-bindings/pinctrl/qcom,pmic-gpio.h>

/ {
@@ -162,6 +163,35 @@ gpio_keys_pin_a: gpio-keys-active {
};
};

+&pm8941_lpg {
+ status = "okay";
+
+ qcom,power-source = <1>;
+
+ multi-led {
+ color = <LED_COLOR_ID_RGB>;
+ function = LED_FUNCTION_STATUS;
+
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ led@7 {
+ reg = <7>;
+ color = <LED_COLOR_ID_RED>;
+ };
+
+ led@6 {
+ reg = <6>;
+ color = <LED_COLOR_ID_GREEN>;
+ };
+
+ led@5 {
+ reg = <5>;
+ color = <LED_COLOR_ID_BLUE>;
+ };
+ };
+};
+
&pronto {
status = "okay";

--
2.36.0

2022-07-17 03:24:06

by Bjorn Andersson

Subject: Re: (subset) [PATCH] ARM: dts: qcom: msm8974-FP2: Add notification LED

On Thu, 5 May 2022 18:30:29 +0200, Luca Weiss wrote:
> FP2 has a RGB LED connected to the TRILED and hence channels 7, 6 and
> 5 of the LPG. Add a node describing this.
>
>

Applied, thanks!

[1/1] ARM: dts: qcom: msm8974-FP2: Add notification LED
commit: a037fcabdbdd8615e17add98c2638df529ff2e3c

Best regards,
--
Bjorn Andersson <[email protected]>