mbox series

[0/2] arm64: dts: qcom: sc8280xp: fix UFS reference clocks and PHY nodes

Message ID 20221104092045.17410-1-johan+linaro@kernel.org
Headers show
Series arm64: dts: qcom: sc8280xp: fix UFS reference clocks and PHY nodes | expand

Message

Johan Hovold Nov. 4, 2022, 9:20 a.m. UTC 
After some initial confusion, we've finally settled how the UFS ref
clocks are used.

The first patch fixes the UFS controller and PHY nodes so that they
reflect the hardware. This one should go in 6.1-rc where the two
previous attempts to address this are heading.

Note this patch depends on first updating the clock driver to reflect
that the device ref clock is fed from CXO. This is needed as the UFS
controller driver requires a valid frequency for the ref clock. A patch
that addresses the this has been submitted, but a v2 is in the works.
[1]

The second patch updates the UFS PHY nodes so that they reflect the
fixed UFS PHY binding that is now in linux-next. This one is 6.2
material but depends on first fixing the clocks.

So to summarise the dependency order:

 1. clock driver should be fixed in 6.1-rc
 2. patch 1/2 should go in 6.1-rc (after 1)
 3. patch 2/2 should go in 6.2 (after 1 and 2)

Johan


[1] https://lore.kernel.org/r/20221030142333.31019-1-quic_shazhuss@quicinc.com


Johan Hovold (2):
  arm64: dts: qcom: sc8280xp: fix UFS reference clocks
  arm64: dts: qcom: sc8280xp: update UFS PHY nodes

 arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 57 ++++++++++----------------
 1 file changed, 21 insertions(+), 36 deletions(-)

Comments

Bjorn Andersson Dec. 2, 2022, 8:58 p.m. UTC | #1 
On Fri, 4 Nov 2022 10:20:43 +0100, Johan Hovold wrote:
> After some initial confusion, we've finally settled how the UFS ref
> clocks are used.
> 
> The first patch fixes the UFS controller and PHY nodes so that they
> reflect the hardware. This one should go in 6.1-rc where the two
> previous attempts to address this are heading.
> 
> [...]

Applied, thanks!

[1/2] arm64: dts: qcom: sc8280xp: fix UFS reference clocks
      commit: f446022b932aff1d6a308ca5d537ec2b512debdc

Best regards,
Bjorn Andersson Dec. 6, 2022, 6:18 p.m. UTC | #2 
On Fri, 4 Nov 2022 10:20:43 +0100, Johan Hovold wrote:
> After some initial confusion, we've finally settled how the UFS ref
> clocks are used.
> 
> The first patch fixes the UFS controller and PHY nodes so that they
> reflect the hardware. This one should go in 6.1-rc where the two
> previous attempts to address this are heading.
> 
> [...]

Applied, thanks!

[2/2] arm64: dts: qcom: sc8280xp: update UFS PHY nodes
      commit: 33c4e6588e4f018abc43381ee21fe2bed37e34a5

Best regards,