[coreboot-gerrit] Change in coreboot[master]: drivers/intel/gma: increase # of brightness steps

[Matt DeVillier (Code Review)]

Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/22523


Change subject: drivers/intel/gma: increase # of brightness steps
......................................................................

drivers/intel/gma: increase # of brightness steps

Increase granularity/number of steps across adjustable brightness
range (1-100), to provide for smoother transitions when adjusting.
Default levels unchanged, just converted to hex format as values
were copied from an ACPI dump of a vendor/UEFI firmware.

TEST: boot Windows on google/wolf, observe smooth transition when
adjusting brightness via keyboard controls.

Change-Id: I3b7a693a163adeceb3623c2703f3942328679ac4
Signed-off-by: Matt DeVillier <matt.devillier at gmail.com>
---
M src/drivers/intel/gma/acpi/default_brightness_levels.asl
1 file changed, 103 insertions(+), 19 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/23/22523/1

diff --git a/src/drivers/intel/gma/acpi/default_brightness_levels.asl b/src/drivers/intel/gma/acpi/default_brightness_levels.asl
index 6c6f35e..8f560ff 100644
--- a/src/drivers/intel/gma/acpi/default_brightness_levels.asl
+++ b/src/drivers/intel/gma/acpi/default_brightness_levels.asl
@@ -1,24 +1,108 @@
 Scope (GFX0)
 {
-	Name (BRIG, Package (0x12)
+	Name (BRIG, Package (0x66)
 	{
-		100, /* default AC */
-		100, /* default Battery */
-		  2,
-		  4,
-		  5,
-		  7,
-		  9,
-		 11,
-		 13,
-		 18,
-		 20,
-		 24,
-		 29,
-		 33,
-		 40,
-		 50,
-		 67,
-		100,
+		0x64, /* default AC */
+		0x64, /* default Battery */
+		0x01,
+		0x02,
+		0x03,
+		0x04,
+		0x05,
+		0x06,
+		0x07,
+		0x08,
+		0x09,
+		0x0A,
+		0x0B,
+		0x0C,
+		0x0D,
+		0x0E,
+		0x0F,
+		0x10,
+		0x11,
+		0x12,
+		0x13,
+		0x14,
+		0x15,
+		0x16,
+		0x17,
+		0x18,
+		0x19,
+		0x1A,
+		0x1B,
+		0x1C,
+		0x1D,
+		0x1E,
+		0x1F,
+		0x20,
+		0x21,
+		0x22,
+		0x23,
+		0x24,
+		0x25,
+		0x26,
+		0x27,
+		0x28,
+		0x29,
+		0x2A,
+		0x2B,
+		0x2C,
+		0x2D,
+		0x2E,
+		0x2F,
+		0x30,
+		0x31,
+		0x32,
+		0x33,
+		0x34,
+		0x35,
+		0x36,
+		0x37,
+		0x38,
+		0x39,
+		0x3A,
+		0x3B,
+		0x3C,
+		0x3D,
+		0x3E,
+		0x3F,
+		0x40,
+		0x41,
+		0x42,
+		0x43,
+		0x44,
+		0x45,
+		0x46,
+		0x47,
+		0x48,
+		0x49,
+		0x4A,
+		0x4B,
+		0x4C,
+		0x4D,
+		0x4E,
+		0x4F,
+		0x50,
+		0x51,
+		0x52,
+		0x53,
+		0x54,
+		0x55,
+		0x56,
+		0x57,
+		0x58,
+		0x59,
+		0x5A,
+		0x5B,
+		0x5C,
+		0x5D,
+		0x5E,
+		0x5F,
+		0x60,
+		0x61,
+		0x62,
+		0x63,
+		0x64
 	})
 }

-- 
To view, visit https://review.coreboot.org/22523
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3b7a693a163adeceb3623c2703f3942328679ac4
Gerrit-Change-Number: 22523
Gerrit-PatchSet: 1
Gerrit-Owner: Matt DeVillier <matt.devillier at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20171119/a753cde5/attachment.html>