On Thu, Mar 05, 2026 at 10:04:53AM +0100, Maxime Ripard wrote: > The EDID parsing code initially introduced the DRM_COLOR_FORMAT_* > defines to represent the sink capabilities. Since a given sink could > support multiple formats, it was first defined as a bitmask. > > However, the core and drivers have since leveraged those defines to > represent both the supported formats but also the current format being > used. > > Considering the latter case, the more natural, and consistent, thing to > do would be to create an enum of all the possible formats, and then list > the supported formats using a bitmask of the individual enum values. > > Let's create a new enum following that pattern, drm_output_color_format, > while maintaining the DRM_COLOR_FORMAT_* compatibility to make the > transition easier. > > Acked-by: Jani Nikula <[email protected]> > Reviewed-by: Liviu Dudau <[email protected]> > Signed-off-by: Maxime Ripard <[email protected]> > --- > include/drm/drm_connector.h | 42 ++++++++++++++++++++++++++++++++++-------- > 1 file changed, 34 insertions(+), 8 deletions(-) >
Reviewed-by: Dmitry Baryshkov <[email protected]> -- With best wishes Dmitry
