gma500: power can be touched in IRQ state
authorAlan Cox <alan@linux.intel.com>
Tue, 5 Jul 2011 14:45:13 +0000 (15:45 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 5 Jul 2011 15:20:45 +0000 (08:20 -0700)
So we need to use a spinlock here

Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/gma500/power.c

index 79eb6703a51bd22434f71ed5ca5deb99be2c1156..5cd5023066377d59ebaf39e84fee5c021a3c606f 100644 (file)
@@ -36,7 +36,7 @@
 #include <linux/pm_runtime.h>
 
 static struct mutex power_mutex;       /* Serialize power ops */
-static struct mutex power_ctrl_mutex;  /* Serialize power claim */
+static spinlock_t power_ctrl_lock;     /* Serialize power claim */
 
 /**
  *     gma_power_init          -       initialise power manager
@@ -55,8 +55,8 @@ void gma_power_init(struct drm_device *dev)
        dev_priv->display_power = true; /* We start active */
        dev_priv->display_count = 0;    /* Currently no users */
        dev_priv->suspended = false;    /* And not suspended */
+       spin_lock_init(&power_ctrl_lock);
        mutex_init(&power_mutex);
-       mutex_init(&power_ctrl_mutex);
 
        dev_priv->ops->init_pm(dev);
 }
@@ -247,13 +247,14 @@ bool gma_power_begin(struct drm_device *dev, bool force_on)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
        int ret;
+       unsigned long flags;
 
-       mutex_lock(&power_ctrl_mutex);
+       spin_lock_irqsave(&power_ctrl_lock, flags);
        /* Power already on ? */
        if (dev_priv->display_power) {
                dev_priv->display_count++;
                pm_runtime_get(&dev->pdev->dev);
-               mutex_unlock(&power_ctrl_mutex);
+               spin_unlock_irqrestore(&power_ctrl_lock, flags);
                return true;
        }
        if (force_on == false)
@@ -266,11 +267,11 @@ bool gma_power_begin(struct drm_device *dev, bool force_on)
                psb_irq_postinstall(dev);
                pm_runtime_get(&dev->pdev->dev);
                dev_priv->display_count++;
-               mutex_unlock(&power_ctrl_mutex);
+               spin_unlock_irqrestore(&power_ctrl_lock, flags);
                return true;
        }
 out_false:
-       mutex_unlock(&power_ctrl_mutex);
+       spin_unlock_irqrestore(&power_ctrl_lock, flags);
        return false;
 }
 
@@ -284,10 +285,11 @@ out_false:
 void gma_power_end(struct drm_device *dev)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
-       mutex_lock(&power_ctrl_mutex);
+       unsigned long flags;
+       spin_lock_irqsave(&power_ctrl_lock, flags);
        dev_priv->display_count--;
        WARN_ON(dev_priv->display_count < 0);
-       mutex_unlock(&power_ctrl_mutex);
+       spin_unlock_irqrestore(&power_ctrl_lock, flags);
        pm_runtime_put(&dev->pdev->dev);
 }