1c25c63c78
Both AGESA.h and cpu/amd/mtrr.h defined TOP_MEM and TOP_MEM2, but since it was defined as unsigned long in AGESA.h, a workaround was needed in cpu/amd/mtrr.h to not have the build fail due to a non-identical redefinition of TOP_MEM and TOP_MEM2. Just removing the workaround without reaming the defines isn't trivially possible, since the stoneyridge romstage.c still ends up including both definitions which can't be easily worked around. Now all non-vendorcode coreboot code uses TOP_MEM_MSR and TOP_MEM2_MSR while the vendorcode part uses TOP_MEM and TOP_MEM2 to avoid this. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: Ibad72dac17bd0b05734709d42c6802b7c8a87455 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74619 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martin.roth@amd.corp-partner.google.com> |
||
|---|---|---|
| .. | ||
| acpi_tables.c | ||
| bootblock.c | ||
| cache_as_ram.S | ||
| def_callouts.c | ||
| eventlog.c | ||
| exit_car.S | ||
| heapmanager.c | ||
| Kconfig | ||
| Makefile.mk | ||
| mtrr_fixme.c | ||
| oem_s3.c | ||
| romstage.c | ||
| s3_mtrr.c | ||
| state_machine.c | ||