<p>Patrick Georgi has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20469">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">util/cbfstool: demote FV handling errors to debug<br><br>It's rather normal that a few bytes are skipped.<br><br>Change-Id: I9371afdbb3ad05de7645bfbf257e4f4bfa2feddb<br>Signed-off-by: Patrick Georgi <pgeorgi@google.com><br>---<br>M util/cbfstool/cbfs-mkpayload.c<br>1 file changed, 2 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/20469/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/util/cbfstool/cbfs-mkpayload.c b/util/cbfstool/cbfs-mkpayload.c<br>index 45d36f4..fd2c4ca 100644<br>--- a/util/cbfstool/cbfs-mkpayload.c<br>+++ b/util/cbfstool/cbfs-mkpayload.c<br>@@ -320,7 +320,7 @@<br>         fh = (ffs_file_header_t *)(input->data + fv->header_length);<br>    while (fh->file_type == FILETYPE_PAD) {<br>            unsigned long offset = (fh->size[2] << 16) | (fh->size[1] << 8) | fh->size[0];<br>-          ERROR("skipping %lu bytes of FV padding\n", offset);<br>+               DEBUG("skipping %lu bytes of FV padding\n", offset);<br>                fh = (ffs_file_header_t *)(((uintptr_t)fh) + offset);<br>         }<br>     if (fh->file_type != FILETYPE_SEC) {<br>@@ -332,7 +332,7 @@<br>  cs = (common_section_header_t *)&fh[1];<br>   while (cs->section_type == SECTION_RAW) {<br>          unsigned long offset = (cs->size[2] << 16) | (cs->size[1] << 8) | cs->size[0];<br>-          ERROR("skipping %lu bytes of section padding\n", offset);<br>+          DEBUG("skipping %lu bytes of section padding\n", offset);<br>           cs = (common_section_header_t *)(((uintptr_t)cs) + offset);<br>   }<br>     if (cs->section_type != SECTION_PE32) {<br></pre><p>To view, visit <a href="https://review.coreboot.org/20469">change 20469</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/20469"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I9371afdbb3ad05de7645bfbf257e4f4bfa2feddb </div>
<div style="display:none"> Gerrit-Change-Number: 20469 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Patrick Georgi <pgeorgi@google.com> </div>