[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Adam Jackson
On 4/15/11 3:19 PM, Jesse Barnes wrote: > Or is there a CEA block extension that allows for more granularity? CEA has bits for the two YCbCr formats too, which we should also parse since there's plenty of 1.3+CEA blocks in the world thanks to HDMI. For CEA blocks version 2 and up (version numb

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Adam Jackson
On 4/15/11 2:40 PM, Jesse Barnes wrote: > @@ -1461,6 +1462,15 @@ static void drm_add_display_info(struct edid *edid, > info->bpp = 0; > break; > } > + > + if (edid->features & DRM_EDID_FEATURE_RGB) > + info->color_formats = DRM_COLOR_FORMAT_RGB444;

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
EDID 1.4 digital displays report the color spaces they support in the features block. Add support for grabbing this data and stuffing it into the display_info struct for driver use. Signed-off-by: Jesse Barnes --- drivers/gpu/drm/drm_edid.c |6 ++ include/drm/drm_crtc.h |5 -

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
EDID 1.4 digital displays report the color spaces they support in the features block. Add support for grabbing this data and stuffing it into the display_info struct for driver use. Signed-off-by: Jesse Barnes --- drivers/gpu/drm/drm_edid.c |6 ++ include/drm/drm_crtc.h |5 -

Re: [PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
On Fri, 15 Apr 2011 15:36:22 -0400 Adam Jackson wrote: > On 4/15/11 3:19 PM, Jesse Barnes wrote: > > > Or is there a CEA block extension that allows for more granularity? > > CEA has bits for the two YCbCr formats too, which we should also parse > since there's plenty of 1.3+CEA blocks in the

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
On Fri, 15 Apr 2011 15:36:22 -0400 Adam Jackson wrote: > On 4/15/11 3:19 PM, Jesse Barnes wrote: > > > Or is there a CEA block extension that allows for more granularity? > > CEA has bits for the two YCbCr formats too, which we should also parse > since there's plenty of 1.3+CEA blocks in the

Re: [PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Adam Jackson
On 4/15/11 3:19 PM, Jesse Barnes wrote: Or is there a CEA block extension that allows for more granularity? CEA has bits for the two YCbCr formats too, which we should also parse since there's plenty of 1.3+CEA blocks in the world thanks to HDMI. For CEA blocks version 2 and up (version num

Re: [PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
On Fri, 15 Apr 2011 12:19:31 -0700 Jesse Barnes wrote: > On Fri, 15 Apr 2011 15:13:02 -0400 > Adam Jackson wrote: > > info->color_formats = DRM_COLOR_FORMAT_RGB444; > > if (edid->features & DRM_EDID_FEATURE_YCRCB444) > > info->color_formats |= DRM_COLOR_FORMAT_YCBCR444; > >

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
On Fri, 15 Apr 2011 12:19:31 -0700 Jesse Barnes wrote: > On Fri, 15 Apr 2011 15:13:02 -0400 > Adam Jackson wrote: > > info->color_formats = DRM_COLOR_FORMAT_RGB444; > > if (edid->features & DRM_EDID_FEATURE_YCRCB444) > > info->color_formats |= DRM_COLOR_FORMAT_YCBCR444; > >

Re: [PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
On Fri, 15 Apr 2011 15:13:02 -0400 Adam Jackson wrote: > On 4/15/11 2:40 PM, Jesse Barnes wrote: > > > @@ -1461,6 +1462,15 @@ static void drm_add_display_info(struct edid *edid, > > info->bpp = 0; > > break; > > } > > + > > + if (edid->features & DRM_EDID_FEATURE_RG

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
On Fri, 15 Apr 2011 15:13:02 -0400 Adam Jackson wrote: > On 4/15/11 2:40 PM, Jesse Barnes wrote: > > > @@ -1461,6 +1462,15 @@ static void drm_add_display_info(struct edid *edid, > > info->bpp = 0; > > break; > > } > > + > > + if (edid->features & DRM_EDID_FEATURE_RG

Re: [PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Adam Jackson
On 4/15/11 2:40 PM, Jesse Barnes wrote: @@ -1461,6 +1462,15 @@ static void drm_add_display_info(struct edid *edid, info->bpp = 0; break; } + + if (edid->features & DRM_EDID_FEATURE_RGB) + info->color_formats = DRM_COLOR_FORMAT_RGB444; +

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
EDID 1.4 digital displays report the color spaces they support in the features block. Add support for grabbing this data and stuffing it into the display_info struct for driver use. Signed-off-by: Jesse Barnes --- drivers/gpu/drm/drm_edid.c | 10 ++ include/drm/drm_crtc.h |6 +

[PATCH] drm: parse color format support for digital displays

2011-04-15 Thread Jesse Barnes
EDID 1.4 digital displays report the color spaces they support in the features block. Add support for grabbing this data and stuffing it into the display_info struct for driver use. Signed-off-by: Jesse Barnes --- drivers/gpu/drm/drm_edid.c | 10 ++ include/drm/drm_crtc.h |6 +