<p style="white-space: pre-wrap; word-wrap: break-word;">>> I don't see a way IGD could be disabled at this stage. But what about<br>>> SKUs that don't have it at all? are there any (on supported boards)?<br>> <br>> 1 - In case we use an external GPU, the IGD is disabled at<br>> arly_init.c (line #686).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Which is run after raminit.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">(BTW: line +949 should be commented)</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It doesn't work without that line. You probably mean 950 should<br>be implemented for the desktop version. Please, go ahead.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">2 - there some desktop's version that do not have IGD at all.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">That's why I was asking. Are the DEVEN bits hard-wired to 0 for them?</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">3 - at function "sdram_power_management", we set integrated_graphics<br>= 1 and use it for test in raminit.c line #2305 .... this do not<br>make sense.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It documents for the human reader that it should only be executed<br>when integrated graphics are present (or maybe only when enabled;<br>that's not clear). This can be useful, for instance, when somebody<br>wants to implement it for system without IGD.</p><p><a href="https://review.coreboot.org/28935">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://review.coreboot.org/28935">change 28935</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/28935"/><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: I51ab94393710ce0222b353ab0cef28621fafaacf </div>
<div style="display:none"> Gerrit-Change-Number: 28935 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Angel Pons <th3fanbus@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 05 Oct 2018 16:22:40 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>