[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <lrbqjf5zxxyjnyfwwk6wniauffi5a3hgkg7jvboybvmrwxyu52@56onam4vvhz4>
Date: Fri, 27 Jun 2025 16:07:01 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Luca Weiss <luca.weiss@...rphone.com>
Cc: Lee Jones <lee@...nel.org>, Pavel Machek <pavel@...nel.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Fenglin Wu <quic_fenglinw@...cinc.com>,
Stephen Boyd <sboyd@...nel.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>,
~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org,
linux-arm-msm@...r.kernel.org, linux-leds@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/3] Add support for PMXR2230 PMIC
On Fri, Jun 27, 2025 at 12:05:08PM +0200, Luca Weiss wrote:
> On Wed Jun 25, 2025 at 11:03 PM CEST, Dmitry Baryshkov wrote:
> > On Wed, Jun 25, 2025 at 05:01:29PM +0200, Luca Weiss wrote:
> >> On Wed Jun 25, 2025 at 4:25 PM CEST, Dmitry Baryshkov wrote:
> >> > On Wed, Jun 25, 2025 at 11:18:36AM +0200, Luca Weiss wrote:
> >> >> The PMXR2230 PMIC is used in conjuction with SM7635. Add binding docs
> >> >> and the devicetree description for it.
> >> >
> >> >
> >> > Please use PM7550 instead.
> >>
> >> I'm happy to not follow downstream naming conventions if being told, but
> >> do you have any details whether PMXR2230 == PM7550, or PM7550 is just
> >> another SW-compatible PMIC as PMXR2230.
> >
> > It is PM7550.
>
> Ok, will update in v2.
>
> >
> >>
> >> Also we already have qcom,pmxr2230-gpio upstream, so that would need to
> >> get updated for the PM7550 name.
> >
> > No, leave it be. We should not change existing compats for no reason.
>
> The compatible is not used in upstream dts though, and cleaning up some
> old naming for the PMIC would be good, no?
Then, I'd say, just use pm7550 everywhere, making the qcom,pmxr2230-gpio
a one-off unused entry, which we might drop at some point.
>
> But I can leave it for now, and we can potentially take care of it
> later.
>
> Regards
> Luca
>
> >
> >>
> >> Regards
> >> Luca
> >>
> >> >
> >> >>
> >> >> Signed-off-by: Luca Weiss <luca.weiss@...rphone.com>
> >> >> ---
> >> >> Luca Weiss (3):
> >> >> dt-bindings: leds: qcom,spmi-flash-led: Add PMXR2230
> >> >> dt-bindings: mfd: qcom-spmi-pmic: Document PMXR2230 PMIC
> >> >> arm64: dts: qcom: Add PMXR2230 PMIC
> >> >>
> >> >> .../bindings/leds/qcom,spmi-flash-led.yaml | 1 +
> >> >> .../devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 1 +
> >> >> arch/arm64/boot/dts/qcom/pmxr2230.dtsi | 63 ++++++++++++++++++++++
> >> >> 3 files changed, 65 insertions(+)
> >> >> ---
> >> >> base-commit: 19272b37aa4f83ca52bdf9c16d5d81bdd1354494
> >> >> change-id: 20250620-sm7635-pmxr2230-ee55a86a8c2b
> >> >>
> >> >> Best regards,
> >> >> --
> >> >> Luca Weiss <luca.weiss@...rphone.com>
> >> >>
> >>
>
--
With best wishes
Dmitry
Powered by blists - more mailing lists