<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED - Linux: REGRESSION: drm: cmdline EDID override mechanisms broken since 4.15"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=107583#c20">Comment # 20</a>
              on <a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED - Linux: REGRESSION: drm: cmdline EDID override mechanisms broken since 4.15"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=107583">bug 107583</a>
              from <span class="vcard"><a class="email" href="mailto:harish.chegondi@intel.com" title="harish.chegondi@intel.com">harish.chegondi@intel.com</a>
</span></b>
        <pre>(In reply to harish.chegondi from <a href="show_bug.cgi?id=107583#c19">comment #19</a>)
<span class="quote">> -      if (connector->force == DRM_FORCE_UNSPECIFIED && !drm_probe_ddc(adapter))
> -          return NULL;
> -
>    edid = drm_do_get_edid(connector, drm_do_probe_ddc_edid, adapter);</span >

I think the above if statement can be removed as all drm_probe_ddc() does is
call drm_do_probe_ddc_edid(). drm_do_get_edid() in the above case first handles
override EDID if necessary and calls drm_do_probe_ddc_edid() which was passed
as a function pointer to drm_do_get_edid().</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are on the CC list for the bug.</li>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>