Hi, On 10/01/2021 13:51, Simon Ser wrote: > 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.
Indeed, but the masks are "private", and would be updated accordingly when introducing new layouts. > > 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 <[email protected]> > Fixes: d6528ec88309 ("drm/fourcc: Add modifier definitions for describing > Amlogic Video Framebuffer Compression") > Cc: Neil Armstrong <[email protected]> > Cc: Sam Ravnborg <[email protected]> > Cc: Kevin Hilman <[email protected]> > Cc: Daniel Vetter <[email protected]> > --- > include/uapi/drm/drm_fourcc.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/uapi/drm/drm_fourcc.h b/include/uapi/drm/drm_fourcc.h > index 723c8e23ca87..5f42a14481bd 100644 > --- a/include/uapi/drm/drm_fourcc.h > +++ b/include/uapi/drm/drm_fourcc.h > @@ -1036,9 +1036,9 @@ drm_fourcc_canonicalize_nvidia_format_mod(__u64 > modifier) > * Not all combinations are valid, and different SoCs may support different > * combinations of layout and options. > */ > -#define __fourcc_mod_amlogic_layout_mask 0xf > +#define __fourcc_mod_amlogic_layout_mask 0xff > #define __fourcc_mod_amlogic_options_shift 8 > -#define __fourcc_mod_amlogic_options_mask 0xf > +#define __fourcc_mod_amlogic_options_mask 0xff > > #define DRM_FORMAT_MOD_AMLOGIC_FBC(__layout, __options) \ > fourcc_mod_code(AMLOGIC, \ > Anyway: Acked-by: Neil Armstrong <[email protected]> Thanks, Neil _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
