Change in coreboot[master]: libgfxinit: Allow to override screen rotation via CMOS option

[Nico Huber (Code Review)]

Nico Huber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38924 )

Change subject: libgfxinit: Allow to override screen rotation via CMOS option
......................................................................

libgfxinit: Allow to override screen rotation via CMOS option

Change-Id: I30a0aca6b7f79b1bbd96b7f527b896885affc77d
Signed-off-by: Nico Huber 
---
M src/drivers/intel/gma/hires_fb/gma-gfx_init.adb
1 file changed, 8 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/38924/1

diff --git a/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb b/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb
index 014e92b..40ed0c3 100644
--- a/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb
+++ b/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb
@@ -1,4 +1,5 @@
 with CB.Config;
+with CB.NVRAM;

use CB;

@@ -56,12 +57,19 @@

procedure Screen_Rotation (rotation : out Rotation_Type)
    is
+      use type word8;
+      raw_rotation : word8 := 255;
    begin
       rotation :=
         (if    Config.DEFAULT_SCREEN_ROTATION_90   then Rotated_90
          elsif Config.DEFAULT_SCREEN_ROTATION_180  then Rotated_180
          elsif Config.DEFAULT_SCREEN_ROTATION_270  then Rotated_270
                                                    else No_Rotation);
+
+      NVRAM.get_option (raw_rotation, "screen_rotation");
+      if raw_rotation <= Rotation_Type'pos (Rotation_Type'last) then
+         rotation := Rotation_Type'val (raw_rotation);
+      end if;
    end Screen_Rotation;

procedure gfxinit (lightup_ok : out Interfaces.C.int)

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38924 )

Change subject: libgfxinit: Allow to override screen rotation via CMOS option
......................................................................

Patch Set 1: Code-Review+1

[Nico Huber (Code Review)]

Hello build bot (Jenkins), Angel Pons, Patrick Rudolph,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/38924

to look at the new patch set (#6).

Change subject: libgfxinit: Allow to override screen rotation via CMOS option
......................................................................

libgfxinit: Allow to override screen rotation via CMOS option

Change-Id: I30a0aca6b7f79b1bbd96b7f527b896885affc77d
Signed-off-by: Nico Huber 
---
M src/drivers/intel/gma/hires_fb/gma-gfx_init.adb
1 file changed, 8 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/38924/6