<p>Iru Cai has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21013">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">kbc1126_ec_dump: fix signed integer error<br><br>The error is found when using kbc1126_ec_dump on coreboot.rom which<br>uses different EC firmware offsets from vendor firmware.<br><br>Change-Id: Iacc4ea50515ea219bc1868dae0446d576e8dce0c<br>Signed-off-by: Iru Cai <mytbk920423@gmail.com><br>---<br>M util/kbc1126/kbc1126_ec_dump.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/13/21013/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/util/kbc1126/kbc1126_ec_dump.c b/util/kbc1126/kbc1126_ec_dump.c<br>index 7470012..cfa4b64 100644<br>--- a/util/kbc1126/kbc1126_ec_dump.c<br>+++ b/util/kbc1126/kbc1126_ec_dump.c<br>@@ -111,8 +111,8 @@<br>   assert(offs[1] + offs[3] == '\xff');<br>  assert(offs[4] + offs[6] == '\xff');<br>  assert(offs[5] + offs[7] == '\xff');<br>- long offw1 = (offs[0] << 16) | (offs[1] << 8);<br>-   long offw2 = (offs[4] << 16) | (offs[5] << 8);<br>+   long offw1 = ((offs[0] << 16) & 0xff0000) | ((offs[1] << 8) & 0xff00);<br>+   long offw2 = ((offs[4] << 16) & 0xff0000) | ((offs[5] << 8) & 0xff00);<br> <br>         dump_fw(fw1, fp, offw1);<br>      dump_fw(fw2, fp, offw2);<br></pre><p>To view, visit <a href="https://review.coreboot.org/21013">change 21013</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/21013"/><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: Iacc4ea50515ea219bc1868dae0446d576e8dce0c </div>
<div style="display:none"> Gerrit-Change-Number: 21013 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Iru Cai <mytbk920423@gmail.com> </div>