Re: [PATCH 5/5] ARM: dts: qcom: msm8974-oneplus-bacon: Add notification LED

Konrad Dybcio
2023-01-23T16:45:13Z

On 22.01.2023 17:48, Luca Weiss wrote:
> Add the node describing the sn3193 that's used to provide notification
> LED.
>
> Unfortunately the driver currently supports neither multicolor API nor
> using the properties function & color, so we use label instead.
>
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
> .../arm/boot/dts/qcom-msm8974pro-oneplus-bacon.dts | 28 ++++++++++++++++++++++
> 1 file changed, 28 insertions(+)
>
> diff --git a/arch/arm/boot/dts/qcom-msm8974pro-oneplus-bacon.dts b/arch/arm/boot/dts/qcom-msm8974pro-oneplus-bacon.dts
> index ffb486ceb6a6..a672c45d7070 100644
> --- a/arch/arm/boot/dts/qcom-msm8974pro-oneplus-bacon.dts
> +++ b/arch/arm/boot/dts/qcom-msm8974pro-oneplus-bacon.dts
> @@ -114,6 +114,34 @@ led@0 {
> default-brightness = <80>;
> };
> };
> +
> + led-controller@68 {
> + compatible = "si-en,sn3193";
> + reg = <0x68>;
> +
> + shutdown-gpios = <&tlmm 45 GPIO_ACTIVE_HIGH>;
> +
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + led@1 {
> + reg = <1>;
> + label = "red:status";
> + led-max-microamp = <17500>;
> + };
> +
> + led@2 {
> + reg = <2>;
> + label = "green:status";
> + led-max-microamp = <17500>;
> + };
> +
> + led@3 {
> + reg = <3>;
> + label = "blue:status";
> + led-max-microamp = <17500>;
> + };
> + };
> };
>
> &blsp1_i2c6 {
>