<p>Matt DeVillier has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22523">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">drivers/intel/gma: increase # of brightness steps<br><br>Increase granularity/number of steps across adjustable brightness<br>range (1-100), to provide for smoother transitions when adjusting.<br>Default levels unchanged, just converted to hex format as values<br>were copied from an ACPI dump of a vendor/UEFI firmware.<br><br>TEST: boot Windows on google/wolf, observe smooth transition when<br>adjusting brightness via keyboard controls.<br><br>Change-Id: I3b7a693a163adeceb3623c2703f3942328679ac4<br>Signed-off-by: Matt DeVillier <matt.devillier@gmail.com><br>---<br>M src/drivers/intel/gma/acpi/default_brightness_levels.asl<br>1 file changed, 103 insertions(+), 19 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/23/22523/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/drivers/intel/gma/acpi/default_brightness_levels.asl b/src/drivers/intel/gma/acpi/default_brightness_levels.asl<br>index 6c6f35e..8f560ff 100644<br>--- a/src/drivers/intel/gma/acpi/default_brightness_levels.asl<br>+++ b/src/drivers/intel/gma/acpi/default_brightness_levels.asl<br>@@ -1,24 +1,108 @@<br> Scope (GFX0)<br> {<br>- Name (BRIG, Package (0x12)<br>+ Name (BRIG, Package (0x66)<br> {<br>- 100, /* default AC */<br>- 100, /* default Battery */<br>- 2,<br>- 4,<br>- 5,<br>- 7,<br>- 9,<br>- 11,<br>- 13,<br>- 18,<br>- 20,<br>- 24,<br>- 29,<br>- 33,<br>- 40,<br>- 50,<br>- 67,<br>- 100,<br>+ 0x64, /* default AC */<br>+ 0x64, /* default Battery */<br>+ 0x01,<br>+ 0x02,<br>+ 0x03,<br>+ 0x04,<br>+ 0x05,<br>+ 0x06,<br>+ 0x07,<br>+ 0x08,<br>+ 0x09,<br>+ 0x0A,<br>+ 0x0B,<br>+ 0x0C,<br>+ 0x0D,<br>+ 0x0E,<br>+ 0x0F,<br>+ 0x10,<br>+ 0x11,<br>+ 0x12,<br>+ 0x13,<br>+ 0x14,<br>+ 0x15,<br>+ 0x16,<br>+ 0x17,<br>+ 0x18,<br>+ 0x19,<br>+ 0x1A,<br>+ 0x1B,<br>+ 0x1C,<br>+ 0x1D,<br>+ 0x1E,<br>+ 0x1F,<br>+ 0x20,<br>+ 0x21,<br>+ 0x22,<br>+ 0x23,<br>+ 0x24,<br>+ 0x25,<br>+ 0x26,<br>+ 0x27,<br>+ 0x28,<br>+ 0x29,<br>+ 0x2A,<br>+ 0x2B,<br>+ 0x2C,<br>+ 0x2D,<br>+ 0x2E,<br>+ 0x2F,<br>+ 0x30,<br>+ 0x31,<br>+ 0x32,<br>+ 0x33,<br>+ 0x34,<br>+ 0x35,<br>+ 0x36,<br>+ 0x37,<br>+ 0x38,<br>+ 0x39,<br>+ 0x3A,<br>+ 0x3B,<br>+ 0x3C,<br>+ 0x3D,<br>+ 0x3E,<br>+ 0x3F,<br>+ 0x40,<br>+ 0x41,<br>+ 0x42,<br>+ 0x43,<br>+ 0x44,<br>+ 0x45,<br>+ 0x46,<br>+ 0x47,<br>+ 0x48,<br>+ 0x49,<br>+ 0x4A,<br>+ 0x4B,<br>+ 0x4C,<br>+ 0x4D,<br>+ 0x4E,<br>+ 0x4F,<br>+ 0x50,<br>+ 0x51,<br>+ 0x52,<br>+ 0x53,<br>+ 0x54,<br>+ 0x55,<br>+ 0x56,<br>+ 0x57,<br>+ 0x58,<br>+ 0x59,<br>+ 0x5A,<br>+ 0x5B,<br>+ 0x5C,<br>+ 0x5D,<br>+ 0x5E,<br>+ 0x5F,<br>+ 0x60,<br>+ 0x61,<br>+ 0x62,<br>+ 0x63,<br>+ 0x64<br> })<br> }<br></pre><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: newchange </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>