diff mbox series

[fix,for,v5.10] ARM: dts: exynos: revert "add input clock to CMU in Exynos4412 Odroid"

Message ID 20201105214702.2738-1-krzk@kernel.org (mailing list archive)
State Accepted
Commit 53bf2776e31376f0b6a1fd7c9e1abc61241825a2
Headers show
Series [fix,for,v5.10] ARM: dts: exynos: revert "add input clock to CMU in Exynos4412 Odroid" | expand

Commit Message

Krzysztof Kozlowski Nov. 5, 2020, 9:47 p.m. UTC
This reverts commit eaf2d2f6895d676dda6c95a652b58594f2887720.

The commit eaf2d2f6895d ("ARM: dts: exynos: add input clock to CMU in
Exynos4412 Odroid") breaks probing of usb3503 USB hub on Odroid U3.

It changes the order of clock drivers probe: the clkout (Exynos PMU)
driver is probed before the main clk-exynos4 driver.  The clkout driver
on Exynos4412 depends on clk-exynos4 but it does not support deferred
probe, therefore this dependency and changed probe order causes probe
failure.

The usb3503 USB hub on Odroid U3 on the other hand requires clkout
clock.  This can be seen in logs:

    [    5.007442] usb3503 0-0008: unable to request refclk (-517)

Reported-by: Marek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Link: https://lore.kernel.org/r/20200921174818.15525-1-krzk@kernel.org
---
 arch/arm/boot/dts/exynos4412-odroid-common.dtsi | 1 -
 1 file changed, 1 deletion(-)

Comments

patchwork-bot+linux-soc@kernel.org Nov. 13, 2020, 1:11 p.m. UTC | #1
Hello:

This patch was applied to soc/soc.git (refs/heads/arm/fixes):

On Thu,  5 Nov 2020 22:47:02 +0100 you wrote:
> This reverts commit eaf2d2f6895d676dda6c95a652b58594f2887720.
> 
> The commit eaf2d2f6895d ("ARM: dts: exynos: add input clock to CMU in
> Exynos4412 Odroid") breaks probing of usb3503 USB hub on Odroid U3.
> 
> It changes the order of clock drivers probe: the clkout (Exynos PMU)
> driver is probed before the main clk-exynos4 driver.  The clkout driver
> on Exynos4412 depends on clk-exynos4 but it does not support deferred
> probe, therefore this dependency and changed probe order causes probe
> failure.
> 
> [...]

Here is the summary with links:
  - [fix,for,v5.10] ARM: dts: exynos: revert "add input clock to CMU in Exynos4412 Odroid"
    https://git.kernel.org/soc/soc/c/53bf2776e313

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/exynos4412-odroid-common.dtsi b/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
index ab291cec650a..2983e91bc7dd 100644
--- a/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
+++ b/arch/arm/boot/dts/exynos4412-odroid-common.dtsi
@@ -122,7 +122,6 @@  &camera {
 };
 
 &clock {
-	clocks = <&clock CLK_XUSBXTI>;
 	assigned-clocks = <&clock CLK_FOUT_EPLL>;
 	assigned-clock-rates = <45158401>;
 };