]> git.baikalelectronics.ru Git - kernel.git/commit
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)
commitdd5e702544593f85bbd9200a13424e8c32f16307
treecea2eb16ede8d3529e10dbb70740cb249d99553b
parent4f8b75b5a8ceb73d39c78bb663dc8270105baec9
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