<p>Nico Huber has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21072">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/common/smbus: Don't clear random bits<br><br>FSP might have done some settings for us there. Use pci_update_config32()<br>since the register is documented to be 32 bits wide.<br><br>Change-Id: I995e8a731a6958f10600174d031bb94f5a0a66db<br>Signed-off-by: Nico Huber <nico.huber@secunet.com><br>---<br>M src/soc/intel/common/block/smbus/smbus.c<br>1 file changed, 2 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/72/21072/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/common/block/smbus/smbus.c b/src/soc/intel/common/block/smbus/smbus.c<br>index 8ee38ee..e526baf 100644<br>--- a/src/soc/intel/common/block/smbus/smbus.c<br>+++ b/src/soc/intel/common/block/smbus/smbus.c<br>@@ -53,12 +53,10 @@<br> static void pch_smbus_init(device_t dev)<br> {<br>       struct resource *res;<br>-        u16 reg16;<br> <br>         /* Enable clock gating */<br>-    reg16 = pci_read_config32(dev, 0x80);<br>-        reg16 &= ~((1 << 8)|(1 << 10)|(1 << 12)|(1 << 14));<br>-      pci_write_config32(dev, 0x80, reg16);<br>+        pci_update_config32(dev, 0x80,<br>+               ~((1 << 8) | (1 << 10) | (1 << 12) | (1 << 14)), 0);<br> <br>       /* Set Receive Slave Address */<br>       res = find_resource(dev, PCI_BASE_ADDRESS_4);<br></pre><p>To view, visit <a href="https://review.coreboot.org/21072">change 21072</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/21072"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I995e8a731a6958f10600174d031bb94f5a0a66db </div>
<div style="display:none"> Gerrit-Change-Number: 21072 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Nico Huber <nico.h@gmx.de> </div>