diff options
| author | Leah Rowe <leah@libreboot.org> | 2024-11-01 15:59:30 +0000 | 
|---|---|---|
| committer | Leah Rowe <leah@libreboot.org> | 2024-11-01 15:59:30 +0000 | 
| commit | 14b4838d49556eb544c0f6a96d72128c3480ec2c (patch) | |
| tree | 71cc7fb04cd7871ddfc4377b9653990a3d712578 /config/coreboot/default/patches/0014-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch | |
| parent | 67c92889a866d67132bcc37de6444e0bc56f548c (diff) | |
coreboot/default: Re-base all patches
There were a lot of unnecessary patches, such as the VRAM
patches; as Nicholas Chin has explained to me, the drivers
for these machines will just allocate what RAM they want
anyway, so in a lot of cases the extra allocated Video RAM
simply reduces the total amount of memory for other uses.
In general, we have a lot of patches that have existed for
years. A much more aggressive sweep will be done in the next
major audit, especially when the revisions are updated again.
Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'config/coreboot/default/patches/0014-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch')
| -rw-r--r-- | config/coreboot/default/patches/0014-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch | 47 | 
1 files changed, 0 insertions, 47 deletions
| diff --git a/config/coreboot/default/patches/0014-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch b/config/coreboot/default/patches/0014-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch deleted file mode 100644 index 23677556..00000000 --- a/config/coreboot/default/patches/0014-fix-speedstep-on-x200-t400-Revert-cpu-intel-model_10.patch +++ /dev/null @@ -1,47 +0,0 @@ -From a1a4312c9bea5b7fb5170174dbd14f914c11637c Mon Sep 17 00:00:00 2001 -From: Leah Rowe <leah@libreboot.org> -Date: Wed, 1 Dec 2021 02:53:00 +0000 -Subject: [PATCH 14/65] fix speedstep on x200/t400: Revert - "cpu/intel/model_1067x: enable PECI" - -This reverts commit 70fea013c7ebd6d85a7806748233fcfd76802f5f. - -Enabling PECI without microcode updates loaded causes the CPUID feature set -to become corrupted. And one consequence is broken SpeedStep. At least, that's -my understanding looking at Intel Errata. This revert is not a fix, because -upstream is correct (upstream assumes microcode updates). We will simply -maintain this revert patch in Libreboot, from now on. ---- - src/cpu/intel/model_1067x/model_1067x_init.c | 9 --------- - 1 file changed, 9 deletions(-) - -diff --git a/src/cpu/intel/model_1067x/model_1067x_init.c b/src/cpu/intel/model_1067x/model_1067x_init.c -index d051e8915b..30ba2bf0c6 100644 ---- a/src/cpu/intel/model_1067x/model_1067x_init.c -+++ b/src/cpu/intel/model_1067x/model_1067x_init.c -@@ -141,8 +141,6 @@ static void configure_emttm_tables(void) - 	wrmsr(MSR_EMTTM_CR_TABLE(5), msr); - } -  --#define IA32_PECI_CTL		0x5a0 -- - static void configure_misc(const int eist, const int tm2, const int emttm) - { - 	msr_t msr; -@@ -185,13 +183,6 @@ static void configure_misc(const int eist, const int tm2, const int emttm) - 		msr.lo |= (1 << 20);	/* Lock Enhanced SpeedStep Enable */ - 		wrmsr(IA32_MISC_ENABLE, msr); - 	} -- --	/* Enable PECI --	   WARNING: due to Erratum AW67 described in Intel document #318733 --	   the microcode must be updated before this MSR is written to. */ --	msr = rdmsr(IA32_PECI_CTL); --	msr.lo |= 1; --	wrmsr(IA32_PECI_CTL, msr); - } -  - #define PIC_SENS_CFG	0x1aa ---  -2.39.5 - | 
