OMAPDSS: fix dss_ovl_unset_manager
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 6 Sep 2012 13:10:28 +0000 (16:10 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Fri, 7 Sep 2012 17:02:14 +0000 (20:02 +0300)
When we removed fifomerge support, we also changed dss_ovl_disable so
that it doesn't wait for the hardware to be finished with the overlay.
This may cause a problem when changing the overlay's manager, as
changing the manager is an immediate change. Thus if the overlay is
still being used by the HW when the manager is changed, there may be
glitches on the screen.

This patch adds a wait into dss_ovl_unset_manager, which ensures the
overlays are disabled in the HW.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
drivers/video/omap2/dss/apply.c

index 4a58d80a63154ce0ae5907e2cdb8b03fb13691f4..6354bb842856ce7197bd2dec82f4310484e01c67 100644 (file)
@@ -1330,6 +1330,13 @@ int dss_ovl_unset_manager(struct omap_overlay *ovl)
                goto err;
        }
 
+       spin_unlock_irqrestore(&data_lock, flags);
+
+       /* wait for pending extra_info updates to ensure the ovl is disabled */
+       wait_pending_extra_info_updates();
+
+       spin_lock_irqsave(&data_lock, flags);
+
        op->channel = -1;
 
        ovl->manager = NULL;