[PATCH] arch: arm64: qcom: sdm845-xiaomi-beryllium: add notification LED

Joel Selvaraj posted 1 patch 2 years, 11 months ago
.../boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi | 11 +++++++++++
1 file changed, 11 insertions(+)
[PATCH] arch: arm64: qcom: sdm845-xiaomi-beryllium: add notification LED
Posted by Joel Selvaraj 2 years, 11 months ago
Enable the Qualcomm Light Pulse Generator (LPG) driver based
notification LED.

Signed-off-by: Joel Selvaraj <joelselvaraj.oss@gmail.com>
---
 .../boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
index e0fda4d754fe..9d11502e5e0e 100644
--- a/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-common.dtsi
@@ -2,6 +2,7 @@
 
 /dts-v1/;
 
+#include <dt-bindings/leds/common.h>
 #include <dt-bindings/pinctrl/qcom,pmic-gpio.h>
 #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
 #include <dt-bindings/sound/qcom,q6afe.h>
@@ -322,6 +323,16 @@ vol_up_pin_a: vol-up-active-state {
 	};
 };
 
+&pmi8998_lpg {
+	status = "okay";
+
+	led@5 {
+		reg = <5>;
+		color = <LED_COLOR_ID_WHITE>;
+		function = LED_FUNCTION_STATUS;
+	};
+};
+
 &pmi8998_wled {
 	status = "okay";
 	qcom,current-boost-limit = <970>;
-- 
2.39.2
Re: [PATCH] arch: arm64: qcom: sdm845-xiaomi-beryllium: add notification LED
Posted by Joel Selvaraj 2 years, 11 months ago
Hi,

The commit title should have had the prefix "arm64: dts: qcom:" instead.
Sorry for the inconsistency. Shall I send a v2 with the commit title fixed?

Regards,
Joel
Re: [PATCH] arch: arm64: qcom: sdm845-xiaomi-beryllium: add notification LED
Posted by Konrad Dybcio 2 years, 11 months ago

On 13.03.2023 07:56, Joel Selvaraj wrote:
> Hi,
> 
> The commit title should have had the prefix "arm64: dts: qcom:" instead.
> Sorry for the inconsistency. Shall I send a v2 with the commit title fixed?
Resending one commit does not seem like a lot of work, so yes please

Konrad
> 
> Regards,
> Joel