On Thu, Nov 06, 2014 at 12:35:48PM +0100, Andrzej Hajda wrote: > Exynos DSI driver uses DSI bus attach/detach callbacks to implement > panel hotplug mechanism. The patch moves panel attachment code > from .detect callback to DSI bus callbacks. It makes the code > simpler and more straightforward. > The patch removes also redundant and lock unprotected dpms_off call > from unbind code. > > Signed-off-by: Andrzej Hajda <a.hajda at samsung.com> > --- > Hi, > > This patch simplifies drm part of hotplug mechanism in dsi attach/detach > callbacks. > It also fixes minor bug with unprotected dpms_off call, which is unneccesary > anyway. > > But the main reason I have looked at this code again is to show that > we do not need zombie panels, we can kill panels properly if we have hot > plug/unplug > callbacks. And it is quite simple: we just need to take mode_config.mutex > locks > to synchronize dsi callbacks with drm, nothing more. > It works nicely because of two things: > - panel callbacks are called always under this lock, > - callbacks are installed after drmdev creation and deinstalled before drmdev > destroy - > this way we assure drmdev is always present in callbacks.
Again, something like what you propose is going to work only for DSI devices. What we need is a solution that works for all other types of panels, too. This is even more important when we consider bridges because they are less likely to be on a DSI bus. Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20141106/09098297/attachment.sig>
