Re: [PATCH 07/12] drm/edid: split drm_edid_block_valid() to check and act parts

On Thu, Mar 31, 2022 at 07:49:10PM +0300, Jani Nikula wrote:

On Thu, 31 Mar 2022, Ville Syrjälä ville.syrjala@linux.intel.com 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+.