diff options
| author | Arthur Heymans <arthur@aheymans.xyz> | 2022-05-31 21:33:43 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-06-20 12:18:43 +0000 |
| commit | 615818f5a98d19079fd5e1725cd56786d39882c2 (patch) | |
| tree | 8a947124587034c8584426ffce9840ec109bdff5 /src/soc/amd/picasso | |
| parent | 99d2d62fa1034ac3935dd701735704ca839c25a1 (diff) | |
| download | coreboot-615818f5a98d19079fd5e1725cd56786d39882c2.tar.gz coreboot-615818f5a98d19079fd5e1725cd56786d39882c2.tar.bz2 coreboot-615818f5a98d19079fd5e1725cd56786d39882c2.zip | |
soc/amd/*: Make mtrr decision based on syscfg
The syscfg has to option to automatically mark the range between 4G and
TOM2, which contains DRAM, as WB. Making it generally not necessary to
allocate MTRRs for memory above 4G if no PCI BARs are placed up there.
Change-Id: Ifbacae28e272ab2f39f268ad034354a9c590d035
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/64868
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Diffstat (limited to 'src/soc/amd/picasso')
| -rw-r--r-- | src/soc/amd/picasso/cpu.c | 19 |
1 files changed, 12 insertions, 7 deletions
diff --git a/src/soc/amd/picasso/cpu.c b/src/soc/amd/picasso/cpu.c index f14ee4964858..d3e80ebdeaa6 100644 --- a/src/soc/amd/picasso/cpu.c +++ b/src/soc/amd/picasso/cpu.c @@ -1,24 +1,25 @@ /* SPDX-License-Identifier: GPL-2.0-only */ +#include <acpi/acpi.h> #include <amdblocks/cpu.h> #include <amdblocks/mca.h> #include <amdblocks/reset.h> #include <amdblocks/smm.h> #include <assert.h> +#include <console/console.h> +#include <cpu/amd/microcode.h> +#include <cpu/amd/mtrr.h> #include <cpu/cpu.h> #include <cpu/x86/mp.h> -#include <cpu/x86/mtrr.h> #include <cpu/x86/msr.h> +#include <cpu/x86/mtrr.h> #include <cpu/x86/smm.h> -#include <acpi/acpi.h> #include <device/device.h> #include <device/pci_ops.h> -#include <soc/pci_devs.h> #include <soc/cpu.h> -#include <soc/smi.h> #include <soc/iomap.h> -#include <console/console.h> -#include <cpu/amd/microcode.h> +#include <soc/pci_devs.h> +#include <soc/smi.h> #include <types.h> _Static_assert(CONFIG_MAX_CPUS == 8, "Do not override MAX_CPUS. To reduce the number of " @@ -35,7 +36,11 @@ _Static_assert(CONFIG_MAX_CPUS == 8, "Do not override MAX_CPUS. To reduce the nu */ static void pre_mp_init(void) { - x86_setup_mtrrs_with_detect_no_above_4gb(); + const msr_t syscfg = rdmsr(SYSCFG_MSR); + if (syscfg.lo & SYSCFG_MSR_TOM2WB) + x86_setup_mtrrs_with_detect_no_above_4gb(); + else + x86_setup_mtrrs_with_detect(); x86_mtrr_check(); } |