<p>Barnali Sarkar <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://review.coreboot.org/20643">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/common/block: Modify fast_spi_lock_bar function<br><br>Use 16bit write to avoid touching the upper two bytes that may cause<br>write cycle to fail in case a prior transaction has not completed.<br><br>This function sets the WRSDIS(Bit 11) and FLOCKDN (Bit 15) of the<br>SPIBAR + BIOS_HSFSTS_CTL. While WRSDIS is lockable with FLOCKDN,<br>writing both in the same cycle is guaranteed to work by design.<br><br>Avoid read->modify->write operation not to clear the RW1C bits<br>unintentionally.<br><br>Change-Id: Ia7880aaca0ed64150c994d49786a0a008bbaa98b<br>Signed-off-by: Barnali Sarkar <barnali.sarkar@intel.com><br>---<br>M src/soc/intel/common/block/fast_spi/fast_spi.c<br>M src/soc/intel/common/block/fast_spi/fast_spi_def.h<br>2 files changed, 10 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/20643/2</pre><p>To view, visit <a href="https://review.coreboot.org/20643">change 20643</a>. To unsubscribe, 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/20643"/><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: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Ia7880aaca0ed64150c994d49786a0a008bbaa98b </div>
<div style="display:none"> Gerrit-Change-Number: 20643 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Barnali Sarkar <barnali.sarkar@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Balaji Manigandan <balaji.manigandan@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Barnali Sarkar <barnali.sarkar@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>