On Tue, 12 Sep 2017, "Chauhan, Madhav" <[email protected]> wrote: >> -----Original Message----- >> From: Intel-gfx [mailto:[email protected]] On Behalf Of >> Vidya Srinivas >> Sent: Tuesday, September 5, 2017 3:15 PM >> To: [email protected] >> Cc: Srinivas, Vidya <[email protected]> >> Subject: [Intel-gfx] [PATCH] Revert "drm/i915/bxt: Disable device ready >> before shutdown command" >> >> From: Uma Shankar <[email protected]> >> >> This reverts commit bbdf0b2ff32aa75c7bd167569130e9391d2e6282. >> >> Disable device ready before shutdown command was added previously to >> avoid a split screen issue seen on dual link DSI panels. As of now, dual >> link is >> not supported and will need some rework in the upstream code. For single >> link DSI panels, the change is not required. This will cause failure in >> sending >> SHUTDOWN packet during disable. Hence reverting the change. Will handle >> the change as part of dual link enabling in upstream. > > Already there are lot of changes wrt to DSI dual link inside intel_dsi.c. > Can't we keep this change as well under the condition if (dual_link && > Broxton)?? > Might help to fetch dual link changes in future instead of losing it.
I've pushed the patch as-is. I don't think there are grounds to keep it even with dual_link. As Uma says, these things may be panel specific too. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
