From 704ccafb392212cffc3810197da5cb2b619f1ba6 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Mon, 16 May 2022 14:55:46 +0200 Subject: vendorcode/amd/agesa/f14: Fix improper use of .data AGESA has a lot of code in the .data section which is for initialized data, that in fact should be .rodata. This adds the 'CONST' keyword everywhere it is needed. TEST: See in the .elf file (e.g. using readelf) that there is nothing in .data section. Change-Id: I657d09f05070f5a88a4a162872c961db869a8df3 Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/c/coreboot/+/64399 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons Reviewed-by: Stefan Reinauer --- src/northbridge/amd/agesa/family14/Kconfig | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/northbridge/amd/agesa/family14/Kconfig') diff --git a/src/northbridge/amd/agesa/family14/Kconfig b/src/northbridge/amd/agesa/family14/Kconfig index 941972b1a197..0e5a2bea6b51 100644 --- a/src/northbridge/amd/agesa/family14/Kconfig +++ b/src/northbridge/amd/agesa/family14/Kconfig @@ -17,4 +17,7 @@ config ECAM_MMCONF_BASE_ADDRESS config ECAM_MMCONF_BUS_NUMBER default 64 +config AGESA_BROKEN_DATA_SECTION + default n + endif # NORTHBRIDGE_AMD_AGESA_FAMILY14 -- cgit v1.2.3