All of lore.kernel.org
 help / color / mirror / Atom feed
From: Luca Weiss <luca@lucaweiss.eu>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: ~postmarketos/upstreaming@lists.sr.ht,
	phone-devel@vger.kernel.org,
	Jassi Brar <jassisinghbrar@gmail.com>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
	devicetree@vger.kernel.org, linux-clk@vger.kernel.org
Subject: Re: [PATCH 5/7] ARM: dts: qcom: msm8226: Add CPU frequency scaling support
Date: Thu, 20 Jun 2024 23:27:19 +0200	[thread overview]
Message-ID: <3116757.mvXUDI8C0e@g550jk> (raw)
In-Reply-To: <2dr3dyqu3epnrdpot3iwatkwq4uxgcmkoh7mzru2pet6xllarr@izq6mmyquffk>

On Donnerstag, 20. Juni 2024 22:54:37 MESZ Dmitry Baryshkov wrote:
> On Wed, Jun 19, 2024 at 11:02:49PM GMT, Luca Weiss wrote:
> > Add a node for the a7pll with its frequencies. With this we can use the
> > apcs-kpss-global driver for the apcs node and use the apcs to scale the
> > CPU frequency according to the opp-table.
> > 
> > At the same time unfortunately we need to provide the gcc node xo_board
> > instead of the XO via rpmcc since otherwise we'll have a circular
> > dependency between apcs, gcc and the rpm.
> 
> But it should be fine, isn't it? Clock controllers can handle orphaned
> clocks.
> 
> The xo_board is really a hack and should eventually be removed.

I can check again what happened but pretty sure there were some issues with
this still being rpmcc.

But there were also some clock issues with apcs-as-syscon usage (that's
the main reason for my influx of patches regarding this topic), so maybe
with the apcs one solved that one's also fine.

I'll check again!

> 
> > 
> > Signed-off-by: Luca Weiss <luca@lucaweiss.eu>
> > ---
> 
> 





  reply	other threads:[~2024-06-20 21:27 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-06-19 21:02 [PATCH 0/7] Add CPU frequency scaling support for MSM8226 Luca Weiss
2024-06-19 21:02 ` [PATCH 1/7] dt-bindings: mailbox: qcom: add compatible for MSM8226 SoC Luca Weiss
2024-06-20  6:50   ` Krzysztof Kozlowski
2025-03-17 17:04   ` Luca Weiss
2024-06-19 21:02 ` [PATCH 2/7] dt-bindings: clock: qcom,a53pll: Allow opp-table subnode Luca Weiss
2024-06-20  6:50   ` Krzysztof Kozlowski
2024-06-19 21:02 ` [PATCH 3/7] dt-bindings: clock: qcom,a53pll: Add msm8226-a7pll compatible Luca Weiss
2024-06-20  6:50   ` Krzysztof Kozlowski
2024-06-19 21:02 ` [PATCH 4/7] clk: qcom: a53-pll: Add MSM8226 a7pll support Luca Weiss
2024-06-19 21:02 ` [PATCH 5/7] ARM: dts: qcom: msm8226: Add CPU frequency scaling support Luca Weiss
2024-06-20 20:54   ` Dmitry Baryshkov
2024-06-20 21:27     ` Luca Weiss [this message]
2024-06-22 11:46   ` Konrad Dybcio
2024-06-19 21:02 ` [PATCH 6/7] ARM: dts: qcom: msm8226: Hook up CPU cooling Luca Weiss
2024-06-20 21:25   ` Dmitry Baryshkov
2024-06-22 11:47   ` Konrad Dybcio
2024-06-19 21:02 ` [PATCH 7/7] ARM: dts: qcom: msm8226: Convert APCS usages to mbox interface Luca Weiss
2024-06-20 21:25   ` Dmitry Baryshkov
2024-06-22 11:48   ` Konrad Dybcio
2024-08-15 20:40 ` (subset) [PATCH 0/7] Add CPU frequency scaling support for MSM8226 Bjorn Andersson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3116757.mvXUDI8C0e@g550jk \
    --to=luca@lucaweiss.eu \
    --cc=andersson@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.baryshkov@linaro.org \
    --cc=jassisinghbrar@gmail.com \
    --cc=konrad.dybcio@linaro.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=phone-devel@vger.kernel.org \
    --cc=robh@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=~postmarketos/upstreaming@lists.sr.ht \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.