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

Bao Zheng uploaded patch set #33 to this change.

View Change

amdfwtool: Use *number in 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 filename. Then the required size of space
related to the firmware type will be reserved.
The TODO in the comment is completed and removed.

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, 41 insertions(+), 15 deletions(-)

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

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: 33
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: Felix Held <felix-coreboot@felixheld.de>
Gerrit-MessageType: newpatchset