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 23:
(4 comments)
https://review.coreboot.org/#/c/22604/23/src/cpu/intel/speedstep/speedstep.c File src/cpu/intel/speedstep/speedstep.c:
https://review.coreboot.org/#/c/22604/23/src/cpu/intel/speedstep/speedstep.c... PS23, Line 45: if (((rdmsr(MSR_EXTENDED_CONFIG).lo >> 27) & 3) == 3) {/*supported line over 80 characters
https://review.coreboot.org/#/c/22604/23/src/cpu/intel/speedstep/speedstep.c... PS23, Line 46: and enabled bits */ line over 80 characters
https://review.coreboot.org/#/c/22604/23/src/cpu/intel/speedstep/speedstep.c... PS23, Line 48: params->slfm = SPEEDSTEP_STATE_FROM_MSR(msr.lo, state_mask); line over 80 characters
https://review.coreboot.org/#/c/22604/23/src/cpu/intel/speedstep/speedstep.c... PS23, Line 80: params->turbo = SPEEDSTEP_STATE_FROM_MSR(msr.hi, state_mask); line over 80 characters