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

vendorcode/amd/pi/00670F00: Remove unneeded includes 

Vendor code has several headers included into source code that are not
needed in order to build them. Remove unneeded #include. This is part of
controlling the build of unneeded procedures within vendor code.

BUG=b:78610011
TEST=Build grunt.

Change-Id: Id7d451b6be564632836fc64fd343131edb85183a
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/26134
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
This commit is contained in:
Richard Spiegel 2018-05-07 07:36:16 -07:00 committed by Patrick Georgi
parent ae27430a3c
commit 9c1dc7cbe1
5 changed files with 15 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/vendorcode/amd/pi/00670F00/Lib/amdlib.c
View File

@ -45,8 +45,6 @@
#include <AGESA.h> #include <AGESA.h>
#include <cpuRegisters.h> #include <cpuRegisters.h>
#include <Ids.h>
#include <Porting.h>
#include "amdlib.h" #include "amdlib.h"
CODE_GROUP (G1_PEICC) CODE_GROUP (G1_PEICC)
RDATA_GROUP (G1_PEICC) RDATA_GROUP (G1_PEICC)

11
src/vendorcode/amd/pi/00670F00/Proc/Fch/Common/FchPeLib.c
View File

@ -43,10 +43,13 @@
#include "FchPlatform.h" #include "FchPlatform.h"
#include "cpuFamilyTranslation.h" #include "cpuFamilyTranslation.h"
//#include "Porting.h" /*
//#include "AMD.h" * Headers removed for coreboot.
//#include "amdlib.h" * #include "Porting.h"
#include "heapManager.h" * #include "AMD.h"
* #include "amdlib.h"
* #include "heapManager.h"
*/
#define FILECODE PROC_FCH_COMMON_FCHPELIB_FILECODE #define FILECODE PROC_FCH_COMMON_FCHPELIB_FILECODE
/*----------------------------------------------------------------------------------------*/ /*----------------------------------------------------------------------------------------*/

5
src/vendorcode/amd/pi/00670F00/Proc/Fch/Common/MemLib.c
View File

@ -41,7 +41,10 @@
* *
***************************************************************************/ ***************************************************************************/
#include "FchPlatform.h" #include "FchPlatform.h"
#include "Ids.h" /*
* Header removed for coreboot.
* #include "Ids.h"
*/
#define FILECODE PROC_FCH_COMMON_MEMLIB_FILECODE #define FILECODE PROC_FCH_COMMON_MEMLIB_FILECODE

5
src/vendorcode/amd/pi/00670F00/Proc/Psp/PspBaseLib/PspBaseLib.c
View File

@ -48,7 +48,10 @@
*/ */
#include "AGESA.h" #include "AGESA.h"
#include "Filecode.h" #include "Filecode.h"
#include "Fch.h" /*
* Header removed for coreboot.
* #include "Fch.h"
*/
#include "PspBaseLib.h" #include "PspBaseLib.h"
#define FILECODE PROC_PSP_PSPBASELIB_PSPBASELIB_FILECODE #define FILECODE PROC_PSP_PSPBASELIB_PSPBASELIB_FILECODE

5
src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c
View File

@ -39,12 +39,7 @@
* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
* *
***************************************************************************/ ***************************************************************************/
// TODO This list needs to be pruned of anything that is not API
#include "AGESA.h"
#include <amdblocks/agesawrapper.h> #include <amdblocks/agesawrapper.h>
#include "cbfs.h"
#include <console/console.h>
#include <commonlib/loglevel.h>
CONST UINT32 ImageSignature = IMAGE_SIGNATURE; CONST UINT32 ImageSignature = IMAGE_SIGNATURE;
CONST UINT32 ModuleSignature = MODULE_SIGNATURE; CONST UINT32 ModuleSignature = MODULE_SIGNATURE;