drm/mgag200: Split DPMS function into helpers
authorThomas Zimmermann <tzimmermann@suse.de>
Tue, 7 Jul 2020 08:24:08 +0000 (10:24 +0200)
committerThomas Zimmermann <tzimmermann@suse.de>
Tue, 14 Jul 2020 07:13:40 +0000 (09:13 +0200)
commit153fef4115e886de8db2e5c58dc3bc63e7a68a09
tree47c77916ada14f87bcff772a43d517c3b3e570df
parent379ba8043db2251302dbe623013effa696e52b2a
drm/mgag200: Split DPMS function into helpers

Of the DPMS code, only ON and OFF states are used. Simplify mode setting
by moving both into separate functions and removing the rest.

The original code busy waited in the middle of updating the screen state
in SEQ1. To simplify the procedure, the new code busy waits first and then
updates SEQ1 in one chunk.

The DPMS code also set the LUT before enabling the screen. The patch moves
this code into the simple-display pipe's enable function.

v2:
* comment on SEQ1 updates in commit message

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Lyude Paul <lyude@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200707082411.6583-5-tzimmermann@suse.de
drivers/gpu/drm/mgag200/mgag200_mode.c
drivers/gpu/drm/mgag200/mgag200_reg.h