linux-kernel - Re: [PATCH 18/24] arm64: dts: qcom: glymur: Add PMIC glink node

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] [day] [month] [year] [list]

Message-ID: <CADhhZXaD=ut7MCQD_uEvY1ew7o=rqUUtviaXwQSkE-nmvCxMhg@mail.gmail.com>
Date: Mon, 10 Nov 2025 19:36:01 +0530
From: Kamal Wadhwa <kamal.wadhwa@....qualcomm.com>
To: Pankaj Patil <pankaj.patil@....qualcomm.com>
Cc: Konrad Dybcio <konrad.dybcio@....qualcomm.com>,
        Bjorn Andersson <andersson@...nel.org>,
        Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>, linux-arm-msm@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 18/24] arm64: dts: qcom: glymur: Add PMIC glink node

Hi Konrad,

On Mon, Nov 3, 2025 at 3:56 PM Kamal Wadhwa
<kamal.wadhwa@....qualcomm.com> wrote:
>
> On Wed, Oct 08, 2025 at 05:25:39PM +0530, Pankaj Patil wrote:
> > On 9/25/2025 4:02 PM, Konrad Dybcio wrote:
> > > On 9/25/25 8:32 AM, Pankaj Patil wrote:
> > >> From: Kamal Wadhwa <kamal.wadhwa@....qualcomm.com>
> > >>
> > >> Add the pmic glink node with connectors.
> > >>
> > >> Signed-off-by: Kamal Wadhwa <kamal.wadhwa@....qualcomm.com>
> > >> Signed-off-by: Pankaj Patil <pankaj.patil@....qualcomm.com>
> > >> ---
> > >>  arch/arm64/boot/dts/qcom/glymur-crd.dts | 28 ++++++++++++++++++++++++++++
> > >>  1 file changed, 28 insertions(+)
> > >>
> > >> diff --git a/arch/arm64/boot/dts/qcom/glymur-crd.dts b/arch/arm64/boot/dts/qcom/glymur-crd.dts
> > >> index b04c0ed28468620673237fffb4013adacc7ef7ba..3f94bdf8b3ccfdff182005d67b8b3f84f956a430 100644
> > >> --- a/arch/arm64/boot/dts/qcom/glymur-crd.dts
> > >> +++ b/arch/arm64/boot/dts/qcom/glymur-crd.dts
> > >> @@ -79,6 +79,34 @@ key-volume-up {
> > >>                    wakeup-source;
> > >>            };
> > >>    };
> > >> +
> > >> +  pmic-glink {
> > >> +          compatible = "qcom,sm8550-pmic-glink",
> > > You *must* include a glymur compatible
> > >
> > >> +                       "qcom,pmic-glink";
> > > Are you sure this is still compatible with 8550 after this
> > > series landed?
> > >
> > > https://lore.kernel.org/linux-arm-msm/20250917-qcom_battmgr_update-v5-0-270ade9ffe13@oss.qualcomm.com/
>
> Sorry for late reply, earlier when we were sending this series our understanding
> was that we only need to support for usb shell, and anyway the device was on the
> debug board so this feature(battery/charging) was not tested.
>
> However, after testing i found that the power supplies are getting registered
> properly however the data is not coming as expected. we are working to fix this
> internally from the firmware guys.

As you had asked,  i was able to check with this patch (and firmware fixes)
https://lore.kernel.org/linux-arm-msm/20250917-qcom_battmgr_update-v5-0-270ade9ffe13@oss.qualcomm.com/

The qcom_battmngr driver is working fine with this patch included as well.
(though i faced some conflicts which i had to manually fix when
pulling this patch on latest linux-next tag)