<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20698">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vendorcode/amd/agesa: f15tn & f16kb: fix assert<br><br>Fixes warning by GCC 7.1:<br>note: did you mean to use logical not?<br><br>Change-Id: If8167c6fe88135ae89eb795eeda09e6937b1684f<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M src/vendorcode/amd/agesa/f15tn/Proc/IDS/Library/IdsLib.c<br>M src/vendorcode/amd/agesa/f16kb/Proc/IDS/Library/IdsLib.c<br>2 files 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/98/20698/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/vendorcode/amd/agesa/f15tn/Proc/IDS/Library/IdsLib.c b/src/vendorcode/amd/agesa/f15tn/Proc/IDS/Library/IdsLib.c<br>index b6867c5..34a12a7 100644<br>--- a/src/vendorcode/amd/agesa/f15tn/Proc/IDS/Library/IdsLib.c<br>+++ b/src/vendorcode/amd/agesa/f15tn/Proc/IDS/Library/IdsLib.c<br>@@ -698,7 +698,7 @@<br>   IDS_EARLY_AP_TASK IdsEarlyTask;<br> <br>   IdentifyCore (StdHeader, &BscSocket, &IgnoredModule, &BscCoreNum, &IgnoredSts);<br>-  ASSERT (~((Socket == BscSocket) && (Core == BscCoreNum)));<br>+  ASSERT (!((Socket == BscSocket) && (Core == BscCoreNum)));<br>   if ((Socket == BscSocket) || (Core == 0)) {<br>     ApUtilRunCodeOnSocketCore (Socket, Core, ApTask, StdHeader);<br>   } else {<br>diff --git a/src/vendorcode/amd/agesa/f16kb/Proc/IDS/Library/IdsLib.c b/src/vendorcode/amd/agesa/f16kb/Proc/IDS/Library/IdsLib.c<br>index 2345e1c..0cb0a63 100644<br>--- a/src/vendorcode/amd/agesa/f16kb/Proc/IDS/Library/IdsLib.c<br>+++ b/src/vendorcode/amd/agesa/f16kb/Proc/IDS/Library/IdsLib.c<br>@@ -781,7 +781,7 @@<br>   IDS_EARLY_AP_TASK IdsEarlyTask;<br> <br>   IdentifyCore (StdHeader, &BscSocket, &IgnoredModule, &BscCoreNum, &IgnoredSts);<br>-  ASSERT (~((Socket == BscSocket) && (Core == BscCoreNum)));<br>+  ASSERT (!((Socket == BscSocket) && (Core == BscCoreNum)));<br>   if ((Socket == BscSocket) || (Core == 0)) {<br>     ApUtilRunCodeOnSocketCore (Socket, Core, ApTask, StdHeader);<br>   } else {<br></pre><p>To view, visit <a href="https://review.coreboot.org/20698">change 20698</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/20698"/><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: If8167c6fe88135ae89eb795eeda09e6937b1684f </div>
<div style="display:none"> Gerrit-Change-Number: 20698 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>