AGESA f14/f15tn/f16kb: Clean up buildOpts.c files

Until now, the buildOpts.c files were primarily made out of copy-pasted
AGESA options, commented-out definitions and several useless comments;
that is, the materialization of technical debt in GCC-parsable form...

Until now.

It is assumed that the boards in the tree still boot. So, by comparing
their settings, we can extract saner defaults to place into AGESA. Many
of the settings were common across all boards of the same family, so we
promote those values to default settings. In some cases flipping a flag
was required, so the macros to alter that option had to be adapted as
well. Since those AGESA versions are expected to never receive updates,
it should not be a problem to change their files to suit our needs.

As a result, all but two buildOpts.c files now have less than 100 lines.
AGESA f14 boards need less than 50 lines, and f15tn/f16kb just require
about 60 or 70 lines in those files. Hopefully, this will make porting
more mainboards using AGESA f14/f15tn/f16kb a substantially easier task.

TEST=Use abuild --timeless to check that all AGESA f14/f15tn/f16kb
mainboards result in identical coreboot binaries.

Change-Id: Ife1ca5177d85441b9a7b24d64d7fcbabde6e0409
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41667
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Mike Banon <mikebdp2@gmail.com>

1 files changed, 7 insertions, 135 deletions

diff --git a/src/mainboard/amd/inagua/buildOpts.c b/src/mainboard/amd/inagua/buildOpts.c
index aecc9a5b08..73216349ce 100644
--- a/src/mainboard/amd/inagua/buildOpts.c
+++ b/src/mainboard/amd/inagua/buildOpts.c
@@ -1,160 +1,32 @@
 /* SPDX-License-Identifier: GPL-2.0-only */
 
-/**
- * @file
- *
- * AMD User options selection for a Brazos platform solution system
- *
- * This file is placed in the user's platform directory and contains the
- * build option selections desired for that platform.
- *
- * For Information about this file, see @ref platforminstall.
- */
+/* Select the CPU family */
+#define INSTALL_FAMILY_14_SUPPORT		TRUE
 
-/* Select the CPU family. */
-#define INSTALL_FAMILY_14_SUPPORT TRUE
-
-/* Select the CPU socket type. */
+/* Select the CPU socket type */
 #define INSTALL_FT1_SOCKET_SUPPORT		TRUE
 
-/*
- * Agesa optional capabilities selection.
- * Uncomment and mark FALSE those features you wish to include in the build.
- * Comment out or mark TRUE those features you want to REMOVE from the build.
- */
-
+/* Agesa optional capabilities selection */
 #define BLDOPT_REMOVE_UDIMMS_SUPPORT		FALSE
 #define BLDOPT_REMOVE_RDIMMS_SUPPORT		TRUE
 #define BLDOPT_REMOVE_LRDIMMS_SUPPORT		FALSE
 #define BLDOPT_REMOVE_ECC_SUPPORT		FALSE
-//#define BLDOPT_REMOVE_DCT_INTERLEAVE		TRUE
 #define BLDOPT_REMOVE_BANK_INTERLEAVE		FALSE
-#define BLDOPT_REMOVE_NODE_INTERLEAVE		TRUE
-#define BLDOPT_REMOVE_PARALLEL_TRAINING		FALSE
-#define BLDOPT_REMOVE_DQS_TRAINING		FALSE
-#define BLDOPT_REMOVE_ONLINE_SPARE_SUPPORT	TRUE
-#define BLDOPT_REMOVE_MULTISOCKET_SUPPORT	TRUE
-#define BLDOPT_REMOVE_ACPI_PSTATES		FALSE
-#define BLDCFG_REMOVE_ACPI_PSTATES_PPC		FALSE
-#define BLDCFG_REMOVE_ACPI_PSTATES_PCT		FALSE
-#define BLDCFG_REMOVE_ACPI_PSTATES_PSD		FALSE
-#define BLDCFG_REMOVE_ACPI_PSTATES_PSS		FALSE
-#define BLDCFG_REMOVE_ACPI_PSTATES_XPSS		FALSE
-#define BLDCFG_FORCE_INDEPENDENT_PSD_OBJECT	FALSE
-#define BLDOPT_REMOVE_SRAT			FALSE
-#define BLDOPT_REMOVE_SLIT			FALSE
 #define BLDOPT_REMOVE_WHEA			FALSE
