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 -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
