<p>Martin Roth <strong>posted comments</strong> on this change.</p><p><a href="https://review.coreboot.org/20638">View Change</a></p><p>Patch set 2:</p><p>(3 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://review.coreboot.org/#/c/20638/2/src/device/Kconfig">File src/device/Kconfig:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/20638/2/src/device/Kconfig@246">Patch Set #2, Line 246:</a> <code style="font-family:monospace,monospace">depends on BOARD_LENOVO_G505S</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Remove this?  It will make it so that the G505s is the only board that can enable this feature.</p></li><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/20638/2/src/device/Kconfig@247">Patch Set #2, Line 247:</a> <code style="font-family:monospace,monospace">n</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not seeing any reason not to default this to y, or or even get rid of the Kconfig option completely and just have all the code protected by the option always run.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Nico - Thoughts?</p></li><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/20638/2/src/device/Kconfig@251">Patch Set #2, Line 251:</a> <code style="font-family:monospace,monospace">'</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">its - the apostrophe isn't needed.</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: 2 </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: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 18 Jul 2017 22:21:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>