]> www.infradead.org Git - users/dwmw2/linux.git/commit
clk: meson: meson8b: do not use cpu_div3 for cpu_scale_out_sel
authorMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Thu, 15 Nov 2018 22:40:44 +0000 (23:40 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 12 Feb 2019 19:02:10 +0000 (20:02 +0100)
commit5e8cb7e03d3dd21e3fde03e4a631c60fb18c6004
treecd5353b55dc1b3558f84884d755979881f2f4346
parentfe6148df4d3f78af4a63238ae701c00980a054d9
clk: meson: meson8b: do not use cpu_div3 for cpu_scale_out_sel

[ Upstream commit a5ac1ead32c9aac285f6436e09b4f6111996e9b8 ]

The cpu_div3 clock (cpu_in divided by 3) generates a signal with a duty
cycle of 33%. The CPU clock however requires a clock signal with a duty
cycle of 50% to run stable.
cpu_div3 was observed to be problematic when cycling through all
available CPU frequencies (with additional patches on top of this one)
while running "stress --cpu 4" in the background. This caused sporadic
hangs where the whole system would fully lock up.

Amlogic's 3.10 kernel code also does not use the cpu_div3 clock either
when changing the CPU clock.

Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Reviewed-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
Link: https://lkml.kernel.org/r/20181115224048.13511-3-martin.blumenstingl@googlemail.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/clk/meson/meson8b.c