<div dir="ltr"><div>Checking up on this patch from a few months back that I'd like to get included..  Acked by Daniel Vetter[1] and Reviewed by Jani Nikula[2].  However ajax has not yet provided comments.  Is this SOL without feedback from ajax?</div>


<div><br></div><div>[1] <a href="http://lists.freedesktop.org/archives/dri-devel/2013-March/036457.html" target="_blank">http://lists.freedesktop.org/archives/dri-devel/2013-March/036457.html</a></div>


<div>[2] <a href="http://lists.freedesktop.org/archives/dri-devel/2013-April/036829.html" target="_blank">http://lists.freedesktop.org/archives/dri-devel/2013-April/036829.html</a></div><div class="gmail_extra"><br><br><div class="gmail_quote">




On Mon, Mar 25, 2013 at 5:58 PM, Dylan Semler <span dir="ltr"><<a href="mailto:dylan.semler@gmail.com" target="_blank">dylan.semler@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">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>
<span><font color="#888888"><br>
--<br>
1.7.11.7<br>
<br>
</font></span></blockquote></div><br></div></div>