Nico Huber has uploaded this change for review.

View Change

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;

To view, visit change 25451. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3f824ffd7d12835935e4e4bde29fe82dc3e16f9d
Gerrit-Change-Number: 25451
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h@gmx.de>