Nico Huber has uploaded this change for review. ( https://review.coreboot.org/25451
Change subject: drivers/intel/gma: Amend stride calculation of linear fb ......................................................................
drivers/intel/gma: Amend stride calculation of linear fb
Aligning the stride up to a multiple of 64 pixels was flawed: We want to actually align up to 64 bytes.
Change-Id: I3f824ffd7d12835935e4e4bde29fe82dc3e16f9d Signed-off-by: Nico Huber nico.huber@secunet.com --- M src/drivers/intel/gma/hires_fb/gma.adb 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/51/25451/1
diff --git a/src/drivers/intel/gma/hires_fb/gma.adb b/src/drivers/intel/gma/hires_fb/gma.adb index b1168ac..086da9b 100644 --- a/src/drivers/intel/gma/hires_fb/gma.adb +++ b/src/drivers/intel/gma/hires_fb/gma.adb @@ -83,7 +83,7 @@ (Width => Width_Type (min_h), Height => Height_Type (min_v), BPC => 8, - Stride => ((Width_Type (min_h) + 63) / 64) * 64, + Stride => Div_Round_Up (Width_Type (min_h), 16) * 16, Offset => 0); for i in Pipe_Index loop exit when configs (i).Port = Disabled;