diff options
| author | Nico Huber <nico.huber@secunet.com> | 2018-03-29 15:11:10 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-04-06 06:45:38 +0000 |
| commit | 06c8c0d1fed29f488fda1c7d3bc13a7edfed1be3 (patch) | |
| tree | 26a37d7d4f88753538de6d9e1cdedc0e0e0713e7 /src/drivers/intel | |
| parent | 1dd7a11ec4853b39e81f861414c113c886ed7774 (diff) | |
| download | coreboot-06c8c0d1fed29f488fda1c7d3bc13a7edfed1be3.tar.xz | |
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 one cacheline (64 bytes) as that's the mini-
mum what the hardware supports.
Change-Id: I3f824ffd7d12835935e4e4bde29fe82dc3e16f9d
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/25451
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Diffstat (limited to 'src/drivers/intel')
| -rw-r--r-- | src/drivers/intel/gma/hires_fb/gma.adb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/intel/gma/hires_fb/gma.adb b/src/drivers/intel/gma/hires_fb/gma.adb index b1168acd4d..086da9b80f 100644 --- a/src/drivers/intel/gma/hires_fb/gma.adb +++ b/src/drivers/intel/gma/hires_fb/gma.adb @@ -83,7 +83,7 @@ is (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; |