Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <8c771e3a76ba1c41bbd955aeabe06a5f70abcece.camel@dylanvanassche.be>
Subject: Re: [PATCH 0/4] arm64: dts: qcom: Add a few LPG LEDs
From:   Dylan Van Assche <me@dylanvanassche.be>
To:     Bjorn Andersson <bjorn.andersson@linaro.org>
Cc:     linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Date:   Fri, 01 Jul 2022 08:08:18 +0200
In-Reply-To: <20220505022706.1692554-1-bjorn.andersson@linaro.org>
References: <20220505022706.1692554-1-bjorn.andersson@linaro.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Precedence: bulk

On Wed, 2022-05-04 at 19:27 -0700, Bjorn Andersson wrote:
> Now that the LPG binding is accepted, sprinkle some lpg definitions
> in various
> PMIC files and define the LPG attached LEDs for RB3 and db820c.
>=20
> Bjorn Andersson (4):
> =C2=A0 arm64: dts: qcom: Add LPG to pm8916, pm8994, pmi8994 and pmi8998
> =C2=A0 arm64: dts: qcom: sdm845: Enable user LEDs on DB845c
> =C2=A0 arm64: dts: qcom: pmi8994: Define MPP block
> =C2=A0 arm64: dts: qcom: db820c: Add user LEDs
>=20
> =C2=A0arch/arm64/boot/dts/qcom/apq8096-db820c.dts | 56
> +++++++++++++++++++++
> =C2=A0arch/arm64/boot/dts/qcom/pm8916.dtsi=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 |=C2=A0 8 +++
> =C2=A0arch/arm64/boot/dts/qcom/pm8994.dtsi=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0=C2=A0 | 10 ++++
> =C2=A0arch/arm64/boot/dts/qcom/pmi8994.dtsi=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0 | 20 ++++++++
> =C2=A0arch/arm64/boot/dts/qcom/pmi8998.dtsi=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=
=C2=A0 | 11 +++-
> =C2=A0arch/arm64/boot/dts/qcom/sdm845-db845c.dts=C2=A0 | 31 ++++++++++++
> =C2=A06 files changed, 135 insertions(+), 1 deletion(-)
>=20

Changes look good to me, I applied similar changes here:
https://patchwork.kernel.org/project/linux-arm-msm/patch/20220512054439.139=
71-1-me@dylanvanassche.be/

Reviewed-by: Dylan Van Assche <me@dylanvanassche.be>

Thanks!
Dylan