<p><a href="https://review.coreboot.org/c/coreboot/+/11791">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/11791/11/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/11/src/mainboard/lenovo/t410/romstage.c@65">Patch Set #11, 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/11/src/mainboard/lenovo/t410/romstage.c@141">Patch Set #11, 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/11/src/mainboard/lenovo/t410/romstage.c@271">Patch Set #11, 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/11/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/11/src/mainboard/lenovo/t410/smihandler.c@51">Patch Set #11, 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/11/src/mainboard/lenovo/t410/smihandler.c@54">Patch Set #11, 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/11/src/mainboard/lenovo/t410/smihandler.c@80">Patch Set #11, 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/11/src/mainboard/lenovo/t410/smihandler.c@88">Patch Set #11, 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/11/src/mainboard/lenovo/t410/smihandler.c@104">Patch Set #11, 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/11/src/mainboard/lenovo/t410/smihandler.c@141">Patch Set #11, 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/11/src/mainboard/lenovo/t410/smihandler.c@165">Patch Set #11, 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: 11 </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-CC: Peter Lemenkov <lemenkov@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 14 Dec 2018 09:19:56 +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>