On Wed, 2019-02-27 at 23:03 +0200, Ville Syrjälä wrote:
So instead of putting this logic into the EDID parser I guess we could just put it into the i915 fixed mode code. But then I suppose we should also fix EDID_QUIRK_FIRST_DETAILED_PREFERRED (assuming it exists for a good reason).
I don't think it has any good reason to exist, tbh. The commit adding it to xserver was for the Philips 107P5, which - being a CRT - would be entirely expected to not have the preferred bit set. We should probably have handled that instead by letting the "target a DPI near 96" logic handle things.
- ajax