<p>Mike Banon <strong>posted comments</strong> on this change.</p><p><a href="https://review.coreboot.org/20638">View Change</a></p><p>Patch set 4:</p><p style="white-space: pre-wrap; word-wrap: break-word;">Thank you for reviewing this option. Maybe it is indeed useless (and even harmful) for discrete GPU support, in which case it should be removed</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://review.coreboot.org/#/c/20638/4//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/20638/4//COMMIT_MSG@12">Patch Set #4, Line 12:</a> <code style="font-family:monospace,monospace">could</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">How? The guarded code limits the selected links to those that</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Currently, enabling this option doesn't make my discrete GPU working... but maybe there is a bug that I can't see, and if someone more knowledgeable than me will fix this bug - my discrete GPU could become working. To attract more attention to this option I hoped to make it possible to enable it from menuconfig</p><p style="white-space: pre-wrap; word-wrap: break-word;">But if you are sure that enabling this option is actually preventing the discrete GPUs from being able to work, then this option with misleading name should be removed from coreboot - together with its' guarded code</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/20638">change 20638</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/20638"/><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: Iade58d3c373e0bc30d4642d9417c4e5224249136 </div>
<div style="display:none"> Gerrit-Change-Number: 20638 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Mike Banon <mikebdp2@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Mike Banon <mikebdp2@gmail.com> </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-Comment-Date: Wed, 19 Jul 2017 08:26:03 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>