summaryrefslogtreecommitdiff
path: root/src/mainboard/google/poppy/variants/rammus
diff options
context:
space:
mode:
authorMichael Niewöhner <foss@mniewoehner.de>2020-10-15 00:07:44 +0200
committerPatrick Georgi <pgeorgi@google.com>2020-10-26 06:49:22 +0000
commit6267cc523b4a6f716060214d06a8226412a65837 (patch)
tree0f50876426b750b3b941f1337fcfb873c5ca2e43 /src/mainboard/google/poppy/variants/rammus
parent03102794115230e623ecb95e50e5d7f899956b84 (diff)
soc/intel: drop unneeded ISST configuration code
The code configuring ISST (Intel SpeedShift Technology) sets the ISST capability bits in CPUID.06H:EAX. It does *not* activate HWP (Hardware P-States), which shall be done by the OS only. Since the capability is enabled by default (opt-out), there is nothing to do for us in the enabled-case. Practically speaking, there is no value at all in disabling the capability, since one can configure the OS to not enable HWP if that is desired. The two other bits for EPP and HWP interrupt that were set by the code are not set anymore, too. It was tested, on three platforms so far (CML-U, KBL-H, SKL-U), that these are set as well by default in the MSRs reset value (0x1cc0). To reduce complexity and duplicated code without actual benefit, this code gets dropped. The remaining dt option will be dropped in CB:46462. Test: Linux on Supermicro X11SSM-F detects and enables HWP: [ 0.415017] intel_pstate: HWP enabled Change-Id: I952720cf1de78b00b1bf749f10e9c0acd6ecb6b7 Signed-off-by: Michael Niewöhner <foss@mniewoehner.de> Reviewed-on: https://review.coreboot.org/c/coreboot/+/46460 Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Felix Singer <felixsinger@posteo.net> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/google/poppy/variants/rammus')
0 files changed, 0 insertions, 0 deletions