On Thu, Mar 31, 2022 at 07:49:10PM +0300, Jani Nikula wrote: > 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. Seems pretty pointless to me. Especially with DisplayID on the scene it seems rather unlikely that there would ever be EDID 1.5+. -- Ville Syrjälä Intel