[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <225b94a0-eea5-4061-aebd-da497d349527@oss.qualcomm.com>
Date: Wed, 25 Jun 2025 00:03:40 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Luca Weiss <luca@...aweiss.eu>
Cc: ~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org,
Bjorn Andersson <andersson@...nel.org>,
Mathieu Poirier <mathieu.poirier@...aro.org>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley
<conor+dt@...nel.org>,
Manivannan Sadhasivam <mani@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>, linux-arm-msm@...r.kernel.org,
linux-remoteproc@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/4] Start using rpmpd for power domains on MSM8974
On 24/06/2025 21:46, Luca Weiss wrote:
> Hi Dmitry,
>
> On 24-06-2025 2:59 a.m., Dmitry Baryshkov wrote:
>> On Sat, Jun 21, 2025 at 03:19:55PM +0200, Luca Weiss wrote:
>>> Switch over the ADSP PIL to use power-domains instead of a regulator,
>>> and have one commit switching over the MSM8974 SoC plus all the devices
>>> to use power-domains.
>>>
>>> Note, that I'm aware that these changes are not backwards compatible and
>>> not really bisectable, but since it only affects the ADSP on these
>>
>> Why? The cx-supply is handled unconditionally. A single-domain usecase
>> is also handled via a special code path. I think this might be
>> backwards-compatible, by the pure luck.
>
> Honestly I have not tried and not looked much. I mostly added this
> paragraph to avoid the response that this change might break and is not
> really backwards compatible. If it does (by accident) work with the
> updated dts without the driver and the other way around, then even better.
I think it's worth checking that new kernel works with older DTS (that's
the usual rule). The platform doesn't have many users upstream, but I
think it has been used by some PmOS users, which might mean
not-yet-upstreamed DT.
>
> Regards
> Luca
>
>>
>>> pretty old boards, I say it's fine to have this. Get all the patches
>>> into the same release (6.17?) and then we're good again.
>>>
>>> Signed-off-by: Luca Weiss <luca@...aweiss.eu>
>>> ---
>>> Luca Weiss (4):
>>> dt-bindings: remoteproc: qcom,adsp: Make msm8974 use CX as
>>> power domain
>>> remoteproc: qcom_q6v5_pas: Use resource with CX PD for MSM8974
>>> ARM: dts: qcom: msm8974: Sort header includes alphabetically
>>> ARM: dts: qcom: msm8974: Start using rpmpd for power domains
>>>
>>> .../devicetree/bindings/remoteproc/qcom,adsp.yaml | 18 ++------
>>> .../arm/boot/dts/qcom/qcom-apq8074-dragonboard.dts | 13 ------
>>> .../qcom/qcom-msm8974-lge-nexus5-hammerhead.dts | 12 ------
>>> .../boot/dts/qcom/qcom-msm8974-samsung-hlte.dts | 12 ------
>>> .../dts/qcom/qcom-msm8974-sony-xperia-rhine.dtsi | 12 ------
>>> arch/arm/boot/dts/qcom/qcom-msm8974.dtsi | 50 +++++++++++
>>> +++++++++--
>>> .../dts/qcom/qcom-msm8974pro-fairphone-fp2.dts | 8 ----
>>> arch/arm/boot/dts/qcom/qcom-msm8974pro-htc-m8.dts | 11 -----
>>> .../dts/qcom/qcom-msm8974pro-oneplus-bacon.dts | 9 ----
>>> .../qcom/qcom-msm8974pro-samsung-klte-common.dtsi | 11 ++---
>>> ...qcom-msm8974pro-sony-xperia-shinano-common.dtsi | 12 ------
>>> drivers/remoteproc/qcom_q6v5_pas.c | 2 +-
>>> 12 files changed, 56 insertions(+), 114 deletions(-)
>>> ---
>>> base-commit: 7fa2fb97cd28e1d9670da538095565b6fba83977
>>> change-id: 20250621-msm8974-rpmpd-switch-b19b166c02be
>>>
>>> Best regards,
>>> --
>>> Luca Weiss <luca@...aweiss.eu>
>>>
>>
>
--
With best wishes
Dmitry
Powered by blists - more mailing lists