[U-Boot] [PATCH v2 1/3] sunxi: sun4i: improve cpu clock selection method
Hans de Goede
hdegoede at redhat.com
Sun Mar 29 14:39:56 CEST 2015
Hi,
On 28-03-15 11:25, Iain Paton wrote:
> clock_set_pll1 would pick the next highest available cpu clock speed if
> a value not in the pre defined table was selected. this potentially
> results in overclocking the soc.
>
> reverse the selection method so that we select the next lowest speed
> and add the missing 912Mhz setting that's requested by sun7i which also
> uses the sun4i clock code.
>
> Signed-off-by: Iain Paton <ipaton0 at gmail.com>
Thanks for the new set.
I've found one small issue with the second patch:
"sunxi: use CONFIG_SYS_CLK_FREQ to set cpu clock"
sun7i.h contained the following:
#define CONFIG_SYS_CLK_FREQ 24000000
#define CONFIG_TIMER_CLK_FREQ CONFIG_SYS_CLK_FREQ
Which is a conflicting usage of CONFIG_SYS_CLK_FREQ compared to the new
usage introduced by your patch. I've fixed this by changing the above
to:
#define CONFIG_TIMER_CLK_FREQ 24000000
And run some compile and runtime tests.
Everything builds and works fine with this change added, so I've pushed
these 3 commits + one other fix to: u-boot-sunxi/master
And I will send out a pull-req shortly to get these fixes included into
the upcoming u-boot v2015.04 release.
Regards,
Hans
> ---
> arch/arm/cpu/armv7/sunxi/clock_sun4i.c | 35 ++++++++++++++++++----------------
> 1 file changed, 19 insertions(+), 16 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/sunxi/clock_sun4i.c b/arch/arm/cpu/armv7/sunxi/clock_sun4i.c
> index 49f4032..c3e04af 100644
> --- a/arch/arm/cpu/armv7/sunxi/clock_sun4i.c
> +++ b/arch/arm/cpu/armv7/sunxi/clock_sun4i.c
> @@ -100,22 +100,23 @@ static struct {
> unsigned int freq;
> } pll1_para[] = {
> /* This array must be ordered by frequency. */
> - { PLL1_CFG(16, 0, 0, 0), 384000000 },
> - { PLL1_CFG(16, 1, 0, 0), 768000000 },
> - { PLL1_CFG(20, 1, 0, 0), 960000000 },
> - { PLL1_CFG(21, 1, 0, 0), 1008000000},
> - { PLL1_CFG(22, 1, 0, 0), 1056000000},
> - { PLL1_CFG(23, 1, 0, 0), 1104000000},
> - { PLL1_CFG(24, 1, 0, 0), 1152000000},
> - { PLL1_CFG(25, 1, 0, 0), 1200000000},
> - { PLL1_CFG(26, 1, 0, 0), 1248000000},
> - { PLL1_CFG(27, 1, 0, 0), 1296000000},
> - { PLL1_CFG(28, 1, 0, 0), 1344000000},
> - { PLL1_CFG(29, 1, 0, 0), 1392000000},
> - { PLL1_CFG(30, 1, 0, 0), 1440000000},
> { PLL1_CFG(31, 1, 0, 0), 1488000000},
> - /* Final catchall entry */
> - { PLL1_CFG(31, 1, 0, 0), ~0},
> + { PLL1_CFG(30, 1, 0, 0), 1440000000},
> + { PLL1_CFG(29, 1, 0, 0), 1392000000},
> + { PLL1_CFG(28, 1, 0, 0), 1344000000},
> + { PLL1_CFG(27, 1, 0, 0), 1296000000},
> + { PLL1_CFG(26, 1, 0, 0), 1248000000},
> + { PLL1_CFG(25, 1, 0, 0), 1200000000},
> + { PLL1_CFG(24, 1, 0, 0), 1152000000},
> + { PLL1_CFG(23, 1, 0, 0), 1104000000},
> + { PLL1_CFG(22, 1, 0, 0), 1056000000},
> + { PLL1_CFG(21, 1, 0, 0), 1008000000},
> + { PLL1_CFG(20, 1, 0, 0), 960000000 },
> + { PLL1_CFG(19, 1, 0, 0), 912000000 },
> + { PLL1_CFG(16, 1, 0, 0), 768000000 },
> + /* Final catchall entry 384MHz*/
> + { PLL1_CFG(16, 0, 0, 0), 0 },
> +
> };
>
> void clock_set_pll1(unsigned int hz)
> @@ -126,10 +127,12 @@ void clock_set_pll1(unsigned int hz)
> (struct sunxi_ccm_reg *)SUNXI_CCM_BASE;
>
> /* Find target frequency */
> - while (pll1_para[i].freq < hz)
> + while (pll1_para[i].freq > hz)
> i++;
>
> hz = pll1_para[i].freq;
> + if (! hz)
> + hz = 384000000;
>
> /* Calculate system clock divisors */
> axi = DIV_ROUND_UP(hz, 432000000); /* Max 450MHz */
>
More information about the U-Boot
mailing list