coreboot-kgpe-d16/payloads/libpayload/drivers/video
Yu-Ping Wu 52889c9c9f libpayload: cbgfx: Clear screen by sequential access
Currently clear_screen() calls set_pixel() to set all pixels. However,
the actual order of pixels being set depends on the framebuffer
orientation. With NORMAL orientation, the framebuffer is accessed
sequentially; with LEFT_UP/RIGHT_UP orientation, it is accessed back and
forth, leading to performance drop (>1 second on bugzzy).

Therefore, ensure sequential access to the framebuffer, regardless of
the orientation.

BUG=b:194967458
TEST=emerge-cherry libpayload
BRANCH=dedede

Change-Id: Iecaff5b6abc24ba4b3859cbc44c0d61b2a90b2d9
Signed-off-by: Yu-Ping Wu <yupingso@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57104
Reviewed-by: Hung-Te Lin <hungte@chromium.org>
Reviewed-by: Julius Werner <jwerner@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
2021-08-27 02:53:38 +00:00
..
bitmap.h treewide: Remove "this file is part of" lines 2020-05-11 17:11:40 +00:00
corebootfb.c payloads: Drop unneeded empty lines 2020-09-21 16:20:57 +00:00
font.c treewide: Remove "this file is part of" lines 2020-05-11 17:11:40 +00:00
font.h treewide: Remove "this file is part of" lines 2020-05-11 17:11:40 +00:00
font8x16.c treewide: Remove "this file is part of" lines 2020-05-11 17:11:40 +00:00
font8x16.h treewide: Remove "this file is part of" lines 2020-05-11 17:11:40 +00:00
geodelx.c payloads: Drop unneeded empty lines 2020-09-21 16:20:57 +00:00
graphics.c libpayload: cbgfx: Clear screen by sequential access 2021-08-27 02:53:38 +00:00
vga.c treewide: Remove "this file is part of" lines 2020-05-11 17:11:40 +00:00
video.c treewide: Remove "this file is part of" lines 2020-05-11 17:11:40 +00:00