<p>Tristan Corrick <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30270">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">nb/intel/haswell: Use DEVEN to disable devices<br><br>This allows devices to be properly disabled when they are set to `off`<br>in the devicetree, or when a device has its `enabled` property set to<br>false.<br><br>A message is printed stating that a device is being disabled, even if<br>it was already disabled via DEVEN. However, it could be useful to have<br>this information, so such messages are kept.<br><br>The device 00:04.0 is a thermal sensor on the Acer C720, but it has not<br>been named as such in this patch. This is because the public datasheets<br>never formally acknowledge what the device is, and how it might differ<br>across platforms.<br><br>Tested on a Supermicro X10SLM+-F. The Mini-HD audio is disabled now,<br>silencing a warning from Linux.<br><br>Also tested on an Acer C720 (Google Peppy). Disabling "device 4" from<br>devicetree.cb works.<br><br>Also tested on an ASRock H81M-HDS. For this device, and all other test<br>devices, there were no regressions observed.<br><br>Change-Id: If1504e620967449a09f113a7c771a1ec30380644<br>Signed-off-by: Tristan Corrick <tristan@corrick.kiwi><br>---<br>M src/northbridge/intel/haswell/northbridge.c<br>1 file changed, 46 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/30270/4</pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30270">change 30270</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/c/coreboot/+/30270"/><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-Change-Id: If1504e620967449a09f113a7c771a1ec30380644 </div>
<div style="display:none"> Gerrit-Change-Number: 30270 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Tristan Corrick <tristan@corrick.kiwi> </div>
<div style="display:none"> Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Tristan Corrick <tristan@corrick.kiwi> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>