<p>Barnali Sarkar <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://review.coreboot.org/20956">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Sideband lock skipped in FSP and done in coreboot<br><br>The Sideband Acces locking code is skipped from FSP by setting an<br>FSP-S UPD called PchSbAccessUnlock This locking is being done in<br>coreboot during finalize.c.<br><br>This is done because Coreboot was failing to disable HECI1 device<br>using Sideband interface during finalize.c if FSP already locks<br>the Sideband access mechanism before that.<br><br>So, as a solution, coreboot passes an UPD to skip the locking<br>in FSP, and in finalize.c, after disabling HECI, it removes the<br>Sideband access.<br><br>BUG=b:63877089<br>BRANCH=none<br>TEST=Build and boot poppy to check lspci not showing Intel ME<br>controller in the PCI device list.<br><br>Change-Id: I8dba4c97480200507969b0f2873337f97bd2ff6a<br>Signed-off-by: Barnali Sarkar <barnali.sarkar@intel.com><br>---<br>M src/soc/intel/skylake/chip_fsp20.c<br>M src/soc/intel/skylake/finalize.c<br>2 files changed, 43 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/20956/4</pre><p>To view, visit <a href="https://review.coreboot.org/20956">change 20956</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/20956"/><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: I8dba4c97480200507969b0f2873337f97bd2ff6a </div>
<div style="display:none"> Gerrit-Change-Number: 20956 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </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: Rizwan Qureshi <rizwan.qureshi@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: dhaval v sharma <dhaval.v.sharma@intel.com> </div>