search

timvideos / edid.tv

EDID Database Website
GNU General Public License v2.0
26 stars 7 forks source link

Implement parsing of CVT Support Definition #18

Closed pp3345 closed 5 years ago

pp3345 commented 5 years ago

This PR adds support for parsing the optional CVT Support Definition included in some Display Range Limits Display Descriptors.

I've also added some CSS for nested tables as I felt that this was a nice and compact way to present the indicated support values: screenshot from 2018-12-10 20-23-15 (yes, this EDID indeed does indicate a preferred vertical refresh rate of 0 Hz)

Note that this commit not only adds new properties to the EDID model, but also changes the type of mrl_max_pixel_clock to allow for the 0.25 MHz precision introduced by the CVT Support Definition.

mithro commented 5 years ago

Looks good to me!

pp3345 commented 5 years ago

Oops, forgot to include the test case in the commit :-)