[PATCH] drm: parse color format support for digital displays
Jesse Barnes
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.
Thanks,
--
Jesse Barnes, Intel Open Source Technology Center
More information about the dri-devel
mailing list