GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

util/cbfstool: demote FV handling errors to debug 

It's rather normal that a few bytes are skipped.

Change-Id: I9371afdbb3ad05de7645bfbf257e4f4bfa2feddb
Signed-off-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-on: https://review.coreboot.org/20469
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
Patrick Georgi 2017-07-05 13:30:37 +02:00
parent 8320b9a997
commit 50bda05e46
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
util/cbfstool/cbfs-mkpayload.c
View File

@ -320,7 +320,7 @@ int parse_fv_to_payload(const struct buffer *input, struct buffer *output,
fh = (ffs_file_header_t *)(input->data + fv->header_length); fh = (ffs_file_header_t *)(input->data + fv->header_length);
while (fh->file_type == FILETYPE_PAD) { while (fh->file_type == FILETYPE_PAD) {
unsigned long offset = (fh->size[2] << 16) | (fh->size[1] << 8) | fh->size[0]; unsigned long offset = (fh->size[2] << 16) | (fh->size[1] << 8) | fh->size[0];
ERROR("skipping %lu bytes of FV padding\n", offset); DEBUG("skipping %lu bytes of FV padding\n", offset);
fh = (ffs_file_header_t *)(((uintptr_t)fh) + offset); fh = (ffs_file_header_t *)(((uintptr_t)fh) + offset);
} }
if (fh->file_type != FILETYPE_SEC) { if (fh->file_type != FILETYPE_SEC) {
@ -332,7 +332,7 @@ int parse_fv_to_payload(const struct buffer *input, struct buffer *output,
cs = (common_section_header_t *)&fh[1]; cs = (common_section_header_t *)&fh[1];
while (cs->section_type == SECTION_RAW) { while (cs->section_type == SECTION_RAW) {
unsigned long offset = (cs->size[2] << 16) | (cs->size[1] << 8) | cs->size[0]; unsigned long offset = (cs->size[2] << 16) | (cs->size[1] << 8) | cs->size[0];
ERROR("skipping %lu bytes of section padding\n", offset); DEBUG("skipping %lu bytes of section padding\n", offset);
cs = (common_section_header_t *)(((uintptr_t)cs) + offset); cs = (common_section_header_t *)(((uintptr_t)cs) + offset);
} }
if (cs->section_type != SECTION_PE32) { if (cs->section_type != SECTION_PE32) {