<p><a href="https://review.coreboot.org/26479">View Change</a></p><p>9 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26479/4/src/soc/intel/skylake/chip_fsp20.c">File src/soc/intel/skylake/chip_fsp20.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/26479/4/src/soc/intel/skylake/chip_fsp20.c@73">Patch Set #4, Line 73:</a> <code style="font-family:monospace,monospace">static void pcie_update_device_tree(struct pcie_entry *pcie_rp_group, int pci_groups)</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/26479/4/src/soc/intel/skylake/chip_fsp20.c@93">Patch Set #4, Line 93:</a> <code style="font-family:monospace,monospace">                 * Increase funtion by 1.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">'funtion' may be misspelled - perhaps 'function'?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26479/4/src/soc/intel/skylake/chip_fsp20.c@97">Patch Set #4, Line 97:</a> <code style="font-family:monospace,monospace">               for (i = 1; i < pcie_rp_group[group].func_count; i++, devfn += inc) {</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/26479/4/src/soc/intel/skylake/chip_fsp20.c@102">Patch Set #4, Line 102:</a> <code style="font-family:monospace,monospace">                        /* Found the first enabled device in a given dev number */</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/26479/4/src/soc/intel/skylake/chip_fsp20.c@103">Patch Set #4, Line 103:</a> <code style="font-family:monospace,monospace">                      printk(BIOS_INFO, "PCI func %d was swapped to func 0.\n", i);</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/26479/4/src/soc/intel/skylake/chip_fsp20.c@127">Patch Set #4, Line 127:</a> <code style="font-family:monospace,monospace"> else if (id_mask == (PCI_DEVICE_ID_INTEL_KBP_H_PCIE_RP1 & ~0xf)) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">else should follow 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/26479/4/src/soc/intel/skylake/chip_fsp20.c@132">Patch Set #4, Line 132:</a> <code style="font-family:monospace,monospace">       else if (id_mask == (PCI_DEVICE_ID_INTEL_SPT_H_PCIE_RP1 & ~0xf)) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">else should follow 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/26479/4/src/soc/intel/skylake/chip_fsp20.c@137">Patch Set #4, Line 137:</a> <code style="font-family:monospace,monospace">       else {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">else should follow 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/26479/4/src/soc/intel/skylake/chip_fsp20.c@138">Patch Set #4, Line 138:</a> <code style="font-family:monospace,monospace">               printk(BIOS_ERR, "[BUG] PCIE Root Port id 0x%x is not found\n", id);</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/26479">change 26479</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/26479"/><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: Ibbc5d6e979977011f5904c8bd4b2f1be16bd23dc </div>
<div style="display:none"> Gerrit-Change-Number: 26479 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Gaggery Tsai <gaggery.tsai@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Gaggery Tsai <gaggery.tsai@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kane Chen <kane.chen@intel.com> </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-Comment-Date: Tue, 29 May 2018 20:10:30 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>