On Thu, 31 Mar 2022, Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> wrote: >> - >> - if (edid->revision > 4) >> - DRM_DEBUG("EDID minor > 4, assuming backward compatibility\n"); > > This debug message seems to disappear. Intentional? Intentional, but failed to mention it in the commit message. Do we want to keep it? With my new approach, it basically means another valid return value that's distinct from just ok. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center