From b80de180c2ebd7bde5ce4da43d5e0f0c260c25e7 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Mon, 16 May 2022 15:29:53 +0200 Subject: 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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/64400 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons Reviewed-by: Stefan Reinauer --- src/northbridge/amd/agesa/family16kb/Kconfig | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/northbridge/amd') 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 -- cgit v1.2.3