* Re: [PATCH v2 8/8] arm64: dts: qcom: eliza: Add support for MM clock controllers
[not found] ` <b280ad04-d4ae-4904-9e99-3d057e3d221b@kernel.org>
@ 2026-05-12 8:52 ` Taniya Das
2026-05-12 9:01 ` Bryan O'Donoghue
0 siblings, 1 reply; 3+ messages in thread
From: Taniya Das @ 2026-05-12 8:52 UTC (permalink / raw)
To: Bryan O'Donoghue, Bjorn Andersson, Michael Turquette,
Stephen Boyd, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Konrad Dybcio, Maxime Coquelin, Alexandre Torgue
Cc: Ajit Pandey, Imran Shaik, Jagadeesh Kona, linux-arm-msm,
linux-clk, devicetree, linux-kernel, linux-stm32,
linux-arm-kernel
On 4/10/2026 1:56 PM, Bryan O'Donoghue wrote:
> On 10/04/2026 04:55, Taniya Das wrote:
>>> Why do these two controllers have no power-domains ?
>> Bryan, on Eliza the videocc and camcc are connected on CX and MXA.
>
> Shouldn't you at least have:
>
> power-domains = <&rpmhpd RPMHPD_CX> ?
>
> And even
>
> power-domains = <&rpmhpd RPMHPD_MX>,
> <&rpmhpd RPMHPD_CX>;
> power-domain-names = "mx",
> "cx";
>
> Konrad's suggestion to me was that MXA should have a vote in my CSIPHY
> series I think he and Jagadeesh discussed it but I'm not sure if they
> _concluded_ what was the right thing to do.
>
> Right now I'm representing the dependency. MXA is always on ... and
> there's nothing to do voting for it @ MX ?
>
The clock controller drivers only request the minimum operating level
for the power domains. Since the cx and mx rails are already at the
minimum operating level when APPS is active, explicit voting for these
power domains is not required from camcc.
--
Thanks,
Taniya Das
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2 3/8] dt-bindings: clock: qcom: Add support for CAMCC for Eliza
[not found] ` <20260410-hasty-pony-of-tempering-4f0a47@quoll>
@ 2026-05-12 8:54 ` Taniya Das
0 siblings, 0 replies; 3+ messages in thread
From: Taniya Das @ 2026-05-12 8:54 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Bjorn Andersson, Michael Turquette, Stephen Boyd, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Konrad Dybcio, Maxime Coquelin,
Alexandre Torgue, Ajit Pandey, Imran Shaik, Jagadeesh Kona,
linux-arm-msm, linux-clk, devicetree, linux-kernel, linux-stm32,
linux-arm-kernel
On 4/10/2026 1:17 PM, Krzysztof Kozlowski wrote:
> On Thu, Apr 09, 2026 at 11:40:44PM +0530, Taniya Das wrote:
>> Update the compatible and the bindings for CAMCC support on Eliza SoC.
>
> I do not see any update here. Also, no improvements after v1 comments.
Sorry for missing that update the reason for Eliza not compatible with
Pakala. I will update the commit in the next patch.
--
Thanks,
Taniya Das
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v2 8/8] arm64: dts: qcom: eliza: Add support for MM clock controllers
2026-05-12 8:52 ` [PATCH v2 8/8] arm64: dts: qcom: eliza: Add support for MM clock controllers Taniya Das
@ 2026-05-12 9:01 ` Bryan O'Donoghue
0 siblings, 0 replies; 3+ messages in thread
From: Bryan O'Donoghue @ 2026-05-12 9:01 UTC (permalink / raw)
To: Taniya Das, Bjorn Andersson, Michael Turquette, Stephen Boyd,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Konrad Dybcio,
Maxime Coquelin, Alexandre Torgue
Cc: Ajit Pandey, Imran Shaik, Jagadeesh Kona, linux-arm-msm,
linux-clk, devicetree, linux-kernel, linux-stm32,
linux-arm-kernel
On 12/05/2026 09:52, Taniya Das wrote:
> The clock controller drivers only request the minimum operating level
> for the power domains. Since the cx and mx rails are already at the
> minimum operating level when APPS is active, explicit voting for these
> power domains is not required from camcc.
>
> --
> Thanks,
> Taniya Das
Great, I'll drop those references.
Thank you for the information.
---
bod
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-05-12 9:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20260409-eliza_mm_cc_v2-v2-0-bc0c6dd77bc5@oss.qualcomm.com>
[not found] ` <20260409-eliza_mm_cc_v2-v2-8-bc0c6dd77bc5@oss.qualcomm.com>
[not found] ` <cb5a40e8-e2e3-4ed9-a9c6-0daa9f408710@nxsw.ie>
[not found] ` <Z8a4y73p6MgF8gQkVtottkgG2Ok8-Vhm8zzckWEbiU_DUO84x2VlwPt-VBL2g0utw3ZwZK0aUvI4ev-p_YRGoQ==@protonmail.internalid>
[not found] ` <328b388c-438e-4f91-9384-0dad903355a5@oss.qualcomm.com>
[not found] ` <b280ad04-d4ae-4904-9e99-3d057e3d221b@kernel.org>
2026-05-12 8:52 ` [PATCH v2 8/8] arm64: dts: qcom: eliza: Add support for MM clock controllers Taniya Das
2026-05-12 9:01 ` Bryan O'Donoghue
[not found] ` <20260409-eliza_mm_cc_v2-v2-3-bc0c6dd77bc5@oss.qualcomm.com>
[not found] ` <20260410-hasty-pony-of-tempering-4f0a47@quoll>
2026-05-12 8:54 ` [PATCH v2 3/8] dt-bindings: clock: qcom: Add support for CAMCC for Eliza Taniya Das
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox