>>>>> "CS" == Christian Schmidt <schm...@digadd.de> writes:
CS> The current logic misunderstands the spec about CEA 18byte descriptors. CS> First, the spec doesn't state "detailed timing descriptors" but "18 byte CS> descriptors", so any data record could be stored, mixed timings and CS> other data, just as in the standard EDID. CS> Second, the lower four bit of byte 3 of the CEA record do not contain CS> the number of descriptors, but "the total number of DTDs defining native CS> formats in the whole EDID [...], starting with the first DTD in the DTD CS> list (which starts in the base EDID block)." A device can of course CS> support non-native formats. CS> As such the number can't be used to determine n, and the existing code CS> will filter non-timing 18byte descriptors anyway. CS> V2 removes an unused variable warning. CS> Signed-off-by: Christian Schmidt <schmidt@digadd,de> Tested-by: James Cloos <cl...@jhcloos.com> Works fine here on top of Linus’ 7f80850d3f9f with a Radeon HD 4290 and an hdmi-connected tv. -JimC -- James Cloos <cl...@jhcloos.com> OpenPGP: 1024D/ED7DAEA6 CS> _______________________________________________ CS> dri-devel mailing list CS> dri-devel@lists.freedesktop.org CS> http://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel
