<p><a href="https://review.coreboot.org/26461">View Change</a></p><p>23 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26461/3/src/drivers/amd/agesa/eventlog.c">File src/drivers/amd/agesa/eventlog.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/26461/3/src/drivers/amd/agesa/eventlog.c@89">Patch Set #3, Line 89:</a> <code style="font-family:monospace,monospace">        const char *statusStrings[] = { "AGESA_SUCCESS", "AGESA_UNSUPPORTED",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">char * array declaration might be better as static const</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26461/3/src/northbridge/amd/amdht/h3finit.h">File src/northbridge/amd/amdht/h3finit.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/26461/3/src/northbridge/amd/amdht/h3finit.h@229">Patch Set #3, Line 229:</a> <code style="font-family:monospace,monospace">       *      @param[in,out] u8 *LinkWidthIn  = modify to change the Link Witdh In</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/26461/3/src/northbridge/amd/amdht/h3finit.h@230">Patch Set #3, Line 230:</a> <code style="font-family:monospace,monospace">    *      @param[in,out] u8 *LinkWidthOut  = modify to change the Link Witdh Out</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/26461/3/src/northbridge/amd/amdht/h3finit.h@231">Patch Set #3, Line 231:</a> <code style="font-family:monospace,monospace">  *      @param[in,out] u32 *FreqCap = modify to change the link's frequency capability</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/26461/3/src/northbridge/amd/amdht/h3finit.h@232">Patch Set #3, Line 232:</a> <code style="font-family:monospace,monospace">      *      @param[in,out] u32 *FeatureCap = modify to change the link's feature capability</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/26461/3/src/northbridge/amd/amdht/h3finit.h@269">Patch Set #3, Line 269:</a> <code style="font-family:monospace,monospace">     *      @param[in,out]  u8 *ABLinkWidthLimit = modify to change the Link Witdh In</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/26461/3/src/northbridge/amd/amdht/h3finit.h@270">Patch Set #3, Line 270:</a> <code style="font-family:monospace,monospace">       *      @param[in,out]  u8 *BALinkWidthLimit = modify to change the Link Witdh Out</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/26461/3/src/northbridge/amd/amdht/h3finit.h@271">Patch Set #3, Line 271:</a> <code style="font-family:monospace,monospace">      *      @param[in,out]  u32 *PCBFreqCap  = modify to change the link's frequency capability</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/26461/3/src/northbridge/amd/amdht/h3finit.h@302">Patch Set #3, Line 302:</a> <code style="font-family:monospace,monospace">         *      @param[in,out]  u8 *DownstreamLinkWidthLimit = modify to change the Link Witdh In</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/26461/3/src/northbridge/amd/amdht/h3finit.h@303">Patch Set #3, Line 303:</a> <code style="font-family:monospace,monospace">       *      @param[in,out]  u8 *UpstreamLinkWidthLimit  = modify to change the Link Witdh Out</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/26461/3/src/northbridge/amd/amdht/h3finit.h@304">Patch Set #3, Line 304:</a> <code style="font-family:monospace,monospace">       *      @param[in,out]  u32 *PCBFreqCap = modify to change the link's frequency capability</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/26461/3/src/northbridge/amd/amdht/h3finit.h@403">Patch Set #3, Line 403:</a> <code style="font-family:monospace,monospace">  *      @param[in,out]  u8 *LinkWidthIn    = modify to change the Link Witdh In</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/26461/3/src/northbridge/amd/amdht/h3finit.h@404">Patch Set #3, Line 404:</a> <code style="font-family:monospace,monospace">         *      @param[in,out]  u8 *LinkWidthOut   = modify to change the Link Witdh Out</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/26461/3/src/northbridge/amd/amdht/h3finit.h@405">Patch Set #3, Line 405:</a> <code style="font-family:monospace,monospace">        *      @param[in,out]  u16 *LinkFrequency = modify to change the link's frequency capability</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/26461/3/src/northbridge/amd/amdht/h3finit.h@434">Patch Set #3, Line 434:</a> <code style="font-family:monospace,monospace">       *      @param[in,out]  u8 *LinkWidthIn = modify to change the Link Witdh In</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/26461/3/src/northbridge/amd/amdht/h3finit.h@435">Patch Set #3, Line 435:</a> <code style="font-family:monospace,monospace">    *      @param[in,out]  u8 *LinkWidthOut = modify to change the Link Witdh Out</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/26461/3/src/northbridge/amd/amdht/h3finit.h@436">Patch Set #3, Line 436:</a> <code style="font-family:monospace,monospace">  *      @param[in,out]  u16 *LinkFrequency  = modify to change the link's frequency capability</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/26461/3/src/northbridge/amd/amdht/h3finit.h@462">Patch Set #3, Line 462:</a> <code style="font-family:monospace,monospace">      *      @param[in]  u8 *pEventData0 = useful data associated with the event.</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/26461/3/src/northbridge/amd/pi/agesawrapper_call.h">File src/northbridge/amd/pi/agesawrapper_call.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/26461/3/src/northbridge/amd/pi/agesawrapper_call.h@35">Patch Set #3, Line 35:</a> <code style="font-family:monospace,monospace"> const char *statusStrings[] = { "AGESA_SUCCESS", "AGESA_UNSUPPORTED",</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">char * array declaration might be better as static const</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/26461/3/src/northbridge/intel/fsp_sandybridge/raminit.c">File src/northbridge/intel/fsp_sandybridge/raminit.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/26461/3/src/northbridge/intel/fsp_sandybridge/raminit.c@24">Patch Set #3, Line 24:</a> <code style="font-family:monospace,monospace">static const char *ecc_decoder[] = {</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/26461/3/src/northbridge/intel/haswell/raminit.c">File src/northbridge/intel/haswell/raminit.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/26461/3/src/northbridge/intel/haswell/raminit.c@65">Patch Set #3, Line 65:</a> <code style="font-family:monospace,monospace">static const char *ecc_decoder[] = {</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/26461/3/src/northbridge/intel/sandybridge/raminit_mrc.c">File src/northbridge/intel/sandybridge/raminit_mrc.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/26461/3/src/northbridge/intel/sandybridge/raminit_mrc.c@133">Patch Set #3, Line 133:</a> <code style="font-family:monospace,monospace">static const char *ecc_decoder[] = {</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/26461/3/src/southbridge/amd/sb700/sm.c">File src/southbridge/amd/sb700/sm.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/26461/3/src/southbridge/amd/sb700/sm.c@51">Patch Set #3, Line 51:</a> <code style="font-family:monospace,monospace">static const char *power_mode_names[] = {</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/26461">change 26461</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/26461"/><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: Ib2bb6cc80ac2bdc389c60c7ffac4bba937f0fca8 </div>
<div style="display:none"> Gerrit-Change-Number: 26461 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </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: Wed, 30 May 2018 07:36:25 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>