[U-Boot] invalid core multipliers?

["Kári Davíðsson" <kari.davidsson@marel.com>]

Hello,

I am bootstrapping an mpc5121e board.

When the cpu speed is calculated (cpu/mpc512x/speed.c) there is a comment about
all core pll settings above 7 are invalid....

I can't see anything about that in the CPU documentation so I am wondering if
anyone has reference to where this is defined as invalid.

The board I am using is using CORE PLL index 0x8, i.e. multiplier of 4.

If this comment in the code is wrong then the following patch should be o.k...

Index: cpu/mpc512x/speed.c
===================================================================
--- cpu/mpc512x/speed.c	(revision 635)
+++ cpu/mpc512x/speed.c	(working copy)
@@ -44,7 +44,7 @@
  	{1, 1}, {3, 2},
  	{2, 1}, {5, 2},
  	{3, 1}, {7, 2},
-	{0, 1}, {0, 1}, /* and all above 7 are not valid too */
+	{4, 1}, {0, 1}, /* and all above 7 are not valid too.... I doubt this claim */
  	{0, 1}, {0, 1},
  	{0, 1}, {0, 1},
  	{0, 1}, {0, 1}

rg
kd