<p><a href="https://review.coreboot.org/27221">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27221/1/src/mainboard/google/poppy/variants/nocturne/gpio.c">File src/mainboard/google/poppy/variants/nocturne/gpio.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27221/1/src/mainboard/google/poppy/variants/nocturne/gpio.c@138">Patch Set #1, Line 138:</a> <code style="font-family:monospace,monospace">NONE</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">If signal is active low, why did you change this definition from "_INVERT" ?  IF it's a double-mappi […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I've talked with Duncan and Aaron about this, and they have better explained to me how and where the various definitions come into play, and based on their input, your change with regard to removing "_INVERT" should be fine, please ignore my last comment.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/27221">change 27221</a>. To unsubscribe, or for help writing mail filters, 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/27221"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: I56c13c797b133dd22669a2299bcd16ef14eed335 </div>
<div style="display:none"> Gerrit-Change-Number: 27221 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vincent Palatin <vpalatin@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Vincent Palatin <vpalatin@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Nick Vaccaro <nvaccaro@google.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 27 Jun 2018 18:35:55 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>