<p>Martin Roth has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20699">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">src/vendorcode/amd/agesa/f15tn: Fix bitmask<br><br>Fixes GCC 7.1 error:<br>error: '<<' in boolean context, did you mean '<' ?<br><br>Change-Id: I1a28522279982b30d25f1a4a4433a1db767f8a02<br>Signed-off-by: Martin Roth <martinroth@google.com><br>---<br>M src/vendorcode/amd/agesa/f15tn/Proc/Fch/Pcie/GppHp.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/99/20699/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/vendorcode/amd/agesa/f15tn/Proc/Fch/Pcie/GppHp.c b/src/vendorcode/amd/agesa/f15tn/Proc/Fch/Pcie/GppHp.c<br>index b3d0c8b..898ff0c 100644<br>--- a/src/vendorcode/amd/agesa/f15tn/Proc/Fch/Pcie/GppHp.c<br>+++ b/src/vendorcode/amd/agesa/f15tn/Proc/Fch/Pcie/GppHp.c<br>@@ -85,7 +85,7 @@<br> <br>   FailedPort = (UINT8) (1 << HpPort);<br>   if (FchGpp->GppGen2 && FchGpp->GppGen2Strap) {<br>-    GppS3Data &= (UINT8) !(1 << HpPort);<br>+    GppS3Data &= (UINT8) ~(1 << HpPort);<br>     if (GppPortPollingLtssm (FchGpp, FailedPort, TRUE, StdHeader)) {<br>       FchGppForceGen1 (FchGpp, FailedPort, StdHeader);<br>       FailedPort = GppPortPollingLtssm (FchGpp, FailedPort, FALSE, StdHeader);<br>@@ -123,7 +123,7 @@<br>   GppS3Data = 0x00;<br>   ReadMem ( ACPI_MMIO_BASE + CMOS_RAM_BASE + 0x0D, AccessWidth8, &GppS3Data);<br>   FchGpp->PortCfg[HpPort].PortDetected = FALSE;<br>-  GppS3Data &= (UINT8) !(1 << (HpPort + 4));<br>+  GppS3Data &= (UINT8) ~(1 << (HpPort + 4));<br> <br>   if (FchGpp->GppGen2 && FchGpp->GppGen2Strap) {<br>     FchGppForceGen2 (FchGpp, (UINT8) (1 << HpPort), StdHeader);<br></pre><p>To view, visit <a href="https://review.coreboot.org/20699">change 20699</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/20699"/><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: I1a28522279982b30d25f1a4a4433a1db767f8a02 </div>
<div style="display:none"> Gerrit-Change-Number: 20699 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Martin Roth <martinroth@google.com> </div>