<p>Tristan Corrick <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://review.coreboot.org/c/coreboot/+/30269">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">nb/intel/haswell: Don't unconditionally set DEVEN<br><br>The existing code sets DEVEN with the intention of enabling the IGD and<br>Mini-HD audio. However, according to the datasheet [1] and some testing<br>on hardware, the bits in DEVEN are set by default if and only if the<br>straps/fuses say the device should be enabled. To illustrate this, here<br>are a few initial values of DEVEN on some Haswell systems:<br><br>    Supermicro X10SLM+-F:  0x0000002d<br>     ASRock H81M-HDS:       0x00000039<br>     Acer C720:             0x000000b1<br><br>On the X10SLM+-F, the IGD is disabled by default, and PEG10 & PEG11<br>are enabled by default. On the C720, the PEG devices are all disabled<br>by default, while the IGD and Mini-HD audio are already enabled.<br><br>There are two issues that result from the existing behaviour: PEG<br>devices are unconditionally disabled, and devices are set as enabled<br>when it's not actually possible to enable them.<br><br>So, don't touch the DEVEN register at this stage, as there are no<br>benefits.<br><br>Interestingly, on an Acer C720 (Google Peppy), a PCI device 00:04.0<br>appears. It is a thermal sensor. `powerstat` was used to measure idle<br>power usage over 30 minutes under Debian GNU/Linux 9.6. There was no<br>change in reported power draw.<br><br>[1] Desktop 4th Generation Intel® Core TM Processor Family, Desktop<br>    Intel® Pentium® Processor Family, and Desktop Intel® Celeron®<br>    Processor Family Datasheet – Volume 2 of 2.<br>    December 2013, revision 003, document number 328898.<br><br>Change-Id: I242f9138472de5a0b26b5852f632b53b2920132d<br>Signed-off-by: Tristan Corrick <tristan@corrick.kiwi><br>---<br>M src/northbridge/intel/haswell/early_init.c<br>1 file changed, 0 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/69/30269/4</pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/30269">change 30269</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/+/30269"/><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: I242f9138472de5a0b26b5852f632b53b2920132d </div>
<div style="display:none"> Gerrit-Change-Number: 30269 </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: 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>