From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Andy Gross <agross@kernel.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Rob Herring <robh+dt@kernel.org>, Stephen Boyd <sboyd@kernel.org>,
Taniya Das <tdas@codeaurora.org>,
Jonathan Marek <jonathan@marek.ca>,
Michael Turquette <mturquette@baylibre.com>
Cc: linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org,
linux-clk@vger.kernel.org,
Bryan O'Donoghue <bryan.odonoghue@linaro.org>,
Mark Brown <broonie@kernel.org>,
Ulf Hansson <ulf.hansson@linaro.org>,
linux-kernel@vger.kernel.org
Subject: [PATCH v6 0/6] clk: qcom: use power-domain for sm8250's clock controllers
Date: Tue, 27 Jul 2021 23:19:56 +0300 [thread overview]
Message-ID: <20210727202004.712665-1-dmitry.baryshkov@linaro.org> (raw)
On SM8250 both the display and video clock controllers are powered up by
the MMCX power domain. Handle this by linking clock controllers to the
proper power domain, and using runtime power management to enable and
disable the MMCX power domain.
Dependencies:
- https://lore.kernel.org/linux-arm-msm/20210703005416.2668319-1-bjorn.andersson@linaro.org/
(pending)
Changes since v5:
- Dropped devm_pm_runtime_enable callback to remove extra dependency
Changes since v4:
- Dropped pm_runtime handling from drivers/clk/qcom/common.c Moved the
code into dispcc-sm8250.c and videocc-sm8250.c
Changes since v3:
- Wrap gdsc_enable/gdsc_disable into pm_runtime_get/put calls rather
than calling pm_runtime_get in gdsc_enabled and _put in gdsc_disable
- Squash gdsc patches together to remove possible dependencies between
two patches.
Changes since v2:
- Move pm_runtime calls from generic genpd code to the gdsc code for
now (as suggested by Ulf & Bjorn)
Changes since v1:
- Rebase on top of Bjorn's patches, removing the need for setting
performance state directly.
- Move runtime PM calls from GDSC code to generic genpd code.
- Always call pm_runtime_enable in the Qualcomm generic clock
controller code.
- Register GDSC power domains as subdomains of the domain powering the
clock controller if there is one.
----------------------------------------------------------------
Dmitry Baryshkov (8):
dt-bindings: clock: qcom,dispcc-sm8x50: add mmcx power domain
dt-bindings: clock: qcom,videocc: add mmcx power domain
clk: qcom: dispcc-sm8250: use runtime PM for the clock controller
clk: qcom: videocc-sm8250: use runtime PM for the clock controller
clk: qcom: gdsc: enable optional power domain support
arm64: dts: qcom: sm8250: remove mmcx regulator
clk: qcom: dispcc-sm8250: stop using mmcx regulator
clk: qcom: videocc-sm8250: stop using mmcx regulator
.../bindings/clock/qcom,dispcc-sm8x50.yaml | 7 +++
.../devicetree/bindings/clock/qcom,videocc.yaml | 7 +++
arch/arm64/boot/dts/qcom/sm8250.dtsi | 11 +---
drivers/clk/qcom/dispcc-sm8250.c | 28 ++++++++--
drivers/clk/qcom/gdsc.c | 59 ++++++++++++++++++++--
drivers/clk/qcom/gdsc.h | 2 +
drivers/clk/qcom/videocc-sm8250.c | 31 +++++++++---
7 files changed, 124 insertions(+), 21 deletions(-)
next reply other threads:[~2021-07-27 20:20 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-27 20:19 Dmitry Baryshkov [this message]
2021-07-27 20:19 ` [PATCH v6 1/8] dt-bindings: clock: qcom,dispcc-sm8x50: add mmcx power domain Dmitry Baryshkov
2021-07-27 20:19 ` [PATCH v6 2/8] dt-bindings: clock: qcom,videocc: " Dmitry Baryshkov
2021-07-27 20:19 ` [PATCH v6 3/8] clk: qcom: dispcc-sm8250: use runtime PM for the clock controller Dmitry Baryshkov
2021-07-27 20:20 ` [PATCH v6 4/8] clk: qcom: videocc-sm8250: " Dmitry Baryshkov
2021-07-27 20:20 ` [PATCH v6 5/8] clk: qcom: gdsc: enable optional power domain support Dmitry Baryshkov
2021-07-27 20:20 ` [PATCH v6 6/8] arm64: dts: qcom: sm8250: remove mmcx regulator Dmitry Baryshkov
2021-07-27 20:20 ` [PATCH v6 7/8] clk: qcom: dispcc-sm8250: stop using " Dmitry Baryshkov
2021-07-27 20:20 ` [PATCH v6 8/8] clk: qcom: videocc-sm8250: " Dmitry Baryshkov
2021-08-10 11:11 ` [PATCH v6 0/6] clk: qcom: use power-domain for sm8250's clock controllers Ulf Hansson
2021-08-26 18:31 ` Stephen Boyd
2021-08-26 21:56 ` Dmitry Baryshkov
2021-08-29 3:51 ` Stephen Boyd
2021-08-29 15:54 ` Dmitry Baryshkov
2021-09-07 14:34 ` Ulf Hansson
2021-09-08 8:50 ` Dmitry Baryshkov
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=20210727202004.712665-1-dmitry.baryshkov@linaro.org \
--to=dmitry.baryshkov@linaro.org \
--cc=agross@kernel.org \
--cc=bjorn.andersson@linaro.org \
--cc=broonie@kernel.org \
--cc=bryan.odonoghue@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=jonathan@marek.ca \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mturquette@baylibre.com \
--cc=robh+dt@kernel.org \
--cc=sboyd@kernel.org \
--cc=tdas@codeaurora.org \
--cc=ulf.hansson@linaro.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 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.