<p>Subrata Banik <strong>posted comments</strong> on this change.</p><p><a href="https://review.coreboot.org/20559">View Change</a></p><p>Patch set 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess if we disable SMBUS as UPD policy then from FSP Silicon init phase, this will make SMBUS function disable which mean device will not visible over bus after FSP-S exit.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But if you see inside pmunil.c we need to get access of TCO base from smbus pci config space as its part of SMBUS space unlike BDW where it was part of PMC. I don't think we can disable SMBUS.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://review.coreboot.org/20559">change 20559</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/20559"/><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: I3db28bb22245a2f1d39b4f795a3f438a9df36a2c </div>
<div style="display:none"> Gerrit-Change-Number: 20559 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Naresh Solanki <naresh.solanki@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Duncan Laurie <dlaurie@chromium.org> </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: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 13 Jul 2017 05:04:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>