On Fri, 15 Apr 2011 12:19:31 -0700 Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 15 Apr 2011 15:13:02 -0400 > Adam Jackson <ajax@xxxxxxxxxx> 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 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
