[coreboot-gerrit] Change in ...coreboot[master]: libgfxinit: Add options to limit framebuffer size

[Nico Huber (Code Review)]

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

Change subject: libgfxinit: Add options to limit framebuffer size
......................................................................

libgfxinit: Add options to limit framebuffer size

Change-Id: I47fba063629260c3a2854caf7a73f1a1e933d063
Signed-off-by: Nico Huber 
---
M src/device/Kconfig
M src/drivers/intel/gma/hires_fb/gma.adb
2 files changed, 16 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/85/30585/1

diff --git a/src/device/Kconfig b/src/device/Kconfig
index 85f1d65..f868d2f 100644
--- a/src/device/Kconfig
+++ b/src/device/Kconfig
@@ -445,6 +445,16 @@
    def_bool y
    depends on VBE_LINEAR_FRAMEBUFFER || GENERIC_LINEAR_FRAMEBUFFER

+config LINEAR_FRAMEBUFFER_MAX_WIDTH
+	int "Maximum width in pixels"
+	depends on MAINBOARD_USE_LIBGFXINIT && LINEAR_FRAMEBUFFER
+	default 4096
+
+config LINEAR_FRAMEBUFFER_MAX_HEIGHT
+	int "Maximum height in pixels"
+	depends on MAINBOARD_USE_LIBGFXINIT && LINEAR_FRAMEBUFFER
+	default 4096
+
 endmenu # "Display"

config SMBUS_HAS_AUX_CHANNELS
diff --git a/src/drivers/intel/gma/hires_fb/gma.adb b/src/drivers/intel/gma/hires_fb/gma.adb
index bd14132..1437ce2 100644
--- a/src/drivers/intel/gma/hires_fb/gma.adb
+++ b/src/drivers/intel/gma/hires_fb/gma.adb
@@ -1,3 +1,7 @@
+with CB.Config;
+
+use CB.Config;
+
 with HW.GFX;
 with HW.GFX.Framebuffer_Filler;
 with HW.GFX.GMA;
@@ -60,8 +64,8 @@

success : boolean;

-      min_h : pos32 := pos32'last;
-      min_v : pos32 := pos32'last;
+      min_h : pos32 := LINEAR_FRAMEBUFFER_MAX_WIDTH;
+      min_v : pos32 := LINEAR_FRAMEBUFFER_MAX_HEIGHT;
    begin
       lightup_ok := 0;