diff options
author | Simon Ser <contact@emersion.fr> | 2021-01-10 13:51:03 +0100 |
---|---|---|
committer | Simon Ser <contact@emersion.fr> | 2021-01-12 15:52:43 +0100 |
commit | cc3283f8f41f741fbaef63d0503d8fb4a7919100 (patch) | |
tree | 41a78de8ef893266d80d58b555f546b5b03f4362 /drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | |
parent | dc25e3776e8f50ff833803ad361ec709dc78f4f9 (diff) |
drm/fourcc: fix Amlogic format modifier masks
The comment says the layout and options use 8 bits, and the shift
uses 8 bits. However the mask is 0xf, ie. 0b00001111 (4 bits).
This could be surprising when introducing new layouts or options
that take more than 4 bits, as this would silently drop the high
bits.
Make the masks consistent with the comment and the shift.
Found when writing a drm_info patch [1].
[1]: https://github.com/ascent12/drm_info/pull/67
Signed-off-by: Simon Ser <contact@emersion.fr>
Fixes: d6528ec88309 ("drm/fourcc: Add modifier definitions for describing Amlogic Video Framebuffer Compression")
Cc: Neil Armstrong <narmstrong@baylibre.com>
Cc: Sam Ravnborg <sam@ravnborg.org>
Cc: Kevin Hilman <khilman@baylibre.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Acked-by: Neil Armstrong <narmstrong@baylibre.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210110125103.15447-1-contact@emersion.fr
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c')
0 files changed, 0 insertions, 0 deletions