On Tue, Sep 5, 2017 at 5:10 AM, Lars-Peter Clausen <lars@xxxxxxxxxx> 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@xxxxxxxxxx> 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@xxxxxxxxxx> thanks -john _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel