[PATCH v3 00/47] clk: qcom: Tie CX power domains to GCCs

Abel Vesa posted 47 patches 1 month ago
There is a newer version of this series
Documentation/devicetree/bindings/clock/qcom,gcc-sdm845.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,gcc-sm8250.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,gcc-sm8350.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,gcc-sm8450.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,qcs615-gcc.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,sm8550-gcc.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,sm8650-gcc.yaml | 8 ++++++++
Documentation/devicetree/bindings/clock/qcom,sm8750-gcc.yaml | 8 ++++++++
arch/arm64/boot/dts/qcom/kaanapali.dtsi                      | 2 ++
arch/arm64/boot/dts/qcom/monaco.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/qdu1000.dtsi                        | 1 +
arch/arm64/boot/dts/qcom/sar2130p.dtsi                       | 1 +
arch/arm64/boot/dts/qcom/sdm670.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sdx75.dtsi                          | 1 +
arch/arm64/boot/dts/qcom/sm4450.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sm6350.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sm8150.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sm8250.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sm8350.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sm8450.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sm8550.dtsi                         | 1 +
arch/arm64/boot/dts/qcom/sm8650.dtsi                         | 2 ++
arch/arm64/boot/dts/qcom/sm8750.dtsi                         | 2 ++
arch/arm64/boot/dts/qcom/talos.dtsi                          | 2 ++
drivers/clk/qcom/gcc-eliza.c                                 | 1 +
drivers/clk/qcom/gcc-kaanapali.c                             | 1 +
drivers/clk/qcom/gcc-qcs615.c                                | 1 +
drivers/clk/qcom/gcc-qcs8300.c                               | 1 +
drivers/clk/qcom/gcc-qdu1000.c                               | 1 +
drivers/clk/qcom/gcc-sa8775p.c                               | 1 +
drivers/clk/qcom/gcc-sar2130p.c                              | 1 +
drivers/clk/qcom/gcc-sc7180.c                                | 1 +
drivers/clk/qcom/gcc-sc7280.c                                | 1 +
drivers/clk/qcom/gcc-sdm845.c                                | 2 ++
drivers/clk/qcom/gcc-sdx75.c                                 | 1 +
drivers/clk/qcom/gcc-sm4450.c                                | 1 +
drivers/clk/qcom/gcc-sm6350.c                                | 1 +
drivers/clk/qcom/gcc-sm8150.c                                | 1 +
drivers/clk/qcom/gcc-sm8250.c                                | 1 +
drivers/clk/qcom/gcc-sm8350.c                                | 1 +
drivers/clk/qcom/gcc-sm8450.c                                | 1 +
drivers/clk/qcom/gcc-sm8550.c                                | 1 +
drivers/clk/qcom/gcc-sm8650.c                                | 1 +
drivers/clk/qcom/gcc-sm8750.c                                | 1 +
drivers/clk/qcom/gcc-x1e80100.c                              | 1 +
46 files changed, 114 insertions(+)
[PATCH v3 00/47] clk: qcom: Tie CX power domains to GCCs
Posted by Abel Vesa 1 month ago
Tie the CX power domain to GCC clock controllers for SDM670, SDM845,
SM8150, SM8250, SM8350, SM8450, SM8550, SM8650, SM8750, SM6350,
SM4450, QDU1000, QCS8300, SDX75, SAR2130P, SC7180, SC7280, SA8775P,
X1E80100, Eliza, Kaanapali, and QCS615.

Also enable the runtime PM in each of the GCC provider drivers.

Update the GCC bindings to document and require the CX power-domain
dependency for the affected compatibles.

Only Kaanapali and Eliza require Fixes tags, as runtime PM support was not
available when the other platforms were upstreamed.

Signed-off-by: Abel Vesa <abel.vesa@oss.qualcomm.com>
---
- Rebased on next-20260821.
- Pick up Dmitry's and Konrad's R-b tags.
- Add binding updates that document and require the CX power domain.
- Add SDM670, SDM845, SM8150, SM6350, SM4450, QDU1000, QCS8300,
  SDX75, SAR2130P, SC7180, SC7280, SA8775P, X1E80100, and Eliza to the
  list.
