GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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>
This commit is contained in:
Martin Roth 2017-11-11 19:03:39 -07:00
parent b564eae5e1
commit b2c0d08a4e
6 changed files with 6 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/soc/amd/common/BiosCallOuts.h
View File

@ -17,9 +17,7 @@
#ifndef __CALLOUTS_AMD_AGESA_H__ #ifndef __CALLOUTS_AMD_AGESA_H__
#define __CALLOUTS_AMD_AGESA_H__ #define __CALLOUTS_AMD_AGESA_H__
#include <Porting.h> #include "agesawrapper.h"
#include <AGESA.h>
#include <FchPlatform.h>
#define BIOS_HEAP_START_ADDRESS 0x010000000 #define BIOS_HEAP_START_ADDRESS 0x010000000
#define BIOS_HEAP_SIZE 0x30000 #define BIOS_HEAP_SIZE 0x30000

5
src/soc/amd/common/agesawrapper.c
View File

@ -13,14 +13,11 @@
* GNU General Public License for more details. * GNU General Public License for more details.
*/ */
#include <AGESA.h> #include "agesawrapper.h"
#include <cbfs.h> #include <cbfs.h>
#include <cbmem.h> #include <cbmem.h>
#include <delay.h> #include <delay.h>
#include <cpu/x86/mtrr.h> #include <cpu/x86/mtrr.h>
#include <FchPlatform.h>
#include <heapManager.h>
#include <agesawrapper.h>
#include <BiosCallOuts.h> #include <BiosCallOuts.h>
#include <string.h> #include <string.h>

2
src/soc/amd/common/agesawrapper_call.h
View File

@ -14,9 +14,9 @@
#ifndef __AGESAWRAPPER_CALL_H__ #ifndef __AGESAWRAPPER_CALL_H__
#define __AGESAWRAPPER_CALL_H__ #define __AGESAWRAPPER_CALL_H__
#include "agesawrapper.h"
#include <stdint.h> #include <stdint.h>
#include <console/console.h> #include <console/console.h>
#include <AGESA.h>
/* /*
* Possible AGESA_STATUS values: * Possible AGESA_STATUS values:

3
src/soc/amd/common/block/include/amdblocks/psp.h
View File

@ -16,11 +16,10 @@
#ifndef __AMD_PSP_H__ #ifndef __AMD_PSP_H__
#define __AMD_PSP_H__ #define __AMD_PSP_H__
#include <agesawrapper.h>
#include <soc/pci_devs.h> #include <soc/pci_devs.h>
#include <stdint.h> #include <stdint.h>
#include <compiler.h> #include <compiler.h>
#include <Porting.h>
#include <Proc/Psp/PspBaseLib/PspBaseLib.h>
/* x86 to PSP commands */ /* x86 to PSP commands */
#define MBOX_BIOS_CMD_DRAM_INFO 0x01 #define MBOX_BIOS_CMD_DRAM_INFO 0x01

5
src/soc/amd/common/def_callouts.c
View File

@ -15,12 +15,9 @@
*/ */
#include <cbfs.h> #include <cbfs.h>
#include <AGESA.h>
#include <amdlib.h> #include <amdlib.h>
#include <Ids.h>
#include <agesawrapper.h>
#include <BiosCallOuts.h> #include <BiosCallOuts.h>
#include "agesawrapper.h"
#include <soc/southbridge.h> #include <soc/southbridge.h>
#if ENV_BOOTBLOCK #if ENV_BOOTBLOCK

3
src/soc/amd/common/heapmanager.c
View File

@ -12,12 +12,11 @@
*/ */
#include <AGESA.h> #include "agesawrapper.h"
#include <amdlib.h> #include <amdlib.h>
#include <arch/acpi.h> #include <arch/acpi.h>
#include <BiosCallOuts.h> #include <BiosCallOuts.h>
#include <cbmem.h> #include <cbmem.h>
#include <heapManager.h>
#include <string.h> #include <string.h>
static void *GetHeapBase(void) static void *GetHeapBase(void)