<p><a href="https://review.coreboot.org/29470">View Change</a></p><p>10 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h">File src/mainboard/portwell/m107/irqroute.h:</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/29470/2/src/mainboard/portwell/m107/irqroute.h@23">Patch Set #2, Line 23:</a> <code style="font-family:monospace,monospace">      IR02h GFX       INT(A)  - PIRQ A</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h@25">Patch Set #2, Line 25:</a> <code style="font-family:monospace,monospace"> IR11h SDIO      INT(A)  - PIRQ B</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h@26">Patch Set #2, Line 26:</a> <code style="font-family:monospace,monospace"> IR12h SD        INT(A)  - PIRQ C</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h@27">Patch Set #2, Line 27:</a> <code style="font-family:monospace,monospace"> IR13h SATA      INT(A)  - PIRQ D</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h@28">Patch Set #2, Line 28:</a> <code style="font-family:monospace,monospace"> IR14h XHCI      INT(A)  - PIRQ E</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h@29">Patch Set #2, Line 29:</a> <code style="font-family:monospace,monospace"> IR15h LP Audio  INT(A)  - PIRQ F</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h@30">Patch Set #2, Line 30:</a> <code style="font-family:monospace,monospace"> IR17h MMC       INT(A)  - PIRQ F</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no space before tabs</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/irqroute.h@39">Patch Set #2, Line 39:</a> <code style="font-family:monospace,monospace">#define PCI_DEV_PIRQ_ROUTES \</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Macros with complex values should be enclosed in parentheses</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/spd/SAMSUNG_K4B8G1646D-MYKO.spd.hex">File src/mainboard/portwell/m107/spd/SAMSUNG_K4B8G1646D-MYKO.spd.hex:</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/29470/2/src/mainboard/portwell/m107/spd/SAMSUNG_K4B8G1646D-MYKO.spd.hex@187">Patch Set #2, Line 187:</a> <code style="font-family:monospace,monospace">00 </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">trailing whitespace</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29470/2/src/mainboard/portwell/m107/spd/SAMSUNG_K4B8G1646D-MYKO.spd.hex@218">Patch Set #2, Line 218:</a> <code style="font-family:monospace,monospace">00 00 00 00 </code></p><p style="white-space: pre-wrap; word-wrap: break-word;">trailing whitespace</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/29470">change 29470</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/29470"/><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: I7d3173fdcf881f894a75cd9798ba173b425d4e62 </div>
<div style="display:none"> Gerrit-Change-Number: 29470 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Frans Hendriks <fhendriks@eltan.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 05 Nov 2018 13:14:58 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>