- Link to v2: https://patch.msgid.link/20260820-qcom-tie-cx-to-gcc-v2-0-e67ea30d8b7b@oss.qualcomm.com

Changes in v2:
- Clarify the runtime PM rationale in the GCC driver commit messages.
- Picked up Konrad's R-b tag for the dts patches.
- Link to v1: https://patch.msgid.link/20260819-qcom-tie-cx-to-gcc-v1-0-840034124e73@oss.qualcomm.com

To: Bjorn Andersson <andersson@kernel.org>
To: Stephen Boyd <sboyd@kernel.org>
To: Brian Masney <bmasney+clk@redhat.com>
To: Jerome Brunet <jbrunet+clk@baylibre.com>
To: Rob Herring <robh@kernel.org>
To: Krzysztof Kozlowski <krzk+dt@kernel.org>
To: Conor Dooley <conor+dt@kernel.org>
To: Taniya Das <taniya.das@oss.qualcomm.com>
To: Vinod Koul <vkoul@kernel.org>
To: Jingyi Wang <jingyi.wang@oss.qualcomm.com>
To: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>
To: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
To: Abel Vesa <abel.vesa@oss.qualcomm.com>
To: Konrad Dybcio <konradybcio@kernel.org>
To: Tengfei Fan <tengfei.fan@oss.qualcomm.com>
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-clk@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Krzysztof Kozlowski <krzk@kernel.org>

