From: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
To: Sibi Sankar <quic_sibis@quicinc.com>,
andersson@kernel.org, konrad.dybcio@linaro.org,
mturquette@baylibre.com, sboyd@kernel.org, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org
Cc: agross@kernel.org, conor+dt@kernel.org, quic_tdas@quicinc.com,
quic_rjendra@quicinc.com, linux-arm-msm@vger.kernel.org,
linux-clk@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, neil.armstrong@linaro.org,
abel.vesa@linaro.org, quic_tsoni@quicinc.com
Subject: Re: [PATCH V2 2/4] clk: qcom: Add Global Clock controller (GCC) driver for X1E80100
Date: Fri, 17 Nov 2023 20:50:56 +0000 [thread overview]
Message-ID: <ec9d03f7-7158-4309-9a04-b08c69b89f39@linaro.org> (raw)
In-Reply-To: <20231117092737.28362-3-quic_sibis@quicinc.com>
On 17/11/2023 09:27, Sibi Sankar wrote:
> * Use shared ops in the x1e80100 gcc driver [Bryan].
This looks better to me now / more consistent with what we have in
sc8280xp - where we do try to hit suspend and => retention/parking matters.
Could you give a bit more detail on why SDCC* doesn't warrant parking on
X1E80100 as on SC8280XP though ?
---
bod
next prev parent reply other threads:[~2023-11-17 20:51 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-11-17 9:27 [PATCH V2 0/4] clk: qcom: Introduce clocks drivers for X1E80100 Sibi Sankar
2023-11-17 9:27 ` [PATCH V2 1/4] dt-bindings: clock: qcom: Add X1E80100 GCC clocks Sibi Sankar
2023-11-17 9:27 ` [PATCH V2 2/4] clk: qcom: Add Global Clock controller (GCC) driver for X1E80100 Sibi Sankar
2023-11-17 20:50 ` Bryan O'Donoghue [this message]
2023-11-18 0:06 ` Konrad Dybcio
2023-11-18 1:52 ` Bryan O'Donoghue
2023-11-20 6:42 ` Sibi Sankar
2023-11-20 11:02 ` Bryan O'Donoghue
2023-11-18 0:01 ` Konrad Dybcio
2023-11-20 11:03 ` Bryan O'Donoghue
2023-11-17 9:27 ` [PATCH V2 3/4] dt-bindings: clock: qcom-rpmhcc: Add RPMHCC bindings " Sibi Sankar
2023-11-17 9:27 ` [PATCH V2 4/4] clk: qcom: rpmh: Add support for X1E80100 rpmh clocks Sibi Sankar
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=ec9d03f7-7158-4309-9a04-b08c69b89f39@linaro.org \
--to=bryan.odonoghue@linaro.org \
--cc=abel.vesa@linaro.org \
--cc=agross@kernel.org \
--cc=andersson@kernel.org \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.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=neil.armstrong@linaro.org \
--cc=quic_rjendra@quicinc.com \
--cc=quic_sibis@quicinc.com \
--cc=quic_tdas@quicinc.com \
--cc=quic_tsoni@quicinc.com \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).