-#define BLDOPT_REMOVE_DMI			TRUE
-#define BLDOPT_REMOVE_HT_ASSIST			TRUE
-#define BLDOPT_REMOVE_ATM_MODE			TRUE
-//#define BLDOPT_REMOVE_MSG_BASED_C1E		TRUE
-//#define BLDOPT_REMOVE_LOW_POWER_STATE_FOR_PROCHOT	TRUE
-#define BLDOPT_REMOVE_MEM_RESTORE_SUPPORT	FALSE
-//#define BLDOPT_REMOVE_C6_STATE		TRUE
-#define BLDOPT_REMOVE_GFX_RECOVERY		TRUE
-#define BLDOPT_REMOVE_EARLY_SAMPLES		TRUE
 
-#define BLDCFG_VRM_CURRENT_LIMIT		24000
-//#define BLDCFG_VRM_NB_CURRENT_LIMIT		0
-#define BLDCFG_VRM_LOW_POWER_THRESHOLD		24000
 #define BLDCFG_VRM_NB_LOW_POWER_THRESHOLD	1
-#define BLDCFG_VRM_SLEW_RATE			5000
-//#define BLDCFG_VRM_NB_SLEW_RATE		5000
-//#define BLDCFG_VRM_ADDITIONAL_DELAY		0
-//#define BLDCFG_VRM_NB_ADDITIONAL_DELAY	0
-#define BLDCFG_VRM_HIGH_SPEED_ENABLE		TRUE
-//#define BLDCFG_VRM_NB_HIGH_SPEED_ENABLE	FALSE
-#define BLDCFG_VRM_INRUSH_CURRENT_LIMIT		6000
-//#define BLDCFG_VRM_NB_INRUSH_CURRENT_LIMIT	0
 
-//#define BLDCFG_PROCESSOR_SCOPE_NAME0		'C'
-//#define BLDCFG_PROCESSOR_SCOPE_NAME1		'0'
-//#define BLDCFG_PROCESSOR_SCOPE_IN_SB		FALSE
-#define BLDCFG_PLAT_NUM_IO_APICS		3
-//#define BLDCFG_PLATFORM_C1E_MODE		C1eModeDisabled
-//#define BLDCFG_PLATFORM_C1E_OPDATA		0
-//#define BLDCFG_PLATFORM_C1E_MODE_OPDATA1	0
-//#define BLDCFG_PLATFORM_C1E_MODE_OPDATA2	0
-#define BLDCFG_PLATFORM_CSTATE_MODE		CStateModeC6
-#define BLDCFG_PLATFORM_CSTATE_OPDATA		0x840
-#define BLDCFG_PLATFORM_CSTATE_IO_BASE_ADDRESS	0x840
-//#define BLDCFG_PLATFORM_CPB_MODE		CpbModeAuto
-#define BLDCFG_CORE_LEVELING_MODE		CORE_LEVEL_LOWEST
 #define BLDCFG_AMD_PLATFORM_TYPE		AMD_PLATFORM_MOBILE
-//#define BLDCFG_STARTING_BUSNUM		0
-//#define BLDCFG_MAXIMUM_BUSNUM			0xf8
-//#define BLDCFG_ALLOCATED_BUSNUMS		0x20
-//#define BLDCFG_PLATFORM_DEEMPHASIS_LIST	0
-//#define BLDCFG_BUID_SWAP_LIST			0
-//#define BLDCFG_HTDEVICE_CAPABILITIES_OVERRIDE_LIST	0
-//#define BLDCFG_HTFABRIC_LIMITS_LIST		0
-//#define BLDCFG_HTCHAIN_LIMITS_LIST		0
-//#define BLDCFG_BUS_NUMBERS_LIST		0
-//#define BLDCFG_IGNORE_LINK_LIST		0
-//#define BLDCFG_LINK_SKIP_REGANG_LIST		0
-//#define BLDCFG_ADDITIONAL_TOPOLOGIES_LIST	0
-//#define BLDCFG_USE_HT_ASSIST			TRUE
-//#define BLDCFG_USE_ATM_MODE			TRUE
-//#define BLDCFG_PLATFORM_CONTROL_FLOW_MODE	Nfcm
 #define BLDCFG_S3_LATE_RESTORE			FALSE
