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

[Jesse Barnes <jbarnes@virtuousgeek.org>]

On Fri, 15 Apr 2011 12:19:31 -0700
Jesse Barnes <jbarnes@virtuousgeek.org> wrote:

> On Fri, 15 Apr 2011 15:13:02 -0400
> Adam Jackson <ajax@redhat.com> wrote:
> > 	info->color_formats = DRM_COLOR_FORMAT_RGB444;
> > 	if (edid->features & DRM_EDID_FEATURE_YCRCB444)
> > 		info->color_formats |= DRM_COLOR_FORMAT_YCBCR444;
> > 	if (edid->features & DRM_EDID_FEATURE_YCRCB422)
> > 		info->color_formats |= DRM_COLOR_FORMAT_YCBCR422;
> > 
> > ... which is corrected to not include RGB uselessly in the 
> > DRM_EDID_FEATURE_* tokens.  I should have noticed that in your first 
> > patch, whoops.
> 
> I don't think EDID supports that?  The docs I have here imply that
> either RGB, RGB + YCrCb444 or RGB + YCrCb444 + YCrCb422 are the only
> things we can report.
> 
> Or is there a CEA block extension that allows for more granularity?

Nevermind, I even had the define correct, I just missed it when adding
the conditionals.  Will fix things to look like the above as it's nicer
than doing the switch.

Thanks,
-- 
Jesse Barnes, Intel Open Source Technology Center