Stefan, On Mon, Jul 25, 2011 at 7:55 AM, Stefan Roese <[email protected]> wrote: > Hi Mike, > > On Thursday 21 July 2011 17:06:03 Mike Williams wrote: >> This code has been changed to read the CPU speed information from the >> CPR registers rather than the bootstrap registers. This is useful when >> changing the clock speed to something other than the default on boot. > > Thanks. Unfortunately this patch introduces compile errors for 460SX boards > (e.g. redwood): > > [stefan@kubuntu u-boot-ppc4xx (master)]$ ./MAKEALL redwood > Configuring for redwood board... > speed.c: In function 'get_sys_info': > speed.c:339: error: 'PLLD_FWDVA_MASK' undeclared (first use in this function) > speed.c:339: error: (Each undeclared identifier is reported only once > speed.c:339: error: for each function it appears in.) > speed.c:340: error: 'PLLD_FWDVB_MASK' undeclared (first use in this function) > speed.c:341: error: 'PLLD_FBDV_MASK' undeclared (first use in this function) > speed.c:344: error: 'OPBDV_MASK' undeclared (first use in this function) > speed.c:348: error: 'PERDV_MASK' undeclared (first use in this function) > speed.c:351: error: 'CPR0_PLBED' undeclared (first use in this function) > speed.c:352: error: 'PLBEDDV_MASK' undeclared (first use in this function) > speed.c:357: error: 'PLLC_FBSEL_MASK' undeclared (first use in this function) > make[1]: *** [speed.o] Error 1 > make[1]: *** Waiting for unfinished jobs.... > make: *** [arch/powerpc/cpu/ppc4xx/libppc4xx.o] Error 2 > make: *** Waiting for unfinished jobs.... > > Could you please check, if the 460SX has the same registers and bits as > 460EX/GT and extend its header as well?
I forgot about the SX. Unfortunately, it looks like the SX has a significantly different clocking system than the EX/GT. For instance, it has two PLLs, different available dividers, etc. Additionally, I don't have an SX board to test with. Perhaps I should save the old code path for that board? Let me know what you think. Mike _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
