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/Proc/GNB | |
parent | c5c21d3fe114063887653a91920dcc3a7c6a436a (diff) |
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/Proc/GNB')
-rw-r--r-- | src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbInitTN/GnbInitTNInstall.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbInitTN/GnbInitTNInstall.h b/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbInitTN/GnbInitTNInstall.h index 60558188d8..1be9d7cafc 100644 --- a/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbInitTN/GnbInitTNInstall.h +++ b/src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbInitTN/GnbInitTNInstall.h @@ -50,9 +50,9 @@ // Specify definition used by module services //----------------------------------------------------------------------- -#include "GnbPcie.h" -#include "GnbPcieFamServices.h" -#include "GnbFamServices.h" +#include <Proc/GNB/Common/GnbPcie.h> +#include <Proc/GNB/Common/GnbPcieFamServices.h> +#include <Proc/GNB/Common/GnbFamServices.h> //----------------------------------------------------------------------- // Export services |