在 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 <icen...@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-Fix-PLL1-setup-to-never-use-dividers.patch
[2] https://github.com/megous/h3-firmware
Maxime
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