---
Abel Vesa (47):
      dt-bindings: clock: qcom: gcc-sdm845: Add CX power domain
      dt-bindings: clock: qcom: gcc-sm8150: Add CX power domain
      dt-bindings: clock: qcom: gcc-sm8250: Add CX power domain
      dt-bindings: clock: qcom: gcc-sm8350: Add CX power domain
      dt-bindings: clock: qcom: gcc-sm8450: Add CX power domain
      dt-bindings: clock: qcom: qcs615-gcc: Add CX power domain
      dt-bindings: clock: qcom: sm8550-gcc: Add CX power domain
      dt-bindings: clock: qcom: sm8650-gcc: Add CX power domain
      dt-bindings: clock: qcom: sm8750-gcc: Add CX power domain
      clk: qcom: gcc-eliza: Tie the CX power domain to controller
      clk: qcom: gcc-kaanapali: Tie the CX power domain to controller
      clk: qcom: gcc-qcs615: Tie the CX power domain to controller
      clk: qcom: gcc-qcs8300: Tie the CX power domain to controller
      clk: qcom: gcc-qdu1000: Tie the CX power domain to controller
      clk: qcom: gcc-sa8775p: Tie the CX power domain to controller
      clk: qcom: gcc-sar2130p: Tie the CX power domain to controller
      clk: qcom: gcc-sc7180: Tie the CX power domain to controller
      clk: qcom: gcc-sc7280: Tie the CX power domain to controller
      clk: qcom: gcc-sdm670: Tie the CX power domain to controller
      clk: qcom: gcc-sdm845: Tie the CX power domain to controller
      clk: qcom: gcc-sdx75: Tie the CX power domain to controller
      clk: qcom: gcc-sm4450: Tie the CX power domain to controller
      clk: qcom: gcc-sm6350: Tie the CX power domain to controller
      clk: qcom: gcc-sm8150: Tie the CX power domain to controller
      clk: qcom: gcc-sm8250: Tie the CX power domain to controller
      clk: qcom: gcc-sm8350: Tie the CX power domain to controller
      clk: qcom: gcc-sm8450: Tie the CX power domain to controller
      clk: qcom: gcc-sm8550: Tie the CX power domain to controller
      clk: qcom: gcc-sm8650: Tie the CX power domain to controller
      clk: qcom: gcc-sm8750: Tie the CX power domain to controller
      clk: qcom: gcc-x1e80100: Tie the CX power domain to controller
      arm64: dts: qcom: kaanapali: Add GCC CX power domain
      arm64: dts: qcom: monaco: Add GCC CX power domain
      arm64: dts: qcom: qdu1000: Add GCC CX power domain
      arm64: dts: qcom: sar2130p: Add GCC CX power domain
      arm64: dts: qcom: sdm670: Add GCC CX power domain
      arm64: dts: qcom: sdx75: Add GCC CX power domain
      arm64: dts: qcom: sm4450: Add GCC CX power domain
      arm64: dts: qcom: sm6350: Add GCC CX power domain
      arm64: dts: qcom: sm8150: Add GCC CX power domain
      arm64: dts: qcom: sm8250: Add GCC CX power domain
      arm64: dts: qcom: sm8350: Add GCC CX power domain
      arm64: dts: qcom: sm8450: Add GCC CX power domain
      arm64: dts: qcom: sm8550: Add GCC CX power domain
      arm64: dts: qcom: sm8650: Add GCC CX power domain
      arm64: dts: qcom: sm8750: Add GCC CX power domain
      arm64: dts: qcom: talos: Add GCC CX power domain

 Documentation/devicetree/bindings/clock/qcom,gcc-sdm845.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8150.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8250.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8350.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,gcc-sm8450.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,qcs615-gcc.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,sm8550-gcc.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,sm8650-gcc.yaml | 8 ++++++++
 Documentation/devicetree/bindings/clock/qcom,sm8750-gcc.yaml | 8 ++++++++
 arch/arm64/boot/dts/qcom/kaanapali.dtsi                      | 2 ++
 arch/arm64/boot/dts/qcom/monaco.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/qdu1000.dtsi                        | 1 +
 arch/arm64/boot/dts/qcom/sar2130p.dtsi                       | 1 +
 arch/arm64/boot/dts/qcom/sdm670.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sdx75.dtsi                          | 1 +
 arch/arm64/boot/dts/qcom/sm4450.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sm6350.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sm8150.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sm8250.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sm8350.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sm8450.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sm8550.dtsi                         | 1 +
 arch/arm64/boot/dts/qcom/sm8650.dtsi                         | 2 ++
 arch/arm64/boot/dts/qcom/sm8750.dtsi                         | 2 ++
 arch/arm64/boot/dts/qcom/talos.dtsi                          | 2 ++
 drivers/clk/qcom/gcc-eliza.c                                 | 1 +
 drivers/clk/qcom/gcc-kaanapali.c                             | 1 +
 drivers/clk/qcom/gcc-qcs615.c                                | 1 +
 drivers/clk/qcom/gcc-qcs8300.c                               | 1 +
 drivers/clk/qcom/gcc-qdu1000.c                               | 1 +
 drivers/clk/qcom/gcc-sa8775p.c                               | 1 +
 drivers/clk/qcom/gcc-sar2130p.c                              | 1 +
 drivers/clk/qcom/gcc-sc7180.c                                | 1 +
 drivers/clk/qcom/gcc-sc7280.c                                | 1 +
 drivers/clk/qcom/gcc-sdm845.c                                | 2 ++
 drivers/clk/qcom/gcc-sdx75.c                                 | 1 +
 drivers/clk/qcom/gcc-sm4450.c                                | 1 +
 drivers/clk/qcom/gcc-sm6350.c                                | 1 +
 drivers/clk/qcom/gcc-sm8150.c                                | 1 +
 drivers/clk/qcom/gcc-sm8250.c                                | 1 +
 drivers/clk/qcom/gcc-sm8350.c                                | 1 +
 drivers/clk/qcom/gcc-sm8450.c                                | 1 +
 drivers/clk/qcom/gcc-sm8550.c                                | 1 +
 drivers/clk/qcom/gcc-sm8650.c                                | 1 +
 drivers/clk/qcom/gcc-sm8750.c                                | 1 +
 drivers/clk/qcom/gcc-x1e80100.c                              | 1 +
 46 files changed, 114 insertions(+)
---
base-commit: 903c1cf6dff9964e71eda98a39e2e5d442050472
change-id: 20260819-qcom-tie-cx-to-gcc-67e6697639b5

