<p><a href="https://review.coreboot.org/25739">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/25739/16/src/include/fit.h">File src/include/fit.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/25739/16/src/include/fit.h@28">Patch Set #16, Line 28:</a> <code style="font-family:monospace,monospace">{</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">open brace '{' following enum go on the same line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25739/16/src/include/fit.h@36">Patch Set #16, Line 36:</a> <code style="font-family:monospace,monospace">{</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">open brace '{' following struct go on the same line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25739/16/src/include/fit.h@46">Patch Set #16, Line 46:</a> <code style="font-family:monospace,monospace">{</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">open brace '{' following struct go on the same line</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/25739/16/src/lib/fit.c">File src/lib/fit.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/25739/16/src/lib/fit.c@194">Patch Set #16, Line 194:</a> <code style="font-family:monospace,monospace">      const char *path[] = { "chosen", NULL };</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">char * array declaration might be better as static const</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25739/16/src/lib/fit.c@202">Patch Set #16, Line 202:</a> <code style="font-family:monospace,monospace">        const char *path[] = { "chosen", NULL };</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">char * array declaration might be better as static const</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25739/16/src/lib/fit.c@225">Patch Set #16, Line 225:</a> <code style="font-family:monospace,monospace">{</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">open brace '{' following struct go on the same line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25739/16/src/lib/fit.c@226">Patch Set #16, Line 226:</a> <code style="font-family:monospace,monospace">      unsigned addr_cells;</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/25739/16/src/lib/fit.c@227">Patch Set #16, Line 227:</a> <code style="font-family:monospace,monospace">       unsigned size_cells;</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/25739/16/src/lib/fit.c@231">Patch Set #16, Line 231:</a> <code style="font-family:monospace,monospace">static uint64_t max_range(unsigned size_cells)</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/25739/16/src/lib/fit.c@242">Patch Set #16, Line 242:</a> <code style="font-family:monospace,monospace">    unsigned *count = (unsigned *)params->data;</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/25739/16/src/lib/fit.c@242">Patch Set #16, Line 242:</a> <code style="font-family:monospace,monospace"> unsigned *count = (unsigned *)params->data;</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/25739/16/src/lib/fit.c@298">Patch Set #16, Line 298:</a> <code style="font-family:monospace,monospace">          * Kernel likes its availabe memory areas at least 1MB</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">'availabe' may be misspelled - perhaps 'available'?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25739/16/src/lib/fit.c@323">Patch Set #16, Line 323:</a> <code style="font-family:monospace,monospace"> unsigned count = 0;</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/25739">change 25739</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/25739"/><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: Iee56db328d7eeffb0eaf829841243b0b9195c199 </div>
<div style="display:none"> Gerrit-Change-Number: 25739 </div>
<div style="display:none"> Gerrit-PatchSet: 16 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> </div>
<div style="display:none"> Gerrit-Reviewer: Julius Werner <jwerner@chromium.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-Comment-Date: Tue, 19 Jun 2018 11:52:55 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>