Re: [PATCH 04/12] arm64: dts: qcom: sm6115-pro1x: Add Caps Lock LED

From: Konrad Dybcio
Date: Mon Jul 22 2024 - 04:38:13 EST


On 22.07.2024 9:10 AM, Dang Huynh wrote:
> The Pro1X has a caps lock LED on the keyboard.
>
> Signed-off-by: Dang Huynh <danct12@xxxxxxxxxx>
> ---
> arch/arm64/boot/dts/qcom/sm6115-fxtec-pro1x.dts | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sm6115-fxtec-pro1x.dts b/arch/arm64/boot/dts/qcom/sm6115-fxtec-pro1x.dts
> index d91d31646b29..60c046fe8e52 100644
> --- a/arch/arm64/boot/dts/qcom/sm6115-fxtec-pro1x.dts
> +++ b/arch/arm64/boot/dts/qcom/sm6115-fxtec-pro1x.dts
> @@ -8,6 +8,7 @@
> #include "sm6115.dtsi"
> #include "pm6125.dtsi"
> #include <dt-bindings/arm/qcom,ids.h>
> +#include <dt-bindings/leds/common.h>
>
> / {
> model = "F(x)tec Pro1X (QX1050)";
> @@ -63,6 +64,16 @@ key-volume-up {
> };
> };
>
> + gpio-leds {
> + compatible = "gpio-leds";
> +
> + capslock-led {
> + color = <LED_COLOR_ID_GREEN>;
> + function = LED_FUNCTION_CAPSLOCK;
> + gpios = <&pca9534 1 GPIO_ACTIVE_HIGH>;

You may want to take a look at msm8998-fxtec-pro1.dts which has a
bit more properties describing it

Konrad