From b2c0d08a4e34b30a9419ddee1b526b006fded6d7 Mon Sep 17 00:00:00 2001
From: Martin Roth <martinroth@google.com>
Date: Sat, 11 Nov 2017 19:03:39 -0700
Subject: soc/amd/common: Remove direct AGESA header includes

All AGESA headers should be included only through agesawrapper.h

BUG=b:66818758
TEST=Build gardenia; Build & boot kahlee

Change-Id: I94140235f46a627dda99540af8619aeca3f4f157
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/21715
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
---
 src/soc/amd/common/BiosCallOuts.h                | 4 +---
 src/soc/amd/common/agesawrapper.c                | 5 +----
 src/soc/amd/common/agesawrapper_call.h           | 2 +-
 src/soc/amd/common/block/include/amdblocks/psp.h | 3 +--
 src/soc/amd/common/def_callouts.c                | 5 +----
 src/soc/amd/common/heapmanager.c                 | 3 +--
 6 files changed, 6 insertions(+), 16 deletions(-)

(limited to 'src/soc/amd/common')

diff --git a/src/soc/amd/common/BiosCallOuts.h b/src/soc/amd/common/BiosCallOuts.h
index a2c719f2f5..5237d52b6a 100644
--- a/src/soc/amd/common/BiosCallOuts.h
+++ b/src/soc/amd/common/BiosCallOuts.h
@@ -17,9 +17,7 @@
 #ifndef __CALLOUTS_AMD_AGESA_H__
 #define __CALLOUTS_AMD_AGESA_H__
 
-#include <Porting.h>
-#include <AGESA.h>
-#include <FchPlatform.h>
+#include "agesawrapper.h"
 
 #define BIOS_HEAP_START_ADDRESS		0x010000000
 #define BIOS_HEAP_SIZE			0x30000
diff --git a/src/soc/amd/common/agesawrapper.c b/src/soc/amd/common/agesawrapper.c
index 17f2e76464..80887ab091 100644
--- a/src/soc/amd/common/agesawrapper.c
+++ b/src/soc/amd/common/agesawrapper.c
@@ -13,14 +13,11 @@
  * GNU General Public License for more details.
  */
 
-#include <AGESA.h>
+#include "agesawrapper.h"
 #include <cbfs.h>
 #include <cbmem.h>
 #include <delay.h>
 #include <cpu/x86/mtrr.h>
-#include <FchPlatform.h>
-#include <heapManager.h>
-#include <agesawrapper.h>
 #include <BiosCallOuts.h>
 #include <string.h>
 
diff --git a/src/soc/amd/common/agesawrapper_call.h b/src/soc/amd/common/agesawrapper_call.h
index 21a1e23d54..01e2620961 100644
--- a/src/soc/amd/common/agesawrapper_call.h
+++ b/src/soc/amd/common/agesawrapper_call.h
@@ -14,9 +14,9 @@
 #ifndef __AGESAWRAPPER_CALL_H__
 #define __AGESAWRAPPER_CALL_H__
 
+#include "agesawrapper.h"
 #include <stdint.h>
 #include <console/console.h>
-#include <AGESA.h>
 
 /*
  * Possible AGESA_STATUS values:
diff --git a/src/soc/amd/common/block/include/amdblocks/psp.h b/src/soc/amd/common/block/include/amdblocks/psp.h
index 74ea73f7f7..57ba64945b 100644
--- a/src/soc/amd/common/block/include/amdblocks/psp.h
+++ b/src/soc/amd/common/block/include/amdblocks/psp.h
@@ -16,11 +16,10 @@
 #ifndef __AMD_PSP_H__
 #define __AMD_PSP_H__
 
+#include <agesawrapper.h>
 #include <soc/pci_devs.h>
 #include <stdint.h>
 #include <compiler.h>
-#include <Porting.h>
-#include <Proc/Psp/PspBaseLib/PspBaseLib.h>
 
 /* x86 to PSP commands */
 #define MBOX_BIOS_CMD_DRAM_INFO    0x01
diff --git a/src/soc/amd/common/def_callouts.c b/src/soc/amd/common/def_callouts.c
index 9cea9658fa..e8d404e465 100644
--- a/src/soc/amd/common/def_callouts.c
+++ b/src/soc/amd/common/def_callouts.c
@@ -15,12 +15,9 @@
  */
 
 #include <cbfs.h>
-
-#include <AGESA.h>
 #include <amdlib.h>
-#include <Ids.h>
-#include <agesawrapper.h>
 #include <BiosCallOuts.h>
+#include "agesawrapper.h"
 #include <soc/southbridge.h>
 
 #if ENV_BOOTBLOCK
diff --git a/src/soc/amd/common/heapmanager.c b/src/soc/amd/common/heapmanager.c
index 5e4fbb4166..b99151487d 100644
--- a/src/soc/amd/common/heapmanager.c
+++ b/src/soc/amd/common/heapmanager.c
@@ -12,12 +12,11 @@
  */
 
 
-#include <AGESA.h>
+#include "agesawrapper.h"
 #include <amdlib.h>
 #include <arch/acpi.h>
 #include <BiosCallOuts.h>
 #include <cbmem.h>
-#include <heapManager.h>
 #include <string.h>
 
 static void *GetHeapBase(void)
-- 
cgit v1.2.3