[PATCH] drm: parse color format support for digital displays
jbarnes at virtuousgeek.org
Fri Apr 15 12:27:52 PDT 2011
On Fri, 15 Apr 2011 12:19:31 -0700
Jesse Barnes <jbarnes at virtuousgeek.org> wrote:
> On Fri, 15 Apr 2011 15:13:02 -0400
> Adam Jackson <ajax at 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.
Jesse Barnes, Intel Open Source Technology Center
More information about the dri-devel