OSDN Git Service

drm/mgag200: Store values (not bits) in struct mgag200_pll_values
authorThomas Zimmermann <tzimmermann@suse.de>
Wed, 14 Jul 2021 14:22:33 +0000 (16:22 +0200)
committerThomas Zimmermann <tzimmermann@suse.de>
Sun, 8 Aug 2021 18:13:50 +0000 (20:13 +0200)
commit2dd040946ecfa5434fb0084fb5b5c2ba55639ea1
treecea2eb16ede8d3529e10dbb70740cb249d99553b
parentd9d992238a5aaf1a1e2ba013dd37dccf7b48c0a7
drm/mgag200: Store values (not bits) in struct mgag200_pll_values

The fields in struct mgag200_pll_values currently hold the bits of
each register. Store the PLL values instead and let the PLL-update
code figure out the bits for each register.

Until now, the compute function either stored plain values or register
bits in struct mgag200_pll_values. The rsp update function used the
values as-is. This made it very hard to correctly interpret the stored
values (e.g., for logging or debugging). With the cleanup, the stored
values now have a clear meaning.

v2:
* add a bit more context in the commit message (Sam)

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20210714142240.21979-7-tzimmermann@suse.de
drivers/gpu/drm/mgag200/mgag200_mode.c