mbox series

[0/5] clk: qcom: gpucc-sc8280xp: fix GX external supply lookup

Message ID 20240326140108.21307-1-johan+linaro@kernel.org (mailing list archive)
Headers show
Series clk: qcom: gpucc-sc8280xp: fix GX external supply lookup | expand

Message

Johan Hovold March 26, 2024, 2:01 p.m. UTC 
The SA8540P platform is closely related to SC8280XP but differs in that
it uses an external supply for the GX power domain.
    
This series adds a new SA8540P GPU clock controller compatible which
can be used to determine whether to look up the external supply.
    
This specifically avoids warnings such as:
    
	gpu_cc-sc8280xp 3d90000.clock-controller: supply vdd-gfx not found, using dummy regulator
    
on SC8280XP, which were introduced in 6.9-rc1.

Note that this also avoids triggering a potential deadlock on SC8280XP
even if the underlying issue still remains for the derivative platforms
like SA8540P and SA8295P that actually use the supply. [1]

Also note that this is a better alternative to simply making the
external supply optional as that would suppress any warnings about
missing supplies on platforms that actually require it. This series
therefore supersedes [2].

Johan


[1] https://lore.kernel.org/lkml/Zf25Sv2x9WaCFuIH@hovoldconsulting.com/
[2] https://lore.kernel.org/lkml/20240325081957.10946-1-johan+linaro@kernel.org/


Johan Hovold (5):
  dt-bindings: clock: qcom: add SA8540P gpucc
  arm64: dts: qcom: sa8540p: use sa8540p gpucc compatible
  clk: qcom: gpucc-sc8280xp: make cc descriptor const
  clk: qcom: gpucc-sc8280xp: fix GX external supply lookup
  arm64: dts: qcom: sa8540p: drop fallback gpucc compatible

 .../devicetree/bindings/clock/qcom,gpucc.yaml |  1 +
 arch/arm64/boot/dts/qcom/sa8540p.dtsi         |  2 +
 drivers/clk/qcom/gpucc-sc8280xp.c             | 42 ++++++++++++++++---
 3 files changed, 39 insertions(+), 6 deletions(-)