diff options
author | Martin Roth <martinroth@google.com> | 2017-11-10 20:28:44 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-11-11 21:08:18 +0000 |
commit | 16995fb7eacf3a2b946795ab327c3c1764ab81ca (patch) | |
tree | dc32128e022b7c10107790513258365b9c57435b /src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c | |
parent | 15129b4db44fd63b1fe7076d7825c1f8bd00588d (diff) | |
download | coreboot-16995fb7eacf3a2b946795ab327c3c1764ab81ca.tar.gz coreboot-16995fb7eacf3a2b946795ab327c3c1764ab81ca.tar.bz2 coreboot-16995fb7eacf3a2b946795ab327c3c1764ab81ca.zip |
vendorcode/amd/pi/00670f00: Set ModuleIdentifier to be 8 bytes
ModuleIdentifier must be 8 bytes. Every other location else that uses
this value explicityly defines it as 8 bytes. If it's initialized here
to less than 8 bytes, it gets passed to those other locations with
garbage at the end and fails to load the AGESA binary.
TEST=Build & boot Kahlee
BUG=B:69165234
Change-Id: I11fc90748f49782e2b16ee5326aee17cfe92d0bc
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/22430
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Justin TerAvest <teravest@chromium.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c')
-rw-r--r-- | src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c b/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c index 5a41897ff268..ec414b472425 100644 --- a/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c +++ b/src/vendorcode/amd/pi/00670F00/binaryPI/AGESA.c @@ -53,7 +53,7 @@ CONST UINT32 ImageSignature = IMAGE_SIGNATURE; CONST UINT32 ModuleSignature = MODULE_SIGNATURE; -CONST CHAR8 ModuleIdentifier[] = AGESA_ID; +CONST CHAR8 ModuleIdentifier[8] = AGESA_ID; /********************************************************************** * Interface call: AmdCreateStruct |