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@aheymans.xyz Signed-off-by: Nico Huber nico.h@gmx.de Reviewed-on: https://review.coreboot.org/c/30311 Reviewed-by: Patrick Georgi pgeorgi@google.com Tested-by: build bot (Jenkins) no-reply@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;