Skip to content

Commit

Permalink
clk: meson: g12a: fix gp0 and hifi ranges
Browse files Browse the repository at this point in the history
While some SoC samples are able to lock with a PLL factor of 55, others
samples can't. ATM, a minimum of 60 appears to work on all the samples
I have tried.

Even with 60, it sometimes takes a long time for the PLL to eventually
lock. The documentation says that the minimum rate of these PLLs DCO
should be 3GHz, a factor of 125. Let's use that to be on the safe side.

With factor range changed, the PLL seems to lock quickly (enough) so far.
It is still unclear if the range was the only reason for the delay.

Fixes: 085a4ea ("clk: meson: g12a: add peripheral clock controller")
Signed-off-by: Jerome Brunet <jbrunet@baylibre.com>
Acked-by: Neil Armstrong <narmstrong@baylibre.com>
Link: https://lore.kernel.org/r/20210429090325.60970-1-jbrunet@baylibre.com
  • Loading branch information
jbrun3t committed May 20, 2021
1 parent 4cbf0cd commit bc794f8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/clk/meson/g12a.c
Original file line number Diff line number Diff line change
Expand Up @@ -1603,7 +1603,7 @@ static struct clk_regmap g12b_cpub_clk_trace = {
};

static const struct pll_mult_range g12a_gp0_pll_mult_range = {
.min = 55,
.min = 125,
.max = 255,
};

Expand Down

0 comments on commit bc794f8

Please sign in to comment.