[coreboot] r1157 - coreboot-v3/northbridge/amd/geodelx

Peter Stuge peter at stuge.se
Thu Mar 19 12:40:41 CET 2009


svn at coreboot.org wrote:
> -	speed = 2 * ((10000 / (((spd_byte0 >> 4) * 10) + (spd_byte0 & 0x0F))));
> +	speed = 20000 / (((spd_byte0 >> 4) * 10) + (spd_byte0 & 0x0F));
..
> -			dimm_speed = 2 * (10000 / (((spd_byte >> 4) * 10) +
> -						(spd_byte & 0x0F)));
> +			dimm_speed = 20000 / (((spd_byte >> 4) * 10) + (spd_byte & 0x0F));
..
> -					dimm_speed = 2 * (10000 / (((spd_byte >> 4) * 10) + (spd_byte & 0x0F)));
> +					dimm_speed = 20000 / (((spd_byte >> 4) * 10) + (spd_byte & 0x0F));
..
..
> -			dimm_speed = 2 * (10000 / (((spd_byte >> 4) * 10) + (spd_byte & 0x0F)));
> +			dimm_speed = 20000 / (((spd_byte >> 4) * 10) + (spd_byte & 0x0F));
..
> -					dimm_speed = 2 * (10000 / (((spd_byte >> 4) * 10) + (spd_byte & 0x0F)));
> +					dimm_speed = 20000 / (((spd_byte >> 4) * 10) + (spd_byte & 0x0F));

Anyone else thinks this calls for a macro? Maybe even a generic SPD
macro?


//Peter




More information about the coreboot mailing list