diff options
| author | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2014-03-29 16:52:46 -0500 |
|---|---|---|
| committer | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2014-12-06 05:43:20 +0100 |
| commit | 986349df6492068a7f01fdfd8a6e65b93eb1bc17 (patch) | |
| tree | acc8cc5283d083be5702078caf3c94a4be29fc4b /src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h | |
| parent | c5c21d3fe114063887653a91920dcc3a7c6a436a (diff) | |
| download | coreboot-986349df6492068a7f01fdfd8a6e65b93eb1bc17.tar.xz | |
vendorcode/amd/agesa/fam15tn: Clean #includes in public headers
Right now, coreboot code using AGESA headers can only build if all the
AGESA path are given to the compiler via the "-I" option. This is sub-
optimal, as it requires us to have every AGESA source directory
specified as a compiler include path. This pollutes our global include
paths.
We restrict the compiler include paths to only allow "AGESA_ROOT/" and
"AGESA_ROOT/Include". We then modify the AGESA headers to specify
non-local include files relative to "AGESA_ROOT/Include".
We use the convention that includes relative to the directory of the
header are included as "path/to/header.h", while includes relative to
AGESA_ROOT are included as <path/to/header.h>.
This change allows building coreboot code based on AGESA with the
limited subset of include paths, but does not allow AGESA itself to
build with this restricted subset.
Change-Id: I31102273c8caa8d6b1d80774bfd35711825bec03
Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Reviewed-on: http://review.coreboot.org/5424
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h')
| -rw-r--r-- | src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h b/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h index 83c28d2fdd..a60667f293 100644 --- a/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h +++ b/src/vendorcode/amd/agesa/f15tn/Include/OptionIdsInstall.h @@ -43,8 +43,8 @@ #define _OPTION_IDS_INSTALL_H_ #include "Ids.h" #include "IdsHt.h" -#include "IdsLib.h" -#include "IdsDebugPrint.h" +#include <Proc/IDS/IdsLib.h> +#include <Proc/IDS/Debug/IdsDebugPrint.h> #ifdef __IDS_EXTENDED__ #include OPTION_IDS_EXT_INSTALL_FILE #endif |