[PATCH 01/35] drm/amd/display: Use udelay when waiting between aux retries

sylvain.bertrand at gmail.com sylvain.bertrand at gmail.com
Mon Feb 4 16:21:16 UTC 2019


On Mon, Feb 04, 2019 at 03:43:36PM +0000, Wentland, Harry wrote:
> DRM actually bumped this to 32 due to an issue with a Dell 4k display.

As I feared there is a retry counter higher in the code. My bad.

> It depends. I wouldn't call one or the other more correct. I seem to remember
> that the DP spec is quite vague on these retries but I could be mistaken.
> Since our driver hasn't show any problems with the DRM code and I believe
> others (such as i915) also pass DP compliance without issues I wouldn't
> proactively change this.

I reacted to this matter since my DP monitor had troubles getting out of DP off
state (iiyama) with logs which did seem to point towards a failure of getting
the EDID. It happens rarely, and I could not figure out a context to reproduce
for sure. Could be gone for good with the latest fixes.

I guess fixes for popular and defective hardware (for instance that dell 4k)
from a compliance stand point have to go in the code unfortunately.

regards,

-- 
Sylvain


More information about the amd-gfx mailing list