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/superio/fintek/f71869ad | |
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/superio/fintek/f71869ad')
0 files changed, 0 insertions, 0 deletions