<p><a href="https://review.coreboot.org/c/coreboot/+/11791">View Change</a></p><p>13 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/mainboard.c">File src/mainboard/lenovo/t410/mainboard.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/11791/7/src/mainboard/lenovo/t410/mainboard.c@42">Patch Set #7, Line 42:</a> <code style="font-family:monospace,monospace">    {1, 1, 1000, {0x7f, 1, 2, {0}, 1, 0}},</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/mainboard.c@43">Patch Set #7, Line 43:</a> <code style="font-family:monospace,monospace">       {2, 1, 500, {0x01, 8, 0, {0}, DEFAULT_PMBASE + LV2, 0}},</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/mainboard.c@44">Patch Set #7, Line 44:</a> <code style="font-family:monospace,monospace">     {2, 17, 250, {0x01, 8, 0, {0}, DEFAULT_PMBASE + LV3, 0}},</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/romstage.c">File src/mainboard/lenovo/t410/romstage.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/11791/7/src/mainboard/lenovo/t410/romstage.c@65">Patch Set #7, Line 65:</a> <code style="font-family:monospace,monospace">                      (pci_read_config8(PCH_LPC_DEV, GEN_PMCON_3) & ~2) | 1);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/romstage.c@141">Patch Set #7, Line 141:</a> <code style="font-family:monospace,monospace"> unsigned i;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/romstage.c@271">Patch Set #7, Line 271:</a> <code style="font-family:monospace,monospace">                             pci_read_config8(PCH_LPC_DEV, GEN_PMCON_3) | 2);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/smihandler.c">File src/mainboard/lenovo/t410/smihandler.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/11791/7/src/mainboard/lenovo/t410/smihandler.c@51">Patch Set #7, Line 51:</a> <code style="font-family:monospace,monospace">       switch (smif) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">switch and case should be at the same indent</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/smihandler.c@54">Patch Set #7, Line 54:</a> <code style="font-family:monospace,monospace">                  udelay(250000);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">long udelay - prefer mdelay; see arch/arm/include/asm/delay.h</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/smihandler.c@80">Patch Set #7, Line 80:</a> <code style="font-family:monospace,monospace"> if ((value = pci_read_config8(PCI_DEV(0, 2, 1), 0xf4)) < 0xf0)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/smihandler.c@88">Patch Set #7, Line 88:</a> <code style="font-family:monospace,monospace">       if ((value = pci_read_config8(PCI_DEV(0, 2, 1), 0xf4)) > 0x10)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/smihandler.c@104">Patch Set #7, Line 104:</a> <code style="font-family:monospace,monospace">     switch (event) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">switch and case should be at the same indent</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/smihandler.c@141">Patch Set #7, Line 141:</a> <code style="font-family:monospace,monospace">       switch (data) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">switch and case should be at the same indent</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/11791/7/src/mainboard/lenovo/t410/smihandler.c@165">Patch Set #7, Line 165:</a> <code style="font-family:monospace,monospace">                        /* we have to use port 0x62/0x66, as 0x1600/0x1604 doesn't</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/11791">change 11791</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/c/coreboot/+/11791"/><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-Change-Id: Id9d872e643dd242e925bfb46d18076e6ad100995 </div>
<div style="display:none"> Gerrit-Change-Number: 11791 </div>
<div style="display:none"> Gerrit-PatchSet: 7 </div>
<div style="display:none"> Gerrit-Owner: Nicolas Reinecke <nr@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Couzens <lynxis@fe80.eu> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nicolas Reinecke <nr@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Eloy </div>
<div style="display:none"> Gerrit-CC: HAOUAS Elyes <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 13 Dec 2018 11:31:30 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>