]> git.baikalelectronics.ru Git - kernel.git/commit
clk: meson-gxbb: set fclk_div3 as CLK_IS_CRITICAL
authorChristian Hewitt <christianshewitt@gmail.com>
Mon, 5 Nov 2018 23:08:20 +0000 (00:08 +0100)
committerStephen Boyd <sboyd@kernel.org>
Thu, 8 Nov 2018 18:12:46 +0000 (10:12 -0800)
commit1a835a358d80e98192c5fb0c8832705066b2b0fd
tree5a9cb10d0477ffc29598096e655907884456d51c
parentad7e3750b02eb2cb717834d031e5766e714518f6
clk: meson-gxbb: set fclk_div3 as CLK_IS_CRITICAL

On the Khadas VIM2 (GXM) and LePotato (GXL) board there are problems
with reboot; e.g. a ~60 second delay between issuing reboot and the
board power cycling (and in some OS configurations reboot will fail
and require manual power cycling).

Similar to 'commit 20529f5f7759b4ad7dcf46c113c398e303ccd20c ("clk:
meson-gxbb: set fclk_div2 as CLK_IS_CRITICAL")' the SCPI Cortex-M4
Co-Processor seems to depend on FCLK_DIV3 being operational.

Until commit 3d584f7f76cc24ec740c29ee5ec50cf17bd67b51 ("clk:
meson: add fdiv clock gates"), this clock was modeled and left on by
the bootloader.

We don't have precise documentation about the SCPI Co-Processor and
its clock requirement so we are learning things the hard way.

Marking this clock as critical solves the problem but it should not
be viewed as final solution. Ideally, the SCPI driver should claim
these clocks. We also depends on some clock hand-off mechanism
making its way to CCF, to make sure the clock stays on between its
registration and the SCPI driver probe.

Fixes: 3d584f7f76cc ("clk: meson: add fdiv clock gates")
Signed-off-by: Christian Hewitt <christianshewitt@gmail.com>
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/meson/gxbb.c