Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/22604 )
Change subject: cpu/intel/speedstep: Add Netburst ......................................................................
Patch Set 70:
(1 comment)
https://review.coreboot.org/#/c/22604/70/src/cpu/intel/speedstep/speedstep.c File src/cpu/intel/speedstep/speedstep.c:
https://review.coreboot.org/#/c/22604/70/src/cpu/intel/speedstep/speedstep.c... PS70, Line 113: params->slfm.power = SPEEDSTEP_SLFM_POWER_MEROM; Wrong indent. I thing this all would be cleaner constructed with if () instead of nested switch(), but cannot tell for sure before seeing.