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,