<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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<br>about<br>>> SKUs that don't have it at all? are there any (on supported<br>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><p style="white-space: pre-wrap; word-wrap: break-word;">The board I have works without that line.<br>seems that nvidia external GPU didn't work properly (I don't have it for test), But ATI works just fine without that line. More, it works better without that line. please see https://review.coreboot.org/#/c/coreboot/+/27985/ (Arthur's comment on <br>Patch Set 3 )</p></blockquote><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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<br>them?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think so, but as you know, intel's datasheet are not helpful :(<br>maybe some one from intel can help ... </p></blockquote><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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<br>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.<br>here I mean why we define "integrated_graphics" = 1 ?<br>the test do not make sense :<br>if (1) {<br>....<br>}else {<br>..... <<=== this will never happen</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">}</p><p style="white-space: pre-wrap; word-wrap: break-word;">so let give a sense to "integrated_graphics" and set it to 1 if IGD is enabled.</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 17:14:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>