From: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
To: Abel Vesa <abel.vesa@oss.qualcomm.com>,
Bjorn Andersson <andersson@kernel.org>,
Abel Vesa <abelvesa@kernel.org>, Stephen Boyd <sboyd@kernel.org>,
Brian Masney <bmasney+clk@redhat.com>,
Jerome Brunet <jbrunet+clk@baylibre.com>
Cc: linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 00/48] clk: qcom: Use common probe data for always-on clocks
Date: Fri, 18 Sep 2026 11:41:35 +0200 [thread overview]
Message-ID: <32f226a1-c0a7-4b8c-b938-7089eab2870d@oss.qualcomm.com> (raw)
In-Reply-To: <20260916-clk-qcom-rework-legacy-always-on-clocks-v1-0-25a24e50f82a@oss.qualcomm.com>
On 9/16/26 12:28 PM, Abel Vesa wrote:
> Many Qualcomm clock controller drivers still enable always-on branches
> with individual qcom_branch_set_clk_en() calls in their probe functions.
> The common probe code already supports describing these register offsets
> through the clk_cbcrs and num_clk_cbcrs fields of qcom_cc_driver_data.
>
> Convert the remaining users of these direct calls to per-controller
> critical_cbcrs arrays. This centralizes branch enable handling in the
> common probe path and makes the always-on register lists easier to review
> alongside the other controller data.
>
> For GCC drivers, also describe DFS RCGs through the common driver data and
> move auxiliary register programming into clk_regs_configure callbacks
> where appropriate. Drivers whose initialization is fully covered by the
> common path can then use qcom_cc_probe() directly. Keep device-specific
> setup in the remaining probe functions, including the separate CBCR lists
> needed by the SA8775P/QCS8300 camera and SM8350/SC8280XP video variants.
>
> Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
> ---
I left a couple comments as replies, but let's have them be additive
touch-ups on top of this so as not to resend 48 patches
for this series:
Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Konrad
prev parent reply other threads:[~2026-09-18 9:41 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-16 10:28 Abel Vesa
2026-09-16 10:28 ` [PATCH 01/48] clk: qcom: camcc-sa8775p: Use common probe handling " Abel Vesa
2026-09-16 10:28 ` [PATCH 02/48] clk: qcom: camcc-sc8280xp: " Abel Vesa
2026-09-16 10:28 ` [PATCH 03/48] clk: qcom: camcc-sm7150: " Abel Vesa
2026-09-16 10:28 ` [PATCH 04/48] clk: qcom: camcc-sm8150: " Abel Vesa
2026-09-16 10:28 ` [PATCH 05/48] clk: qcom: dispcc-sc7280: " Abel Vesa
2026-09-16 10:28 ` [PATCH 06/48] clk: qcom: dispcc-sc8280xp: " Abel Vesa
2026-09-16 10:28 ` [PATCH 07/48] clk: qcom: dispcc-sm4450: " Abel Vesa
2026-09-16 10:28 ` [PATCH 08/48] clk: qcom: dispcc-sm6115: " Abel Vesa
2026-09-16 10:28 ` [PATCH 09/48] clk: qcom: dispcc-sm7150: " Abel Vesa
2026-09-16 10:28 ` [PATCH 10/48] clk: qcom: dispcc-sm8250: " Abel Vesa
2026-09-16 10:28 ` [PATCH 11/48] clk: qcom: dispcc-sm8550: " Abel Vesa
2026-09-16 10:28 ` [PATCH 12/48] clk: qcom: dispcc-sm8750: " Abel Vesa
2026-09-16 10:28 ` [PATCH 13/48] clk: qcom: dispcc-x1e80100: " Abel Vesa
2026-09-16 10:28 ` [PATCH 14/48] clk: qcom: dispcc0-sa8775p: " Abel Vesa
2026-09-16 10:28 ` [PATCH 15/48] clk: qcom: dispcc1-sa8775p: " Abel Vesa
2026-09-16 10:28 ` [PATCH 16/48] clk: qcom: gcc-qcs615: Use common clock controller probe data Abel Vesa
2026-09-16 10:28 ` [PATCH 17/48] clk: qcom: gcc-qcs8300: " Abel Vesa
2026-09-16 10:28 ` [PATCH 18/48] clk: qcom: gcc-sa8775p: " Abel Vesa
2026-09-16 10:28 ` [PATCH 19/48] clk: qcom: gcc-sar2130p: " Abel Vesa
2026-09-16 10:28 ` [PATCH 20/48] clk: qcom: gcc-sc7180: " Abel Vesa
2026-09-16 10:28 ` [PATCH 21/48] clk: qcom: gcc-sc7280: " Abel Vesa
2026-09-16 10:28 ` [PATCH 22/48] clk: qcom: gcc-sc8280xp: " Abel Vesa
2026-09-16 10:28 ` [PATCH 23/48] clk: qcom: gcc-sdx55: " Abel Vesa
2026-09-16 10:28 ` [PATCH 24/48] clk: qcom: gcc-sdx65: " Abel Vesa
2026-09-16 10:28 ` [PATCH 25/48] clk: qcom: gcc-sdx75: " Abel Vesa
2026-09-16 10:28 ` [PATCH 26/48] clk: qcom: gcc-sm4450: " Abel Vesa
2026-09-16 10:28 ` [PATCH 27/48] clk: qcom: gcc-sm6375: " Abel Vesa
2026-09-18 9:38 ` Konrad Dybcio
2026-09-16 10:28 ` [PATCH 28/48] clk: qcom: gcc-sm7150: " Abel Vesa
2026-09-16 10:28 ` [PATCH 29/48] clk: qcom: gcc-sm8250: " Abel Vesa
2026-09-18 9:39 ` Konrad Dybcio
2026-09-16 10:28 ` [PATCH 30/48] clk: qcom: gcc-sm8350: " Abel Vesa
2026-09-16 10:28 ` [PATCH 31/48] clk: qcom: gcc-sm8450: " Abel Vesa
2026-09-16 10:28 ` [PATCH 32/48] clk: qcom: gcc-sm8550: " Abel Vesa
2026-09-16 10:28 ` [PATCH 33/48] clk: qcom: gcc-sm8650: " Abel Vesa
2026-09-16 10:28 ` [PATCH 34/48] clk: qcom: gcc-sm8750: " Abel Vesa
2026-09-16 10:28 ` [PATCH 35/48] clk: qcom: gcc-x1e80100: " Abel Vesa
2026-09-16 10:28 ` [PATCH 36/48] clk: qcom: gpucc-sar2130p: Use common probe handling for always-on clocks Abel Vesa
2026-09-16 10:28 ` [PATCH 37/48] clk: qcom: gpucc-sc7280: " Abel Vesa
2026-09-16 10:28 ` [PATCH 38/48] clk: qcom: gpucc-sc8280xp: " Abel Vesa
2026-09-16 10:28 ` [PATCH 39/48] clk: qcom: gpucc-sm4450: " Abel Vesa
2026-09-16 10:28 ` [PATCH 40/48] clk: qcom: gpucc-sm8550: " Abel Vesa
2026-09-16 10:28 ` [PATCH 41/48] clk: qcom: gpucc-x1e80100: " Abel Vesa
2026-09-16 10:28 ` [PATCH 42/48] clk: qcom: gpucc-x1p42100: " Abel Vesa
2026-09-16 10:28 ` [PATCH 43/48] clk: qcom: lpasscorecc-sc7180: " Abel Vesa
2026-09-16 10:29 ` [PATCH 44/48] clk: qcom: videocc-sa8775p: " Abel Vesa
2026-09-16 10:29 ` [PATCH 45/48] clk: qcom: videocc-sm6350: " Abel Vesa
2026-09-16 10:29 ` [PATCH 46/48] clk: qcom: videocc-sm7150: " Abel Vesa
2026-09-16 10:29 ` [PATCH 47/48] clk: qcom: videocc-sm8250: " Abel Vesa
2026-09-16 10:29 ` [PATCH 48/48] clk: qcom: videocc-sm8350: " Abel Vesa
2026-09-18 9:41 ` Konrad Dybcio [this message]
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=32f226a1-c0a7-4b8c-b938-7089eab2870d@oss.qualcomm.com \
--to=konrad.dybcio@oss.qualcomm.com \
--cc=abel.vesa@oss.qualcomm.com \
--cc=abelvesa@kernel.org \
--cc=andersson@kernel.org \
--cc=bmasney+clk@redhat.com \
--cc=jbrunet+clk@baylibre.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.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
all inboxes | Powered by JetHome®