linux-kernel - Re: [PATCH] arm64: dts: qcom/sdm845-shift-axolotl: Enable pmi9889 LPG LED

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3b011dfa0aa2a880a113e005a49e2e126c1bef67.camel@dylanvanassche.be>
Date:   Fri, 01 Jul 2022 08:09:17 +0200
From:   Dylan Van Assche <me@...anvanassche.be>
To:     Bjorn Andersson <bjorn.andersson@...aro.org>
Cc:     Andy Gross <agross@...nel.org>, Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        ~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom/sdm845-shift-axolotl: Enable pmi9889
 LPG LED

On Thu, 2022-06-30 at 16:56 -0500, Bjorn Andersson wrote:
> On Thu 12 May 00:44 CDT 2022, Dylan Van Assche wrote:
> 
> > Enables the RGB notification LED on the SHIFT 6mq (sdm845-shift-
> > axolotl)
> > with the Qualcomm Light Pulse Generator bindings by Bjorn Andersson
> > [1].
> > Patches are merged in for-next branch of linux-leds.
> > Tested these changes on the SHIFT 6mq.
> > 
> > [1]
> > https://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds.git/commit/?h=for-next&id=a8e53db46f19f67be6a26488aafb7d10c78e33bd
> > 
> > Signed-off-by: Dylan Van Assche <me@...anvanassche.be>
> > ---
> >  .../boot/dts/qcom/sdm845-shift-axolotl.dts    | 29
> > +++++++++++++++++++
> >  1 file changed, 29 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> > b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> > index 847f6217a77b..af412ac2c9d0 100644
> > --- a/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> > +++ b/arch/arm64/boot/dts/qcom/sdm845-shift-axolotl.dts
> > @@ -2,11 +2,13 @@
> >  /*
> >   * Copyright (c) 2022, Alexander Martinz
> > <amartinz@...ftphones.com>
> >   * Copyright (c) 2022, Caleb Connolly <caleb@...nolly.tech>
> > + * Copyright (c) 2022, Dylan Van Assche <me@...anvanassche.be>
> >   */
> >  
> >  /dts-v1/;
> >  
> >  #include <dt-bindings/gpio/gpio.h>
> > +#include <dt-bindings/leds/common.h>
> >  #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> >  #include "sdm845.dtsi"
> >  #include "pm8998.dtsi"
> > @@ -554,6 +556,33 @@ &pmi8998_smb2 {
> >         monitored-battery = <&battery>;
> >  };
> >  
> > +&pmi8998_lpg {
> 
> We don't have this node, because no one has reviewed
> https://lore.kernel.org/all/20220505022706.1692554-1-bjorn.andersson@linaro.org/#t
> yet.
> 
> If anyone could take a look I'd be happy to merge this as well.
> 
> Thanks,
> Bjorn
> 

I had a look, looks fine to me so I sent a Reviewed-by out.
Thanks!

Dylan

> > +       status = "okay";
> > +
> > +       multi-led {
> > +               color = <LED_COLOR_ID_RGB>;
> > +               function = LED_FUNCTION_STATUS;
> > +
> > +               #address-cells = <1>;
> > +               #size-cells = <0>;
> > +
> > +               led@3 {
> > +                       reg = <3>;
> > +                       color = <LED_COLOR_ID_BLUE>;
> > +               };
> > +
> > +               led@4 {
> > +                       reg = <4>;
> > +                       color = <LED_COLOR_ID_GREEN>;
> > +               };
> > +
> > +               led@5 {
> > +                       reg = <5>;
> > +                       color = <LED_COLOR_ID_RED>;
> > +               };
> > +       };
> > +};
> > +
> >  &qup_uart9_default {
> >         pinconf-rx {
> >                 pins = "gpio5";
> > -- 
> > 2.36.1
> >