On Tue, 2018-07-24 at 21:15 +0530, Uma Shankar wrote: > --- a/include/uapi/drm/drm_mode.h > +++ b/include/uapi/drm/drm_mode.h > @@ -209,6 +209,17 @@ > #define DRM_MODE_CONTENT_PROTECTION_DESIRED 1 > #define DRM_MODE_CONTENT_PROTECTION_ENABLED 2 > > +enum extended_colorimetry { > + EXTENDED_COLORIMETRY_XV_YCC_601 = 0, > + EXTENDED_COLORIMETRY_XV_YCC_709, > + EXTENDED_COLORIMETRY_S_YCC_601, > + EXTENDED_COLORIMETRY_ADOBE_YCC_601, > + EXTENDED_COLORIMETRY_ADOBE_RGB, > + EXTENDED_COLORIMETRY_BT2020_RGB, > + EXTENDED_COLORIMETRY_BT2020_YCC, > + EXTENDED_COLORIMETRY_BT2020_CYCC, > +}; This doesn't give any way to distinguish "not set" from BT.601, which I'm not sure I like. Is this enum simply built to match the values you're injecting into the InfoFrame? Would we need a different enum for DisplayPort? - ajax _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx