diff options
author | Yu-Ping Wu <yupingso@chromium.org> | 2021-08-23 16:27:18 +0800 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2021-08-27 02:53:38 +0000 |
commit | 52889c9c9f6d634f580c003b77889301916ef5e5 (patch) | |
tree | 84ae4d20b16a01a558c63c485fe883555b21c584 /src/superio/nuvoton/nct5572d/superio.c | |
parent | b69d1dea47552fa0939c884523570b9ce6e42017 (diff) |
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>
Diffstat (limited to 'src/superio/nuvoton/nct5572d/superio.c')
0 files changed, 0 insertions, 0 deletions