<p><a href="https://review.coreboot.org/25105">View Change</a></p><p>7 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c">File src/drivers/spi/winbond.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/25105/18/src/drivers/spi/winbond.c@33">Patch Set #18, Line 33:</a> <code style="font-family:monospace,monospace">#define CMD_VOLATILE_SREG_WREN     0x50    /* Write Enable for Volatile Status Register */</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/25105/18/src/drivers/spi/winbond.c@494">Patch Set #18, Line 494:</a> <code style="font-family:monospace,monospace">                val = (union status_reg1_bp3){ .bp = bp, .tb = tb, .sec = 0 }.u;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@495">Patch Set #18, Line 495:</a> <code style="font-family:monospace,monospace">             mask = (union status_reg1_bp3){ .bp = ~0, .tb = 1, .sec = 1 }.u;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@497">Patch Set #18, Line 497:</a> <code style="font-family:monospace,monospace">             val = (union status_reg1_bp4){ .bp = bp, .tb = tb }.u;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@498">Patch Set #18, Line 498:</a> <code style="font-family:monospace,monospace">               mask = (union status_reg1_bp4){ .bp = ~0, .tb = 1 }.u;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@505">Patch Set #18, Line 505:</a> <code style="font-family:monospace,monospace">       val = (union status_reg2){ .cmp = cmp }.u;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/25105/18/src/drivers/spi/winbond.c@506">Patch Set #18, Line 506:</a> <code style="font-family:monospace,monospace">   mask = (union status_reg2){ .cmp = 1 }.u;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space required after that close brace '}'</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/25105">change 25105</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/25105"/><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: Ie3765b013855538eca37bc7800d3f9d5d09b8402 </div>
<div style="display:none"> Gerrit-Change-Number: 25105 </div>
<div style="display:none"> Gerrit-PatchSet: 18 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com> </div>
<div style="display:none"> Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org> </div>
<div style="display:none"> Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com> </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-CC: Julius Werner <jwerner@chromium.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 20 Sep 2018 11:14:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>