@@ -42,16 +42,13 @@ DEFINE_MUTEX(mdp_mutex);
static int locked_enable_mdp_irq(struct mdp_info *mdp, uint32_t mask)
{
- int ret = 0;
-
BUG_ON(!mask);
/* if the mask bits are already set return an error, this interrupt
* is already enabled */
if (mdp_irq_mask & mask) {
- printk(KERN_ERR "mdp irq already on already on %x %x\n",
- mdp_irq_mask, mask);
- ret = -1;
+ pr_err("mdp irq already on %x %x\n", mdp_irq_mask, mask);
+ return -1;
}
/* if the mdp irq is not already enabled enable it */
if (!mdp_irq_mask) {
@@ -172,9 +169,9 @@ static int mdp_wait(struct mdp_info *mdp, uint32_t mask, wait_queue_head_t *wq)
spin_lock_irqsave(&mdp->lock, irq_flags);
if (mdp_irq_mask & mask) {
+ pr_warning("%s: timeout waiting for mdp to complete 0x%x\n",
+ __func__, mask);
locked_disable_mdp_irq(mdp, mask);
- printk(KERN_WARNING "timeout waiting for mdp to complete %x\n",
- mask);
ret = -ETIMEDOUT;
}
spin_unlock_irqrestore(&mdp->lock, irq_flags);