[U-Boot-Users] Bug in cpu/74xx_7xx/speed.c ?

Hello,
during test with the 1.3.1 Version of u-boot, i have noticed, that the calculation of the cpu speed for a 1GHz 750GX is wrong. Here is a patch to fix the problem, it is done the same way as for the other supported cpu's.
Kind regards
Reinhard Arlt
------------------------------------------------------------------------- Dipl.-Ing. Reinhard Arlt Head SD1
esd electronic system design gmbh Vahrenwalder Str. 207 - 30165 Hannover - GERMANY Phone: +49-511-37298-0 - Fax: +49-511-37298-68 Please visit our homepage http://www.esd.eu Quality Products - Made in Germany
--- u-boot-1.3.1_20080212/cpu/74xx_7xx/speed.c 2008-02-12 15:01:13.000000000 +0100 +++ u-boot-1.3.1_head/cpu/74xx_7xx/speed.c 2008-02-12 12:59:46.000000000 +0100 @@ -149,8 +149,8 @@ int get_clocks (void)
case CPU_750GX: case CPU_750FX: - clock = (gd->bus_clk / 10) * - hid1_fx_multipliers_x_10[get_hid1 () >> 27]; + clock = gd->bus_clk * + hid1_fx_multipliers_x_10[get_hid1 () >> 27] / 10; break;
case CPU_7450:

In message 47B1B89D.1080302@esd-electronics.com you wrote:
during test with the 1.3.1 Version of u-boot, i have noticed, that the calculation of the cpu speed for a 1GHz 750GX is wrong. Here is a patch to fix the problem, it is done the same way as for the other supported cpu's.
Please resubmit as prper (git) patch and at least add your signed-off-by line!
Please see http://www.denx.de/wiki/UBoot/Patches for details.
Best regards,
Wolfgang Denk
participants (2)
-
Reinhard Arlt
-
Wolfgang Denk