<p><a href="https://review.coreboot.org/26530">View Change</a></p><p>24 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/azalia.c">File src/southbridge/intel/bd82x6x/azalia.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/26530/1/src/southbridge/intel/bd82x6x/azalia.c@340">Patch Set #1, Line 340:</a> <code style="font-family:monospace,monospace">static void azalia_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/azalia.c@340">Patch Set #1, Line 340:</a> <code style="font-family:monospace,monospace">static void azalia_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/azalia.c@340">Patch Set #1, Line 340:</a> <code style="font-family:monospace,monospace">static void azalia_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/lpc.c">File src/southbridge/intel/bd82x6x/lpc.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/26530/1/src/southbridge/intel/bd82x6x/lpc.c@643">Patch Set #1, Line 643:</a> <code style="font-family:monospace,monospace">static void set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/lpc.c@643">Patch Set #1, Line 643:</a> <code style="font-family:monospace,monospace">static void set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/lpc.c@687">Patch Set #1, Line 687:</a> <code style="font-family:monospace,monospace">  struct device *dev = dev_find_slot(0, PCI_DEVFN(0x1f,0));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that ',' (ctx:VxV)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/lpc.c@824">Patch Set #1, Line 824:</a> <code style="font-family:monospace,monospace">      struct device *dev = dev_find_slot(0, PCI_DEVFN(0x1f,0));</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that ',' (ctx:VxV)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/pci.c">File src/southbridge/intel/bd82x6x/pci.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/26530/1/src/southbridge/intel/bd82x6x/pci.c@103">Patch Set #1, Line 103:</a> <code style="font-family:monospace,monospace">static void set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/pci.c@103">Patch Set #1, Line 103:</a> <code style="font-family:monospace,monospace">static void set_subsystem(struct device *dev, unsigned vendor, unsigned device)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/pcie.c">File src/southbridge/intel/bd82x6x/pcie.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/26530/1/src/southbridge/intel/bd82x6x/pcie.c@309">Patch Set #1, Line 309:</a> <code style="font-family:monospace,monospace">static void pcie_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/pcie.c@309">Patch Set #1, Line 309:</a> <code style="font-family:monospace,monospace">static void pcie_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/pcie.c@309">Patch Set #1, Line 309:</a> <code style="font-family:monospace,monospace">static void pcie_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/sata.c">File src/southbridge/intel/bd82x6x/sata.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/26530/1/src/southbridge/intel/bd82x6x/sata.c@236">Patch Set #1, Line 236:</a> <code style="font-family:monospace,monospace">static void sata_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/sata.c@236">Patch Set #1, Line 236:</a> <code style="font-family:monospace,monospace">static void sata_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/sata.c@236">Patch Set #1, Line 236:</a> <code style="font-family:monospace,monospace">static void sata_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/smbus.c">File src/southbridge/intel/bd82x6x/smbus.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/26530/1/src/southbridge/intel/bd82x6x/smbus.c@75">Patch Set #1, Line 75:</a> <code style="font-family:monospace,monospace">static void smbus_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/smbus.c@75">Patch Set #1, Line 75:</a> <code style="font-family:monospace,monospace">static void smbus_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/smbus.c@75">Patch Set #1, Line 75:</a> <code style="font-family:monospace,monospace">static void smbus_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/usb_ehci.c">File src/southbridge/intel/bd82x6x/usb_ehci.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/26530/1/src/southbridge/intel/bd82x6x/usb_ehci.c@77">Patch Set #1, Line 77:</a> <code style="font-family:monospace,monospace">static void usb_ehci_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/usb_ehci.c@77">Patch Set #1, Line 77:</a> <code style="font-family:monospace,monospace">static void usb_ehci_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/usb_ehci.c@77">Patch Set #1, Line 77:</a> <code style="font-family:monospace,monospace">static void usb_ehci_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26530/1/src/southbridge/intel/bd82x6x/usb_xhci.c">File src/southbridge/intel/bd82x6x/usb_xhci.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/26530/1/src/southbridge/intel/bd82x6x/usb_xhci.c@59">Patch Set #1, Line 59:</a> <code style="font-family:monospace,monospace">static void xhci_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/usb_xhci.c@59">Patch Set #1, Line 59:</a> <code style="font-family:monospace,monospace">static void xhci_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</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/26530/1/src/southbridge/intel/bd82x6x/usb_xhci.c@59">Patch Set #1, Line 59:</a> <code style="font-family:monospace,monospace">static void xhci_set_subsystem(struct device *dev, unsigned vendor, unsigned device)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Prefer 'unsigned int' to bare use of 'unsigned'</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/26530">change 26530</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/26530"/><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: I05f23504148d934109814b8f3c1c2a334366496a </div>
<div style="display:none"> Gerrit-Change-Number: 26530 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-CC: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 25 May 2018 06:36:27 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>