diff options
| author | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2011-11-25 17:26:13 +0200 |
|---|---|---|
| committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2011-12-02 08:54:55 +0200 |
| commit | 9f808956d3cd07c8dc69e213931b19bad56344fb (patch) | |
| tree | d33508726c6532fbe00df6ef3fadb3c6a7f650c3 /drivers/video | |
| parent | cc372d84e231ea9f574ef5de9a7ab39dee5a021d (diff) | |
| download | blackbird-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.c | 24 |
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; } } |
