Attention is currently required from: Raul Rangel, Martin Roth, Felix Held.

Bao Zheng uploaded patch set #38 to this change.

View Change

amdfwtool: Use "*SIZE" in fw config file as NVRAM entry

Sometimes the PSP or BIOS entry is all 0xFF. It doesn't make any
sense to integrate a dummy binary. Only the entry type and size
matter.
So, add the entry like
---------------
XXXX_FW_TYPE_STRING *0xXXXXX
---------------
to fw config file. The asterisk is the prefix to detect if the current
entry specifies size instead of a filename. Then the required size of
space related to the firmware type will be reserved.
The TODO comment is completed and removed.

TEST=Bilby. FTPM code can run.

Change-Id: I98c94b4087f8a174f7d5753a30695a10f1b796ae
Signed-off-by: Zheng Bao <fishbaozi@gmail.com>
---
M src/soc/amd/picasso/Makefile.inc
M src/soc/amd/stoneyridge/Makefile.inc
M src/southbridge/amd/pi/hudson/Makefile.inc
M util/amdfwtool/amdfwtool.c
M util/amdfwtool/amdfwtool.h
M util/amdfwtool/data_parse.c
6 files changed, 43 insertions(+), 19 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/49015/38

To view, visit change 49015. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I98c94b4087f8a174f7d5753a30695a10f1b796ae
Gerrit-Change-Number: 49015
Gerrit-PatchSet: 38
Gerrit-Owner: Bao Zheng <fishbaozi@gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot@felixheld.de>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk@gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd@gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Raul Rangel <rrangel@chromium.org>
Gerrit-Reviewer: Zheng Bao
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Attention: Raul Rangel <rrangel@chromium.org>
Gerrit-Attention: Martin Roth <martinroth@google.com>
Gerrit-Attention: Felix Held <felix-coreboot@felixheld.de>
Gerrit-MessageType: newpatchset