<p><a href="https://review.coreboot.org/26942">View Change</a></p><p>37 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26942/8/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/8/src/device/oprom/yabel/compat/of.h@38">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@53">Patch Set #8, Line 53:</a> <code style="font-family:monospace,monospace">phandle_t of_peer(phandle_t);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'phandle_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/8/src/device/oprom/yabel/compat/of.h@54">Patch Set #8, Line 54:</a> <code style="font-family:monospace,monospace">phandle_t of_child(phandle_t);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'phandle_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/8/src/device/oprom/yabel/compat/of.h@55">Patch Set #8, Line 55:</a> <code style="font-family:monospace,monospace">phandle_t of_parent(phandle_t);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'phandle_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/8/src/device/oprom/yabel/compat/of.h@56">Patch Set #8, Line 56:</a> <code style="font-family:monospace,monospace">int of_getprop(phandle_t, const char *, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'phandle_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/8/src/device/oprom/yabel/compat/of.h@56">Patch Set #8, Line 56:</a> <code style="font-family:monospace,monospace">int of_getprop(phandle_t, const char *, void *, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'const 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/8/src/device/oprom/yabel/compat/of.h@56">Patch Set #8, Line 56:</a> <code style="font-family:monospace,monospace">int of_getprop(phandle_t, const char *, 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/8/src/device/oprom/yabel/compat/of.h@56">Patch Set #8, Line 56:</a> <code style="font-family:monospace,monospace">int of_getprop(phandle_t, const char *, 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/8/src/device/oprom/yabel/compat/of.h@57">Patch Set #8, Line 57:</a> <code style="font-family:monospace,monospace">void *of_call_method_3(const char *, ihandle_t, int);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'const 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/8/src/device/oprom/yabel/compat/of.h@57">Patch Set #8, Line 57:</a> <code style="font-family:monospace,monospace">void *of_call_method_3(const char *, ihandle_t, 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/8/src/device/oprom/yabel/compat/of.h@57">Patch Set #8, Line 57:</a> <code style="font-family:monospace,monospace">void *of_call_method_3(const char *, ihandle_t, 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/8/src/device/oprom/yabel/compat/of.h@60">Patch Set #8, Line 60:</a> <code style="font-family:monospace,monospace">ihandle_t of_open(const char *);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">function definition argument 'const 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/8/src/device/oprom/yabel/compat/of.h@62">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@62">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@62">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@63">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@63">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@63">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@64">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@64">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@64">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@66">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@66">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@66">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@67">Patch Set #8, Line 67:</a> <code style="font-family:monospace,monospace">void of_release(void *, 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/8/src/device/oprom/yabel/compat/of.h@67">Patch Set #8, Line 67:</a> <code style="font-family:monospace,monospace">void of_release(void *, 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/8/src/device/oprom/yabel/compat/of.h@70">Patch Set #8, Line 70:</a> <code style="font-family:monospace,monospace">void *of_set_callback(void *);</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/8/src/device/oprom/yabel/compat/of.h@72">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@72">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@72">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@73">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@73">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@73">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@74">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@74">Patch Set #8, 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/8/src/device/oprom/yabel/compat/of.h@74">Patch Set #8, 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/8/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/8/src/northbridge/amd/amdht/ht_wrapper.c@58">Patch Set #8, 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></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: 8 </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 11:52:41 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>