[coreboot] [PATCH] disabling microcode update

Peter Stuge peter at stuge.se
Sat Feb 26 19:17:46 CET 2011


xdrudis wrote:
> > HT Frequencies are multiples of 200MHz AFAIK, so there are no
> > 300MHz and 500MHz.
..
> Oh! You may well be right. All others are multiples of 200 MHz .
> 
> Then we should maybe remove 2 #elif in the following code. But I
> wonder whether the break in the progression of values indicates
> that all values from there on should be changed too.

A safer change might be:

> #elif CONFIG_EXPERT && CONFIG_LIMIT_HT_SPEED_300
> #elif CONFIG_EXPERT && CONFIG_LIMIT_HT_SPEED_500

#elif CONFIG_EXPERT && defined(CONFIG_LIMIT_HT_SPEED_300) && CONFIG_LIMIT_HT_SPEED_300
#elif CONFIG_EXPERT && defined(CONFIG_LIMIT_HT_SPEED_500) && CONFIG_LIMIT_HT_SPEED_500


I would ack that if it builds.


//Peter




More information about the coreboot mailing list