在 2017-04-10 14:59，Maxime Ripard 写道：
On Mon, Apr 10, 2017 at 12:19:41AM +0800, Icenowy Zheng wrote:
According to the researching result of Ondrej Jirman, the factor M of PLL1 shouldn't be used and the factor P should be used only if the intended frequency is lower than 288MHz. This is proven by the clk-sun8iw7_tbl.c in the BSP source code -- in there the M value is always 0 and the maximum frequency that P is not 0 is 224MHz.
As P is ignored on sun6i, it's not currently used. This patch removed the usage of M.
This patch is an original work by Ondrej Jirman, however, he didn't add a Signed-off-by tag here to his commit. So I take this code and added my Signed-off-by.
Signed-off-by: Icenowy Zheng icenowy@aosc.io
This is a critical patch, and should be added to 2017.05.
It has been verified by the Armbian.
This doesn't mean anything. How has this been verified?
Armbian has included this patch since several months ago, and it's proven that without it DVFS will be not stable. (And it proved at least it won't make the system more unstable). See [1].
And for Ondrej's research, please see [2].
I also verified it on my own board.
P.S. it's also the behavior of the clk-sun8iw7_tbl.c, in which the M part of PLL_CPUX is always 0.
[1] https://github.com/igorpecovnik/lib/blob/master/patch/u-boot/u-boot-sunxi/h3... [2] https://github.com/megous/h3-firmware
Maxime