[PATCH 1/4] drm/bridge: adv7511: Properly update EDID when no EDID was found

John Stultz john.stultz at linaro.org
Tue Sep 5 16:44:45 UTC 2017


On Tue, Sep 5, 2017 at 5:10 AM, Lars-Peter Clausen <lars at metafoo.de> wrote:
> Currently adv7511_get_modes() bails out early when no EDID could be
> retrieved. This leaves the previous EDID in place, which is typically not
> the intended behavior and might confuse applications. Instead the EDID
> should be cleared when no EDID could be retrieved.
>
> All functions that are called after the EDID check handle the case where
> the EDID is NULL just fine and exhibit the expected behavior, so just drop
> the check.
>
> Signed-off-by: Lars-Peter Clausen <lars at metafoo.de>

No objections from me. Ran these on my HiKey board ontop of Linus's
HEAD, and everything seemed ok.

So for the whole patchset:
Tested-by: John Stultz <john.stultz at linaro.org>

thanks
-john


More information about the dri-devel mailing list