linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [v4.13-rc1 fixes] ARM: dts: exynos: Add clocks to audss block to fix silent hang on Exynos4412
@ 2017-07-17 19:45 Krzysztof Kozlowski
  2017-07-27 10:54 ` Arnd Bergmann
  0 siblings, 1 reply; 2+ messages in thread
From: Krzysztof Kozlowski @ 2017-07-17 19:45 UTC (permalink / raw)
  To: Arnd Bergmann, Olof Johansson, arm, linux-samsung-soc
  Cc: linux-arm-kernel, linux-kernel, Krzysztof Kozlowski

Add necessary parent clocks for audss (Audio SubSystem, MAUDIO) clock
controller block.

This allows driver to keep EPLL enabled before accessing any MAUDIO
registers thus fixing silent hang.  This silent hang appeared with
commit 6edfa11cb396 ("clk: samsung: Add enable/disable operation for
PLL36XX clocks"), e.g. on Odroid U3 usually with last (but unrelated)
messages:

	[    2.382741] input: gpio_keys as /devices/platform/gpio_keys/input/input0
	[    2.405686] usb 1-3: new high-speed USB device number 3 using exynos-ehci
	[    2.419843] max77686-rtc max77686-rtc: setting system clock to 2017-06-21 17:04:13 UTC (1498064653)

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---

Hi Arnd, Olof,

I have only one patch in fixes queue for v4.13, so how about applying this
directly without a pull request?


Best regards,
Krzysztof

---
 arch/arm/boot/dts/exynos4.dtsi | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
index 497a9470c888..5739389f5bb8 100644
--- a/arch/arm/boot/dts/exynos4.dtsi
+++ b/arch/arm/boot/dts/exynos4.dtsi
@@ -59,6 +59,9 @@
 		compatible = "samsung,exynos4210-audss-clock";
 		reg = <0x03810000 0x0C>;
 		#clock-cells = <1>;
+		clocks = <&clock CLK_FIN_PLL>, <&clock CLK_FOUT_EPLL>,
+			 <&clock CLK_SCLK_AUDIO0>, <&clock CLK_SCLK_AUDIO0>;
+		clock-names = "pll_ref", "pll_in", "sclk_audio", "sclk_pcm_in";
 	};
 
 	i2s0: i2s@03830000 {
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [v4.13-rc1 fixes] ARM: dts: exynos: Add clocks to audss block to fix silent hang on Exynos4412
  2017-07-17 19:45 [v4.13-rc1 fixes] ARM: dts: exynos: Add clocks to audss block to fix silent hang on Exynos4412 Krzysztof Kozlowski
@ 2017-07-27 10:54 ` Arnd Bergmann
  0 siblings, 0 replies; 2+ messages in thread
From: Arnd Bergmann @ 2017-07-27 10:54 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Olof Johansson, arm-soc,
	moderated list:ARM/SAMSUNG EXYNOS ARM ARCHITECTURES, Linux ARM,
	Linux Kernel Mailing List

On Mon, Jul 17, 2017 at 9:45 PM, Krzysztof Kozlowski <krzk@kernel.org> wrote:
> Add necessary parent clocks for audss (Audio SubSystem, MAUDIO) clock
> controller block.
>
> This allows driver to keep EPLL enabled before accessing any MAUDIO
> registers thus fixing silent hang.  This silent hang appeared with
> commit 6edfa11cb396 ("clk: samsung: Add enable/disable operation for
> PLL36XX clocks"), e.g. on Odroid U3 usually with last (but unrelated)
> messages:
>
>         [    2.382741] input: gpio_keys as /devices/platform/gpio_keys/input/input0
>         [    2.405686] usb 1-3: new high-speed USB device number 3 using exynos-ehci
>         [    2.419843] max77686-rtc max77686-rtc: setting system clock to 2017-06-21 17:04:13 UTC (1498064653)
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>
> Hi Arnd, Olof,
>
> I have only one patch in fixes queue for v4.13, so how about applying this
> directly without a pull request?

No problem, I was just a little slow here. Applied to fixes now, thanks.

      Arnd

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-07-27 10:54 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-17 19:45 [v4.13-rc1 fixes] ARM: dts: exynos: Add clocks to audss block to fix silent hang on Exynos4412 Krzysztof Kozlowski
2017-07-27 10:54 ` Arnd Bergmann

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).