From 50bda05e46d5f34f6519a7814e843d1e777ecf10 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Wed, 5 Jul 2017 13:30:37 +0200 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/20469 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- util/cbfstool/cbfs-mkpayload.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/util/cbfstool/cbfs-mkpayload.c b/util/cbfstool/cbfs-mkpayload.c index 45d36f4afe..fd2c4ca011 100644 --- a/util/cbfstool/cbfs-mkpayload.c +++ b/util/cbfstool/cbfs-mkpayload.c @@ -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); while (fh->file_type == FILETYPE_PAD) { 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); } 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]; while (cs->section_type == SECTION_RAW) { 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); } if (cs->section_type != SECTION_PE32) {