drivers/intel/gma: Depend less on default fb values

Instead of hard-coding a lot of default values of the framebuffer config,
we use the values provided by Display_Probing.Scan_Ports() and only
overwrite what is necessary. This way we are more independent from
changes inside libgfxinit.

Change-Id: I121bbd926532c27321446282aa334cc45cdbeef1
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/25452
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Nico Huber 2018-03-29 15:15:10 +02:00 committed by Patrick Georgi
parent 06c8c0d1fe
commit 71cbd71eb5
2 changed files with 10 additions and 12 deletions

View File

@ -79,12 +79,11 @@ is
min_v := pos16'min (min_v, configs (i).Mode.V_Visible); min_v := pos16'min (min_v, configs (i).Mode.V_Visible);
end loop; end loop;
fb := fb := configs (Primary).Framebuffer;
(Width => Width_Type (min_h), fb.Width := Width_Type (min_h);
Height => Height_Type (min_v), fb.Height := Height_Type (min_v);
BPC => 8, fb.Stride := Div_Round_Up (fb.Width, 16) * 16;
Stride => Div_Round_Up (Width_Type (min_h), 16) * 16,
Offset => 0);
for i in Pipe_Index loop for i in Pipe_Index loop
exit when configs (i).Port = Disabled; exit when configs (i).Port = Disabled;

View File

@ -51,12 +51,11 @@ is
vga_io_init; vga_io_init;
vga_textmode_init; vga_textmode_init;
configs (Primary).Framebuffer := -- override probed framebuffer config
(Width => 640, configs (Primary).Framebuffer.Width := 640;
Height => 400, configs (Primary).Framebuffer.Height := 400;
BPC => Auto_BPC, -- ignored for VGA plane configs (Primary).Framebuffer.Offset :=
Stride => 320, -- ignored VGA_PLANE_FRAMEBUFFER_OFFSET;
Offset => VGA_PLANE_FRAMEBUFFER_OFFSET);
HW.GFX.GMA.Dump_Configs (configs); HW.GFX.GMA.Dump_Configs (configs);
HW.GFX.GMA.Update_Outputs (configs); HW.GFX.GMA.Update_Outputs (configs);