summaryrefslogtreecommitdiff
path: root/src/northbridge
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2022-05-16 15:29:53 +0200
committerMartin L Roth <gaumless@tutanota.com>2022-05-28 04:19:20 +0000
commitb80de180c2ebd7bde5ce4da43d5e0f0c260c25e7 (patch)
treebf61ae47c74d627362d88675822bcc1db8b37b9e /src/northbridge
parent704ccafb392212cffc3810197da5cb2b619f1ba6 (diff)
vendorcode/amd/agesa/fam16kb: 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: Ie8817434ee0bc6c195eabe090f195512c0043ae5 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/64400 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/northbridge')
-rw-r--r--src/northbridge/amd/agesa/family16kb/Kconfig3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/northbridge/amd/agesa/family16kb/Kconfig b/src/northbridge/amd/agesa/family16kb/Kconfig
index 8cf919a2d5..01f542d585 100644
--- a/src/northbridge/amd/agesa/family16kb/Kconfig
+++ b/src/northbridge/amd/agesa/family16kb/Kconfig
@@ -24,4 +24,7 @@ config VGA_BIOS_ID
The default VGA BIOS PCI vendor/device ID should be set to the
result of the map_oprom_vendev() function in northbridge.c.
+config AGESA_BROKEN_DATA_SECTION
+ default n
+
endif # NORTHBRIDGE_AMD_AGESA_FAMILY16_KB