[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