drm/exynos/fimc: replace mutex by spinlock
authorAndrzej Hajda <a.hajda@samsung.com>
Mon, 19 May 2014 10:54:09 +0000 (12:54 +0200)
committerInki Dae <daeinki@gmail.com>
Sun, 1 Jun 2014 17:07:11 +0000 (02:07 +0900)
Function fimc_dst_set_buf_seq is called by irq handler
so it should not use mutexes. This patch fixes it.

Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Signed-off-by: Inki Dae <inki.dae@samsung.com>
drivers/gpu/drm/exynos/exynos_drm_fimc.c

index 7a66f18..9860d38 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/clk.h>
 #include <linux/pm_runtime.h>
 #include <linux/of.h>
+#include <linux/spinlock.h>
 
 #include <drm/drmP.h>
 #include <drm/exynos_drm.h>
@@ -158,7 +159,7 @@ struct fimc_context {
        struct exynos_drm_ippdrv        ippdrv;
        struct resource *regs_res;
        void __iomem    *regs;
-       struct mutex    lock;
+       spinlock_t      lock;
        struct clk      *clocks[FIMC_CLKS_MAX];
        u32             clk_frequency;
        struct regmap   *sysreg;
@@ -1146,10 +1147,11 @@ static int fimc_dst_set_buf_seq(struct fimc_context *ctx, u32 buf_id,
        u32 cfg;
        u32 mask = 0x00000001 << buf_id;
        int ret = 0;
+       unsigned long flags;
 
        DRM_DEBUG_KMS("buf_id[%d]buf_type[%d]\n", buf_id, buf_type);
 
-       mutex_lock(&ctx->lock);
+       spin_lock_irqsave(&ctx->lock, flags);
 
        /* mask register set */
        cfg = fimc_read(ctx, EXYNOS_CIFCNTSEQ);
@@ -1183,7 +1185,7 @@ static int fimc_dst_set_buf_seq(struct fimc_context *ctx, u32 buf_id,
                fimc_mask_irq(ctx, false);
 
 err_unlock:
-       mutex_unlock(&ctx->lock);
+       spin_unlock_irqrestore(&ctx->lock, flags);
        return ret;
 }
 
@@ -1794,7 +1796,7 @@ static int fimc_probe(struct platform_device *pdev)
 
        DRM_DEBUG_KMS("id[%d]ippdrv[0x%x]\n", ctx->id, (int)ippdrv);
 
-       mutex_init(&ctx->lock);
+       spin_lock_init(&ctx->lock);
        platform_set_drvdata(pdev, ctx);
 
        pm_runtime_set_active(dev);
@@ -1825,7 +1827,6 @@ static int fimc_remove(struct platform_device *pdev)
        struct exynos_drm_ippdrv *ippdrv = &ctx->ippdrv;
 
        exynos_drm_ippdrv_unregister(ippdrv);
-       mutex_destroy(&ctx->lock);
 
        fimc_put_clocks(ctx);
        pm_runtime_set_suspended(dev);