[coreboot] Enabling SpeedStep for all Denverton SoC variants

Sumo kingsumos at gmail.com
Tue May 29 16:48:18 CEST 2018


Hi,

Today denverton_core_init() is enabling the Intel SpeedStep only if turbo
mode is available. As a result, the SpeedStep is not enable for the C3558
variant (and others).  Any clues of why it was implemented this way?
I think it is safe to remove the "if (get_turbo_state() == TURBO_ENABLED)"
test and always enable the SpeedStep technology.

Thanks,
Sumo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot/attachments/20180529/f4628b55/attachment.html>


More information about the coreboot mailing list