Skip to content

Commit

Permalink
OMAPDSS: APPLY: move spinlock outside the struct
Browse files Browse the repository at this point in the history
dss_cache struct contains a spinlock used to protect the struct. A more
logical place for the spinlock is outside the struct that it is
protecting. So move it there.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
  • Loading branch information
Tomi Valkeinen committed Dec 2, 2011
1 parent af3d64b commit 063fd70
Showing 1 changed file with 12 additions and 10 deletions.
22 changes: 12 additions & 10 deletions drivers/video/omap2/dss/apply.c
Original file line number Diff line number Diff line change
Expand Up @@ -89,13 +89,15 @@ struct mgr_priv_data {
};

static struct {
spinlock_t lock;
struct ovl_priv_data ovl_priv_data_array[MAX_DSS_OVERLAYS];
struct mgr_priv_data mgr_priv_data_array[MAX_DSS_MANAGERS];

bool irq_enabled;
} dss_cache;

/* protects dss_cache */
static spinlock_t data_lock;

static struct ovl_priv_data *get_ovl_priv(struct omap_overlay *ovl)
{
return &dss_cache.ovl_priv_data_array[ovl->id];
Expand All @@ -108,7 +110,7 @@ static struct mgr_priv_data *get_mgr_priv(struct omap_overlay_manager *mgr)

void dss_apply_init(void)
{
spin_lock_init(&dss_cache.lock);
spin_lock_init(&data_lock);
}

static bool ovl_manual_update(struct omap_overlay *ovl)
Expand Down Expand Up @@ -149,10 +151,10 @@ int dss_mgr_wait_for_go(struct omap_overlay_manager *mgr)
unsigned long flags;
bool shadow_dirty, dirty;

spin_lock_irqsave(&dss_cache.lock, flags);
spin_lock_irqsave(&data_lock, flags);
dirty = mp->dirty;
shadow_dirty = mp->shadow_dirty;
spin_unlock_irqrestore(&dss_cache.lock, flags);
spin_unlock_irqrestore(&data_lock, flags);

if (!dirty && !shadow_dirty) {
r = 0;
Expand Down Expand Up @@ -212,10 +214,10 @@ int dss_mgr_wait_for_go_ovl(struct omap_overlay *ovl)
unsigned long flags;
bool shadow_dirty, dirty;

spin_lock_irqsave(&dss_cache.lock, flags);
spin_lock_irqsave(&data_lock, flags);
dirty = op->dirty;
shadow_dirty = op->shadow_dirty;
spin_unlock_irqrestore(&dss_cache.lock, flags);
spin_unlock_irqrestore(&data_lock, flags);

if (!dirty && !shadow_dirty) {
r = 0;
Expand Down Expand Up @@ -464,7 +466,7 @@ static void dss_apply_irq_handler(void *data, u32 mask)
for (i = 0; i < num_mgrs; i++)
mgr_busy[i] = dispc_mgr_go_busy(i);

spin_lock(&dss_cache.lock);
spin_lock(&data_lock);

for (i = 0; i < num_ovls; ++i) {
ovl = omap_dss_get_overlay(i);
Expand Down Expand Up @@ -498,7 +500,7 @@ static void dss_apply_irq_handler(void *data, u32 mask)
dss_unregister_vsync_isr();

end:
spin_unlock(&dss_cache.lock);
spin_unlock(&data_lock);
}

static int omap_dss_mgr_apply_ovl(struct omap_overlay *ovl)
Expand Down Expand Up @@ -620,7 +622,7 @@ int omap_dss_mgr_apply(struct omap_overlay_manager *mgr)
if (r)
return r;

spin_lock_irqsave(&dss_cache.lock, flags);
spin_lock_irqsave(&data_lock, flags);

/* Configure overlays */
list_for_each_entry(ovl, &mgr->overlays, list)
Expand All @@ -641,7 +643,7 @@ int omap_dss_mgr_apply(struct omap_overlay_manager *mgr)
dss_write_regs();
}

spin_unlock_irqrestore(&dss_cache.lock, flags);
spin_unlock_irqrestore(&data_lock, flags);

dispc_runtime_put();

Expand Down

0 comments on commit 063fd70

Please sign in to comment.