<div dir="ltr"><div>On Mon, Mar 25, 2013 at 5:58 PM, Dylan Semler <<a href="mailto:dylan.semler@gmail.com" target="_blank">dylan.semler@gmail.com</a>> wrote:<br>
><br>> Changes in this version<br>> * rename do_force_quirk_modes() -> do_force_quirk_mode()<br>> * use list_for_each_entry() instead of list_for_each_entry_safe() in<br>> do_force_quirk_mode()<br>
> * remove num_modes from do_force_quirk_mode(), just return 1 or 0 as<br>
> appropriate<br>> * remove unused quirks argument from add_force_quirk_modes()<br>> * fixes to allow cases of forcing multiple modes<br>> * adjusted comments to adhere closer to style guides<br>><br>
> Changes in version 3<br>> * Uses drm_cvt_mode() instead of drm_gtf_mode() to build modeline<br>> * Adds bool to specify reduced blanking to edid_quirk_force_mode<br>> * Removes preferred bit from all other modes<br>
><br>> Changes in version 2<br>> * none<br>><br>> There is at least one monitor that doesn't report its native resolution<br>> in its EDID block. This enhancement extends the EDID quirk logic to<br>
> make monitors like this "just work".<br>><br>> The first patch in this series sets up a new quirk list where monitors'<br>> correct width, height, refresh rate, and reduced blanking parameters are<br>
> specified. When a matching monitor is attached the full mode is<br>> calculated with drm_cvt_mode() and added to the connector. The<br>> DRM_MODE_TYPE_PREFERRED bit is set on the new mode and unset from all<br>
> other modes.<br>><br>> The first patch also defines a new quirk bit: EDID_QUIRK_FORCE_MODE.<br>> This bit needs to be set for the new quirk list described above to be<br>> checked.<br>><br>> The second patch adds the offending monitor to the quirk lists.<br>
><br>> Dylan Semler (2):<br>> drm: Enhance EDID quirks to explicitly set a mode<br>> drm: Add EDID force quirk for MMT Monitor2Go HD+<br>><br>> drivers/gpu/drm/drm_edid.c | 89 ++++++++++++++++++++++++++++++++++++++++++++++<br>
> 1 file changed, 89 insertions(+)<br>><br>> --<br>> 1.7.11.7<br>><br></div><div><br></div><div><div>Version 3 was Acked by Daniel Vetter[1]. Any chance ajax can give his comments?</div><div><br></div><div>
[1] <a href="http://lists.freedesktop.org/archives/dri-devel/2013-March/036457.html">http://lists.freedesktop.org/archives/dri-devel/2013-March/036457.html</a></div></div></div>