<p>Nico Huber <strong>posted comments</strong> on this change.</p><p><a href="https://review.coreboot.org/22523">View Change</a></p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't get it, doesn't this mean you have to hit the key 99 times to<br>get from lowest to highest brightness level?</p><p style="white-space: pre-wrap; word-wrap: break-word;">I guess the old values are from a random dump. But they don't look bad<br>(for defaults). And having more than (let's say) 15 steps is pretty<br>weird, IMHO. OTOH, it makes sense to have the table per device. Some-<br>times one step is barely noticeable with one panel while it feels to<br>big with another.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://review.coreboot.org/22523">change 22523</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/22523"/><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: I3b7a693a163adeceb3623c2703f3942328679ac4 </div>
<div style="display:none"> Gerrit-Change-Number: 22523 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Matt DeVillier <matt.devillier@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Sun, 19 Nov 2017 22:43:39 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>