[coreboot-gerrit] Change in coreboot[master]: drivers/intel/fsp1_1: Drop dead MMA code

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/51951 )

Change subject: drivers/intel/fsp1_1: Drop dead MMA code
......................................................................

drivers/intel/fsp1_1: Drop dead MMA code

The only FSP 1.1 platform with MMA support is Skylake. As it now uses
Kaby Lake FSP 2.0, this code is no longer useful. Drop it.

Change-Id: I819c3152bdea0fdad629793d96136ef134429fbd
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51951
Tested-by: build bot (Jenkins) 
Reviewed-by: Arthur Heymans 
Reviewed-by: Michael Niewöhner 
---
M src/drivers/intel/fsp1_1/Makefile.inc
D src/drivers/intel/fsp1_1/mma_core.c
2 files changed, 0 insertions(+), 40 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved
  Michael Niewöhner: Looks good to me, approved

diff --git a/src/drivers/intel/fsp1_1/Makefile.inc b/src/drivers/intel/fsp1_1/Makefile.inc
index 4259237..4b8e0a8 100644
--- a/src/drivers/intel/fsp1_1/Makefile.inc
+++ b/src/drivers/intel/fsp1_1/Makefile.inc
@@ -16,14 +16,12 @@
 romstage-y += hob.c
 romstage-y += raminit.c
 romstage-y += romstage.c
-romstage-$(CONFIG_MMA) += mma_core.c

ramstage-y += fsp_relocate.c
 ramstage-y += fsp_util.c
 ramstage-y += hob.c
 ramstage-y += ramstage.c
 ramstage-$(CONFIG_INTEL_GMA_ADD_VBT) += vbt.c
-ramstage-$(CONFIG_MMA) += mma_core.c

CPPFLAGS_common += -Isrc/drivers/intel/fsp1_1/include

diff --git a/src/drivers/intel/fsp1_1/mma_core.c b/src/drivers/intel/fsp1_1/mma_core.c
deleted file mode 100644
index 6e50373..0000000
--- a/src/drivers/intel/fsp1_1/mma_core.c
+++ /dev/null
@@ -1,38 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-
-#include <console/console.h>
-#include <fsp/util.h>
-#include <fsp/romstage.h>
-#include <fsp/soc_binding.h>
-
-#define FSP_MMA_RESULTS_GUID	{ 0x8f4e928, 0xf5f, 0x46d4, \
-		{ 0x84, 0x10, 0x47, 0x9f, 0xda, 0x27, 0x9d, 0xb6 } }
-
-int fsp_locate_mma_results(const void **mma_hob, size_t *mma_hob_size)
-{
-	const void *mma_hob_start;
-	const EFI_GUID mma_results_guid = FSP_MMA_RESULTS_GUID;
-
-	mma_hob_start = get_first_guid_hob(&mma_results_guid);
-	if (!mma_hob_start)
-		return -1;
-	*mma_hob = GET_GUID_HOB_DATA(mma_hob_start);
-	*mma_hob_size = GET_HOB_LENGTH(mma_hob);
-
-	if (!(*mma_hob_size) || !(*mma_hob))
-		return -1;
-	return 0;
-}
-
-void setup_mma(MEMORY_INIT_UPD *memory_cfg)
-{
-	struct mma_config_param mma_cfg;
-
-	if (mma_locate_param(&mma_cfg)) {
-		printk(BIOS_DEBUG, "MMA: set up failed\n");
-		return;
-	}
-
-	soc_update_memory_params_for_mma(memory_cfg, &mma_cfg);
-	printk(BIOS_DEBUG, "MMA: set up completed successfully\n");
-}