Re: [PATCH 3/3] clk: meson: clk-pll: drop hard-coded rates from pll tables

From: Jerome Brunet
Date: Thu Jul 19 2018 - 04:59:11 EST


On Thu, 2018-07-19 at 10:44 +0200, Neil Armstrong wrote:
> We could even add ranges instead of table when we know the PLL supports a well-known continuous dividers range.

I was thinking about this too.
I did not went for it because it would mean yet another rework of the pll
driver, which I did not had time to do now.

I suspect that the min and max value of 'm' the pll can lock on might depend on
the input rate of the DCO, so past the 'n' prediv.

So, to replace the tuple (m, n) table with ranges, I think it would be best to
take the predivider 'n' out first and try to clarify the contraints on the input
rate of the DCO with amlogic ... we can also try and see ;)

>
> Acked-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx>