<p><a href="https://review.coreboot.org/24992">View Change</a></p><p>30 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h">File src/include/blob_provider.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/24992/11/src/include/blob_provider.h@32">Patch Set #11, Line 32:</a> <code style="font-family:monospace,monospace">       ID_ACPI_SLIC,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@33">Patch Set #11, Line 33:</a> <code style="font-family:monospace,monospace">       ID_ACPI_DSDT,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@34">Patch Set #11, Line 34:</a> <code style="font-family:monospace,monospace">       ID_ACPI_SSDT,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@35">Patch Set #11, Line 35:</a> <code style="font-family:monospace,monospace">       ID_MICROCODE,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@36">Patch Set #11, Line 36:</a> <code style="font-family:monospace,monospace">       ID_MICROCODE_RMU,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@37">Patch Set #11, Line 37:</a> <code style="font-family:monospace,monospace">       ID_MRC_CACHE,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@38">Patch Set #11, Line 38:</a> <code style="font-family:monospace,monospace">       ID_VGA_VBT,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@39">Patch Set #11, Line 39:</a> <code style="font-family:monospace,monospace">       ID_NVRAM_CMOS_LAYOUT,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@40">Patch Set #11, Line 40:</a> <code style="font-family:monospace,monospace">       ID_NVRAM_CMOS_DEFAULT,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@41">Patch Set #11, Line 41:</a> <code style="font-family:monospace,monospace">       ID_NVRAM_VPD,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@42">Patch Set #11, Line 42:</a> <code style="font-family:monospace,monospace">       ID_SPD,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@43">Patch Set #11, Line 43:</a> <code style="font-family:monospace,monospace">       ID_MAC,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@44">Patch Set #11, Line 44:</a> <code style="font-family:monospace,monospace">       ID_AMD_S3NV,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@45">Patch Set #11, Line 45:</a> <code style="font-family:monospace,monospace">       ID_MRC_CACHE_RW_REGION,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@46">Patch Set #11, Line 46:</a> <code style="font-family:monospace,monospace">       ID_NVRAM_VPD_RO_REGION,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@47">Patch Set #11, Line 47:</a> <code style="font-family:monospace,monospace">       ID_AMD_AGESA,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@48">Patch Set #11, Line 48:</a> <code style="font-family:monospace,monospace">       ID_AMD_AGESA_PRE_MEM,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@49">Patch Set #11, Line 49:</a> <code style="font-family:monospace,monospace">       ID_AMD_AGESA_POST_MEM,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@50">Patch Set #11, Line 50:</a> <code style="font-family:monospace,monospace">       ID_AMD_PSP,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@51">Patch Set #11, Line 51:</a> <code style="font-family:monospace,monospace">       ID_INTEL_MRC,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@52">Patch Set #11, Line 52:</a> <code style="font-family:monospace,monospace">       ID_INTEL_FSP_S,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@53">Patch Set #11, Line 53:</a> <code style="font-family:monospace,monospace">       ID_INTEL_FSP_M,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@54">Patch Set #11, Line 54:</a> <code style="font-family:monospace,monospace">       ID_INTEL_MMA,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@55">Patch Set #11, Line 55:</a> <code style="font-family:monospace,monospace">       ID_NVIDIA_MTC,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@56">Patch Set #11, Line 56:</a> <code style="font-family:monospace,monospace">       ID_QUALCOMM_CDT,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@57">Patch Set #11, Line 57:</a> <code style="font-family:monospace,monospace">       ID_QUALCOMM_DDR,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@58">Patch Set #11, Line 58:</a> <code style="font-family:monospace,monospace">       ID_QUALCOMM_TZ,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/include/blob_provider.h@59">Patch Set #11, Line 59:</a> <code style="font-family:monospace,monospace">       ID_QUALCOMM_RPM,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/24992/11/src/lib/blob_provider.c">File src/lib/blob_provider.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/24992/11/src/lib/blob_provider.c@71">Patch Set #11, Line 71:</a> <code style="font-family:monospace,monospace">       if (lookup_rdev(locator, &data) < 0) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">braces {} are not necessary for single statement blocks</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/24992/11/src/lib/blob_provider.c@84">Patch Set #11, Line 84:</a> <code style="font-family:monospace,monospace">      if (lookup_rdev(locator, data) < 0) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">braces {} are not necessary for single statement blocks</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/24992">change 24992</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/24992"/><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: I30abc5be5d47a4c697c0b94babb068b0f0ae4794 </div>
<div style="display:none"> Gerrit-Change-Number: 24992 </div>
<div style="display:none"> Gerrit-PatchSet: 11 </div>
<div style="display:none"> Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net> </div>
<div style="display:none"> Gerrit-Reviewer: Werner Zeh <werner.zeh@siemens.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Aaron Durbin <adurbin@chromium.org> </div>
<div style="display:none"> Gerrit-CC: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-CC: Youness Alaoui <snifikino@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 09 Aug 2018 01:26:17 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>