On Fri, Apr 07, 2017 at 07:39:20PM +0300, Shashank Sharma wrote: > From: Jose Abreu <jose.ab...@synopsys.com> > > HDMI 2.0 spec adds support for ycbcr420 subsampled output. > CEA-861-F adds two new blocks in EDID, to provide information about > sink's support for ycbcr420 output. > > These new blocks are: > - ycbcr420 video data (vdb) block: video modes which can be supported > only in ycbcr420 output mode. > - ycbcr420 video capability data (vcb) block: video modes which can be > support in ycbcr420 output mode also (along with RGB, YCBCR 444/422 etc) > > This patch adds parsing and handling of ycbcr420-vdb in the DRM > layer. > > This patch is a modified version of Jose's RFC patch: > https://patchwork.kernel.org/patch/9492327/ > so the authorship is maintained. > > Cc: Ville Syrjala <ville.syrj...@linux.intel.com> > Signed-off-by: Jose Abreu <joab...@synopsys.com> > Signed-off-by: Shashank Sharma <shashank.sha...@intel.com> > --- > drivers/gpu/drm/drm_edid.c | 54 > +++++++++++++++++++++++++++++++++++++++++++-- > drivers/gpu/drm/drm_modes.c | 10 +++++++-- > include/drm/drm_connector.h | 1 + > include/uapi/drm/drm_mode.h | 6 +++++ > 4 files changed, 67 insertions(+), 4 deletions(-) <snip> > diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h > index 4eeda12..cef76b2 100644 > --- a/include/drm/drm_connector.h > +++ b/include/drm/drm_connector.h > @@ -199,6 +199,7 @@ struct drm_display_info { > #define DRM_COLOR_FORMAT_RGB444 (1<<0) > #define DRM_COLOR_FORMAT_YCRCB444 (1<<1) > #define DRM_COLOR_FORMAT_YCRCB422 (1<<2) > +#define DRM_COLOR_FORMAT_YCRCB420 (1<<2) > > /** > * @color_formats: HDMI Color formats, selects between RGB and YCrCb > diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h > index 8c67fc0..1e74d8e 100644 > --- a/include/uapi/drm/drm_mode.h > +++ b/include/uapi/drm/drm_mode.h > @@ -84,6 +84,12 @@ extern "C" { > #define DRM_MODE_FLAG_3D_L_DEPTH_GFX_GFX_DEPTH (6<<14) > #define DRM_MODE_FLAG_3D_TOP_AND_BOTTOM (7<<14) > #define DRM_MODE_FLAG_3D_SIDE_BY_SIDE_HALF (8<<14) > +/* > + * HDMI 2.0 > + */ > +#define DRM_MODE_FLAG_420_MASK (0x03<<23) > +#define DRM_MODE_FLAG_420 (1<<23) > +#define DRM_MODE_FLAG_420_ONLY (1<<24)
Adding those would again break the uabi. We can't add new mode flags without some kind of client cap. But I think we agreed that new user space visible mode flags aren't needed, and instad we can keep it all internal? -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
