Commit fc22a843 authored by Archit Taneja's avatar Archit Taneja

OMAPDSS: APPLY: Remove omap_dss_device references in wait_for_go functions

The functions dss_mgr_wait_for_go() and dss_mgr_wait_for_go_ovl() check if there
is an enabled display connected to the manager before trying to see the state of
the GO bit.

The checks related to the display can be replaced by checking the state of the
manager, i.e, whether the manager is enabled or not. This makes more sense than
checking with the connected display as the GO bit behaviour is more connected
with the manager state rather than the display state. A GO bit can only be set
if the manager is enabled. If a manager isn't enabled, we can safely assume that
the GO bit is not set.
Signed-off-by: default avatarArchit Taneja <archit@ti.com>
parent 9e7e9372
...@@ -421,17 +421,25 @@ static void wait_pending_extra_info_updates(void) ...@@ -421,17 +421,25 @@ static void wait_pending_extra_info_updates(void)
int dss_mgr_wait_for_go(struct omap_overlay_manager *mgr) int dss_mgr_wait_for_go(struct omap_overlay_manager *mgr)
{ {
unsigned long timeout = msecs_to_jiffies(500); unsigned long timeout = msecs_to_jiffies(500);
struct mgr_priv_data *mp; struct mgr_priv_data *mp = get_mgr_priv(mgr);
u32 irq; u32 irq;
unsigned long flags;
int r; int r;
int i; int i;
struct omap_dss_device *dssdev = mgr->device;
if (!dssdev || dssdev->state != OMAP_DSS_DISPLAY_ACTIVE) spin_lock_irqsave(&data_lock, flags);
if (mgr_manual_update(mgr)) {
spin_unlock_irqrestore(&data_lock, flags);
return 0; return 0;
}
if (mgr_manual_update(mgr)) if (!mp->enabled) {
spin_unlock_irqrestore(&data_lock, flags);
return 0; return 0;
}
spin_unlock_irqrestore(&data_lock, flags);
r = dispc_runtime_get(); r = dispc_runtime_get();
if (r) if (r)
...@@ -439,10 +447,8 @@ int dss_mgr_wait_for_go(struct omap_overlay_manager *mgr) ...@@ -439,10 +447,8 @@ int dss_mgr_wait_for_go(struct omap_overlay_manager *mgr)
irq = dispc_mgr_get_vsync_irq(mgr->id); irq = dispc_mgr_get_vsync_irq(mgr->id);
mp = get_mgr_priv(mgr);
i = 0; i = 0;
while (1) { while (1) {
unsigned long flags;
bool shadow_dirty, dirty; bool shadow_dirty, dirty;
spin_lock_irqsave(&data_lock, flags); spin_lock_irqsave(&data_lock, flags);
...@@ -486,21 +492,30 @@ int dss_mgr_wait_for_go_ovl(struct omap_overlay *ovl) ...@@ -486,21 +492,30 @@ int dss_mgr_wait_for_go_ovl(struct omap_overlay *ovl)
{ {
unsigned long timeout = msecs_to_jiffies(500); unsigned long timeout = msecs_to_jiffies(500);
struct ovl_priv_data *op; struct ovl_priv_data *op;
struct omap_dss_device *dssdev; struct mgr_priv_data *mp;
u32 irq; u32 irq;
unsigned long flags;
int r; int r;
int i; int i;
if (!ovl->manager) if (!ovl->manager)
return 0; return 0;
dssdev = ovl->manager->device; mp = get_mgr_priv(ovl->manager);
spin_lock_irqsave(&data_lock, flags);
if (!dssdev || dssdev->state != OMAP_DSS_DISPLAY_ACTIVE) if (ovl_manual_update(ovl)) {
spin_unlock_irqrestore(&data_lock, flags);
return 0; return 0;
}
if (ovl_manual_update(ovl)) if (!mp->enabled) {
spin_unlock_irqrestore(&data_lock, flags);
return 0; return 0;
}
spin_unlock_irqrestore(&data_lock, flags);
r = dispc_runtime_get(); r = dispc_runtime_get();
if (r) if (r)
...@@ -511,7 +526,6 @@ int dss_mgr_wait_for_go_ovl(struct omap_overlay *ovl) ...@@ -511,7 +526,6 @@ int dss_mgr_wait_for_go_ovl(struct omap_overlay *ovl)
op = get_ovl_priv(ovl); op = get_ovl_priv(ovl);
i = 0; i = 0;
while (1) { while (1) {
unsigned long flags;
bool shadow_dirty, dirty; bool shadow_dirty, dirty;
spin_lock_irqsave(&data_lock, flags); spin_lock_irqsave(&data_lock, flags);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment