<p><a href="https://review.coreboot.org/c/coreboot/+/30109">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/30109/1/src/drivers/emmc/bh7720/bh720.c">File src/drivers/emmc/bh7720/bh720.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/30109/1/src/drivers/emmc/bh7720/bh720.c@97">Patch Set #1, Line 97:</a> <code style="font-family:monospace,monospace">              if(flag) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required before the open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/30109/1/src/drivers/emmc/bh7720/bh720.c@97">Patch Set #1, Line 97:</a> <code style="font-family:monospace,monospace">            if(flag) {</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/30109/1/src/drivers/emmc/bh7720/bh720.c@108">Patch Set #1, Line 108:</a> <code style="font-family:monospace,monospace">         if (mode & MODE_DIS_SD) {</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/30109/1/src/drivers/emmc/bh7720/bh720.c@110">Patch Set #1, Line 110:</a> <code style="font-family:monospace,monospace">              if (mode & MODE_VCCQ_18) {</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/30109/1/src/drivers/emmc/bh7720/bh720.c@112">Patch Set #1, Line 112:</a> <code style="font-family:monospace,monospace">             if (mode & MODE_POWER_SAVE) {</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/30109/1/src/drivers/emmc/bh7720/bh720.c@114">Patch Set #1, Line 114:</a> <code style="font-family:monospace,monospace">          if (mode & MODE_L1_TIMER) {</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/30109/1/src/drivers/emmc/bh7720/bh720.c@116">Patch Set #1, Line 116:</a> <code style="font-family:monospace,monospace">            if (mode & MODE_IDDQ) {</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/30109/1/src/drivers/emmc/bh7720/bh720.c@118">Patch Set #1, Line 118:</a> <code style="font-family:monospace,monospace">                if (mode & MODE_TUNNING) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">'TUNNING' may be misspelled - perhaps 'TUNING'?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/30109/1/src/drivers/emmc/bh7720/bh720.c@118">Patch Set #1, Line 118:</a> <code style="font-family:monospace,monospace">             if (mode & MODE_TUNNING) {</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/30109/1/src/drivers/emmc/bh7720/bh720.c@120">Patch Set #1, Line 120:</a> <code style="font-family:monospace,monospace">             if (mode & MODE_DLL_CLK_PHASE) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">braces {} are not necessary for single statement blocks</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/30109/1/src/include/device/emmc/bh720.h">File src/include/device/emmc/bh720.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/30109/1/src/include/device/emmc/bh720.h@35">Patch Set #1, Line 35:</a> <code style="font-family:monospace,monospace">#define MODE_TUNNING              BIT(6)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">'TUNNING' may be misspelled - perhaps 'TUNING'?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/30109/1/src/include/device/emmc/bh720.h@43">Patch Set #1, Line 43:</a> <code style="font-family:monospace,monospace"> *      adress.</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">'adress' may be misspelled - perhaps 'address'?</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/30109/1/src/include/device/emmc/bh720.h@48">Patch Set #1, Line 48:</a> <code style="font-family:monospace,monospace"> * Note 2: MODE_DIS_SD, MODE_VCCQ_18, MODE_POWER_SAVE, MODE_IDDQ, MODE_TUNNING,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">'TUNNING' may be misspelled - perhaps 'TUNING'?</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30109">change 30109</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/c/coreboot/+/30109"/><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-Change-Id: Ifd2a307ead37c62abca811654873c43c02ac2a4a </div>
<div style="display:none"> Gerrit-Change-Number: 30109 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>
<div style="display:none"> Gerrit-CC: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 07 Dec 2018 21:27:50 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>