[coreboot-gerrit] Change in ...coreboot[master]: 3rdparty/libgfxinit: Update submodule pointer

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/30311 )

Change subject: 3rdparty/libgfxinit: Update submodule pointer
......................................................................

3rdparty/libgfxinit: Update submodule pointer

Updates to current master.

This includes:
- A fix for textmode scaling on G45
- Refactor things to rely less on inline proving
- Increased width of modeline fields to 32 bits

Change-Id: Iab2915b747f6e4fa4e78eb28fea29bb3a9b3b687
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
Signed-off-by: Nico Huber <nico.h at gmx.de>
Reviewed-on: https://review.coreboot.org/c/30311
Reviewed-by: Patrick Georgi <pgeorgi at google.com>
Tested-by: build bot (Jenkins) <no-reply at coreboot.org>
---
M 3rdparty/libgfxinit
M src/drivers/intel/gma/hires_fb/gma.adb
2 files changed, 5 insertions(+), 5 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Georgi: Looks good to me, approved
  Nico Huber: Looks good to me, approved



diff --git a/3rdparty/libgfxinit b/3rdparty/libgfxinit
index 718c79b..f70edda 160000
--- a/3rdparty/libgfxinit
+++ b/3rdparty/libgfxinit
@@ -1 +1 @@
-Subproject commit 718c79bb0713b5b90c9cc44e03197dc777066e3d
+Subproject commit f70eddafbc2c6045a14e2f8bbb3273ee738fbaf7
diff --git a/src/drivers/intel/gma/hires_fb/gma.adb b/src/drivers/intel/gma/hires_fb/gma.adb
index ca0f7084..bd14132 100644
--- a/src/drivers/intel/gma/hires_fb/gma.adb
+++ b/src/drivers/intel/gma/hires_fb/gma.adb
@@ -60,8 +60,8 @@
 
       success : boolean;
 
-      min_h : pos16 := pos16'last;
-      min_v : pos16 := pos16'last;
+      min_h : pos32 := pos32'last;
+      min_v : pos32 := pos32'last;
    begin
       lightup_ok := 0;
 
@@ -75,8 +75,8 @@
             for i in Pipe_Index loop
                exit when configs (i).Port = Disabled;
 
-               min_h := pos16'min (min_h, configs (i).Mode.H_Visible);
-               min_v := pos16'min (min_v, configs (i).Mode.V_Visible);
+               min_h := pos32'min (min_h, configs (i).Mode.H_Visible);
+               min_v := pos32'min (min_v, configs (i).Mode.V_Visible);
             end loop;
 
             fb := configs (Primary).Framebuffer;

-- 
To view, visit https://review.coreboot.org/c/coreboot/+/30311
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iab2915b747f6e4fa4e78eb28fea29bb3a9b3b687
Gerrit-Change-Number: 30311
Gerrit-PatchSet: 3
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Angel Pons <th3fanbus at gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h at gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi at google.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181221/ebf37c79/attachment.html>