Best regards,
--  
Abel Vesa <abel.vesa@oss.qualcomm.com>
Re: [PATCH v3 00/47] clk: qcom: Tie CX power domains to GCCs
Posted by Vinod Koul 1 month ago
On 24-08-26, 14:53, Abel Vesa wrote:
> Tie the CX power domain to GCC clock controllers for SDM670, SDM845,
> SM8150, SM8250, SM8350, SM8450, SM8550, SM8650, SM8750, SM6350,
> SM4450, QDU1000, QCS8300, SDX75, SAR2130P, SC7180, SC7280, SA8775P,
> X1E80100, Eliza, Kaanapali, and QCS615.

> 
> ---
> Abel Vesa (47):
>       dt-bindings: clock: qcom: gcc-sdm845: Add CX power domain
...
>       arm64: dts: qcom: kaanapali: Add GCC CX power domain
>       arm64: dts: qcom: monaco: Add GCC CX power domain
>       arm64: dts: qcom: qdu1000: Add GCC CX power domain
>       arm64: dts: qcom: sar2130p: Add GCC CX power domain
>       arm64: dts: qcom: sdm670: Add GCC CX power domain
>       arm64: dts: qcom: sdx75: Add GCC CX power domain
>       arm64: dts: qcom: sm4450: Add GCC CX power domain
>       arm64: dts: qcom: sm6350: Add GCC CX power domain
>       arm64: dts: qcom: sm8150: Add GCC CX power domain
>       arm64: dts: qcom: sm8250: Add GCC CX power domain
>       arm64: dts: qcom: sm8350: Add GCC CX power domain
>       arm64: dts: qcom: sm8450: Add GCC CX power domain
>       arm64: dts: qcom: sm8550: Add GCC CX power domain
>       arm64: dts: qcom: sm8650: Add GCC CX power domain
>       arm64: dts: qcom: sm8750: Add GCC CX power domain
>       arm64: dts: qcom: talos: Add GCC CX power domain

Don't see 845 dts updated?

-- 
~Vinod
Re: [PATCH v3 00/47] clk: qcom: Tie CX power domains to GCCs
Posted by Abel Vesa 1 month ago
On 26-08-24 21:58:55, Vinod Koul wrote:
> On 24-08-26, 14:53, Abel Vesa wrote:
> > Tie the CX power domain to GCC clock controllers for SDM670, SDM845,
> > SM8150, SM8250, SM8350, SM8450, SM8550, SM8650, SM8750, SM6350,
> > SM4450, QDU1000, QCS8300, SDX75, SAR2130P, SC7180, SC7280, SA8775P,
> > X1E80100, Eliza, Kaanapali, and QCS615.
> 
> > 
> > ---
> > Abel Vesa (47):
> >       dt-bindings: clock: qcom: gcc-sdm845: Add CX power domain
> ...
> >       arm64: dts: qcom: kaanapali: Add GCC CX power domain
> >       arm64: dts: qcom: monaco: Add GCC CX power domain
> >       arm64: dts: qcom: qdu1000: Add GCC CX power domain
> >       arm64: dts: qcom: sar2130p: Add GCC CX power domain
> >       arm64: dts: qcom: sdm670: Add GCC CX power domain
> >       arm64: dts: qcom: sdx75: Add GCC CX power domain
> >       arm64: dts: qcom: sm4450: Add GCC CX power domain
> >       arm64: dts: qcom: sm6350: Add GCC CX power domain
> >       arm64: dts: qcom: sm8150: Add GCC CX power domain
> >       arm64: dts: qcom: sm8250: Add GCC CX power domain
> >       arm64: dts: qcom: sm8350: Add GCC CX power domain
> >       arm64: dts: qcom: sm8450: Add GCC CX power domain
> >       arm64: dts: qcom: sm8550: Add GCC CX power domain
> >       arm64: dts: qcom: sm8650: Add GCC CX power domain
> >       arm64: dts: qcom: sm8750: Add GCC CX power domain
> >       arm64: dts: qcom: talos: Add GCC CX power domain
> 
> Don't see 845 dts updated?

The sdm845 already has the CX attached to GCC, it's just the driver
that needs to change, to enable the runtime PM.

I'll fix the cover letter as well to be consistent.

Thanks for reviewing.

Abel