<p><a href="https://review.coreboot.org/26942">View Change</a></p><p>29 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h">File src/device/oprom/yabel/compat/of.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/26942/5/src/device/oprom/yabel/compat/of.h@38">Patch Set #5, Line 38:</a> <code style="font-family:monospace,monospace">#define p32cast (int) (unsigned long) (void *)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Macros with complex values should be enclosed in parentheses</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@62">Patch Set #5, Line 62:</a> <code style="font-family:monospace,monospace">int of_read(ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited before that ',' (ctx:WxW)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@62">Patch Set #5, Line 62:</a> <code style="font-family:monospace,monospace">int of_read(ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'ihandle_t' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@62">Patch Set #5, Line 62:</a> <code style="font-family:monospace,monospace">int of_read(ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'void *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@62">Patch Set #5, Line 62:</a> <code style="font-family:monospace,monospace">int of_read(ihandle_t , void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@63">Patch Set #5, Line 63:</a> <code style="font-family:monospace,monospace">int of_write(ihandle_t, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'ihandle_t' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@63">Patch Set #5, Line 63:</a> <code style="font-family:monospace,monospace">int of_write(ihandle_t, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'void *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@63">Patch Set #5, Line 63:</a> <code style="font-family:monospace,monospace">int of_write(ihandle_t, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@64">Patch Set #5, Line 64:</a> <code style="font-family:monospace,monospace">int of_seek(ihandle_t, int, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'ihandle_t' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@64">Patch Set #5, Line 64:</a> <code style="font-family:monospace,monospace">int of_seek(ihandle_t, int, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@64">Patch Set #5, Line 64:</a> <code style="font-family:monospace,monospace">int of_seek(ihandle_t, int, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@66">Patch Set #5, Line 66:</a> <code style="font-family:monospace,monospace">void * of_claim(void *, unsigned int, unsigned int );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo * bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@66">Patch Set #5, Line 66:</a> <code style="font-family:monospace,monospace">void * of_claim(void *, unsigned int, unsigned int );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited before that close parenthesis ')'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@66">Patch Set #5, Line 66:</a> <code style="font-family:monospace,monospace">void * of_claim(void *, unsigned int, unsigned int );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'void *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@66">Patch Set #5, Line 66:</a> <code style="font-family:monospace,monospace">void * of_claim(void *, unsigned int, unsigned int );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@66">Patch Set #5, Line 66:</a> <code style="font-family:monospace,monospace">void * of_claim(void *, unsigned int, unsigned int );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@72">Patch Set #5, Line 72:</a> <code style="font-family:monospace,monospace">int vpd_read(unsigned int, unsigned int, char *);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@72">Patch Set #5, Line 72:</a> <code style="font-family:monospace,monospace">int vpd_read(unsigned int, unsigned int, char *);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@72">Patch Set #5, Line 72:</a> <code style="font-family:monospace,monospace">int vpd_read(unsigned int, unsigned int, char *);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'char *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@73">Patch Set #5, Line 73:</a> <code style="font-family:monospace,monospace">int vpd_write(unsigned int, unsigned int, char *);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@73">Patch Set #5, Line 73:</a> <code style="font-family:monospace,monospace">int vpd_write(unsigned int, unsigned int, char *);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@73">Patch Set #5, Line 73:</a> <code style="font-family:monospace,monospace">int vpd_write(unsigned int, unsigned int, char *);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'char *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@74">Patch Set #5, Line 74:</a> <code style="font-family:monospace,monospace">int write_mm_log(char *, unsigned int, unsigned short );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited before that close parenthesis ')'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@74">Patch Set #5, Line 74:</a> <code style="font-family:monospace,monospace">int write_mm_log(char *, unsigned int, unsigned short );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'char *' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@74">Patch Set #5, Line 74:</a> <code style="font-family:monospace,monospace">int write_mm_log(char *, unsigned int, unsigned short );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned int' should also have an identifier name</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/26942/5/src/device/oprom/yabel/compat/of.h@74">Patch Set #5, Line 74:</a> <code style="font-family:monospace,monospace">int write_mm_log(char *, unsigned int, unsigned short );</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'unsigned short' should also have an identifier name</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/5/src/northbridge/amd/amdht/ht_wrapper.c">File src/northbridge/amd/amdht/ht_wrapper.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/26942/5/src/northbridge/amd/amdht/ht_wrapper.c@58">Patch Set #5, Line 58:</a> <code style="font-family:monospace,monospace">static const char *event_class_string_decodes[] = {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">static const char * array should probably be static const char * const</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/5/src/southbridge/amd/cimx/sb800/late.c">File src/southbridge/amd/cimx/sb800/late.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/26942/5/src/southbridge/amd/cimx/sb800/late.c@46">Patch Set #5, Line 46:</a> <code style="font-family:monospace,monospace"> * prototype UINT32 (*SBCIM_HOOK_ENTRY)(UINT32 Param1, UINT32 Param2, void *pConfig)</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/26942/5/src/southbridge/amd/cimx/sb900/late.c">File src/southbridge/amd/cimx/sb900/late.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/26942/5/src/southbridge/amd/cimx/sb900/late.c@41">Patch Set #5, Line 41:</a> <code style="font-family:monospace,monospace"> * prototype UINT32 (*SBCIM_HOOK_ENTRY)(UINT32 Param1, UINT32 Param2, void * pConfig)</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/26942">change 26942</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/26942"/><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: I8e4118c5c5d70719ad7dc5f9ff9f86d93fa498ac </div>
<div style="display:none"> Gerrit-Change-Number: 26942 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 07 Jun 2018 10:02:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>