summaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorTomi Valkeinen <tomi.valkeinen@ti.com>2011-11-25 17:26:13 +0200
committerTomi Valkeinen <tomi.valkeinen@ti.com>2011-12-02 08:54:55 +0200
commit9f808956d3cd07c8dc69e213931b19bad56344fb (patch)
treed33508726c6532fbe00df6ef3fadb3c6a7f650c3 /drivers/video
parentcc372d84e231ea9f574ef5de9a7ab39dee5a021d (diff)
downloadblackbird-op-linux-9f808956d3cd07c8dc69e213931b19bad56344fb.tar.gz
blackbird-op-linux-9f808956d3cd07c8dc69e213931b19bad56344fb.zip
OMAPDSS: APPLY: fix need_isr
need_isr() should return true whenever there's work that needs to be handled in the VSYNC/FRAMEDONE irq handler. However, need_isr() is missing some cases where we need the irq handler. This patch adds checks for shadow dirty flags, so that the irq handler is started/kept running while shadow dirty flags are set. Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video')
-rw-r--r--drivers/video/omap2/dss/apply.c24
1 files changed, 23 insertions, 1 deletions
diff --git a/drivers/video/omap2/dss/apply.c b/drivers/video/omap2/dss/apply.c
index 5d205ada30de..98fef08da3ba 100644
--- a/drivers/video/omap2/dss/apply.c
+++ b/drivers/video/omap2/dss/apply.c
@@ -358,16 +358,38 @@ static bool need_isr(void)
if (mp->info_dirty)
return true;
+ /* to set GO bit */
+ if (mp->shadow_info_dirty)
+ return true;
+
list_for_each_entry(ovl, &mgr->overlays, list) {
struct ovl_priv_data *op;
op = get_ovl_priv(ovl);
+ /*
+ * NOTE: we check extra_info flags even for
+ * disabled overlays, as extra_infos need to be
+ * always written.
+ */
+
+ /* to write new values to registers */
+ if (op->extra_info_dirty)
+ return true;
+
+ /* to set GO bit */
+ if (op->shadow_extra_info_dirty)
+ return true;
+
if (!op->enabled)
continue;
/* to write new values to registers */
- if (op->info_dirty || op->extra_info_dirty)
+ if (op->info_dirty)
+ return true;
+
+ /* to set GO bit */
+ if (op->shadow_info_dirty)
return true;
}
}
OpenPOWER on IntegriCloud