build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22604 )
Change subject: cpu/intel/speedstep/acpi.c: Add Netburst CPUs ......................................................................
Patch Set 22:
(4 comments)
https://review.coreboot.org/#/c/22604/22/src/cpu/intel/speedstep/speedstep.c File src/cpu/intel/speedstep/speedstep.c:
https://review.coreboot.org/#/c/22604/22/src/cpu/intel/speedstep/speedstep.c... PS22, Line 45: if (((rdmsr(MSR_EXTENDED_CONFIG).lo >> 27) & 3) == 3) {/*supported line over 80 characters
https://review.coreboot.org/#/c/22604/22/src/cpu/intel/speedstep/speedstep.c... PS22, Line 46: and enabled bits */ line over 80 characters
https://review.coreboot.org/#/c/22604/22/src/cpu/intel/speedstep/speedstep.c... PS22, Line 48: params->slfm = SPEEDSTEP_STATE_FROM_MSR(msr.lo, state_mask); line over 80 characters
https://review.coreboot.org/#/c/22604/22/src/cpu/intel/speedstep/speedstep.c... PS22, Line 80: params->turbo = SPEEDSTEP_STATE_FROM_MSR(msr.hi, state_mask); line over 80 characters