-//#define BLDCFG_USE_32_BYTE_REFRESH		FALSE
-//#define BLDCFG_USE_VARIABLE_MCT_ISOC_PRIORITY	FALSE
-//#define BLDCFG_PLATFORM_POWER_POLICY_MODE	Performance
-//#define BLDCFG_SET_HTCRC_SYNC_FLOOD		FALSE
-//#define BLDCFG_USE_UNIT_ID_CLUMPING		FALSE
-//#define BLDCFG_SYSTEM_PHYSICAL_SOCKET_MAP	0
 #define BLDCFG_CFG_GNB_HD_AUDIO			TRUE
-//#define BLDCFG_CFG_ABM_SUPPORT		FALSE
-//#define BLDCFG_CFG_DYNAMIC_REFRESH_RATE	0
-//#define BLDCFG_CFG_LCD_BACK_LIGHT_CONTROL	0
-//#define BLDCFG_MEM_INIT_PSTATE		0
-//#define BLDCFG_AMD_PSTATE_CAP_VALUE		0
-#define BLDCFG_MEMORY_BUS_FREQUENCY_LIMIT	DDR1333_FREQUENCY
-#define BLDCFG_MEMORY_MODE_UNGANGED		TRUE
-//#define BLDCFG_MEMORY_QUAD_RANK_CAPABLE	TRUE
-//#define BLDCFG_MEMORY_QUADRANK_TYPE		QUADRANK_UNBUFFERED
 #define BLDCFG_MEMORY_SODIMM_CAPABLE		TRUE
 #define BLDCFG_MEMORY_LRDIMM_CAPABLE		FALSE
-#define BLDCFG_MEMORY_ENABLE_BANK_INTERLEAVING	TRUE
-#define BLDCFG_MEMORY_ENABLE_NODE_INTERLEAVING	FALSE
 #define BLDCFG_MEMORY_CHANNEL_INTERLEAVING	FALSE
-#define BLDCFG_MEMORY_POWER_DOWN		TRUE
-#define BLDCFG_POWER_DOWN_MODE			POWER_DOWN_BY_CHIP_SELECT
-//#define BLDCFG_ONLINE_SPARE			FALSE
-//#define BLDCFG_MEMORY_PARITY_ENABLE		FALSE
-#define BLDCFG_BANK_SWIZZLE			TRUE
-#define BLDCFG_TIMING_MODE_SELECT		TIMING_MODE_AUTO
-#define BLDCFG_MEMORY_CLOCK_SELECT		DDR1333_FREQUENCY
-#define BLDCFG_DQS_TRAINING_CONTROL		TRUE
-#define BLDCFG_IGNORE_SPD_CHECKSUM		FALSE
-#define BLDCFG_USE_BURST_MODE			FALSE
-#define BLDCFG_MEMORY_ALL_CLOCKS_ON		FALSE
-//#define BLDCFG_ENABLE_ECC_FEATURE		TRUE
-//#define BLDCFG_ECC_REDIRECTION		FALSE
-//#define BLDCFG_SCRUB_DRAM_RATE		0
-//#define BLDCFG_SCRUB_L2_RATE			0
-//#define BLDCFG_SCRUB_L3_RATE			0
-//#define BLDCFG_SCRUB_IC_RATE			0
-//#define BLDCFG_SCRUB_DC_RATE			0
-//#define BLDCFG_ECC_SYNC_FLOOD			0
-//#define BLDCFG_ECC_SYMBOL_SIZE		0
-//#define BLDCFG_1GB_ALIGN			FALSE
-#define BLDCFG_UMA_ALLOCATION_MODE		UMA_AUTO
-#define BLDCFG_UMA_ALLOCATION_SIZE		0
-#define BLDCFG_UMA_ABOVE4G_SUPPORT		FALSE
-#define BLDCFG_UMA_ALIGNMENT			NO_UMA_ALIGNED
-#define BLDCFG_HEAP_DRAM_ADDRESS		0xB0000
-#define BLDCFG_CFG_TEMP_PCIE_MMIO_BASE_ADDRESS	0xD0000000
 
-/*
- * Agesa configuration values selection.
- * Uncomment and specify the value for the configuration options
- * needed by the system.
- */
+/* Agesa configuration values selection */
 #include <AGESA.h>
 
-/* Include the files that instantiate the configuration definitions. */
-
+/* Include the files that instantiate the configuration definitions */
 #include "cpuRegisters.h"
 #include "cpuFamRegisters.h"
 #include "cpuFamilyTranslation.h"
@@ -167,5 +39,5 @@
 #include "cpuLateInit.h"
 #include "GnbInterface.h"
 
-// Instantiate all solution relevant data.
+/* Instantiate all solution relevant data */
 #include <PlatformInstall.h>