Re: [coreboot] [PATCH] disabling microcode update

[Peter Stuge]

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