[media] exynos-gsc: Simplify system PM
authorUlf Hansson <ulf.hansson@linaro.org>
Wed, 9 Nov 2016 14:23:57 +0000 (12:23 -0200)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Wed, 16 Nov 2016 18:33:52 +0000 (16:33 -0200)
It's not needed to keep a local flag about the current system PM state.
Let's just remove that code and the corresponding debug print.

[mszyprow: rebased onto v4.9-rc4]

Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Tested-by: Javier Martinez Canillas <javier@osg.samsung.com>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/platform/exynos-gsc/gsc-core.c
drivers/media/platform/exynos-gsc/gsc-core.h

index 3ac588f..5aeb8b4 100644 (file)
@@ -1182,20 +1182,6 @@ static int gsc_runtime_suspend(struct device *dev)
 #ifdef CONFIG_PM_SLEEP
 static int gsc_resume(struct device *dev)
 {
-       struct gsc_dev *gsc = dev_get_drvdata(dev);
-       unsigned long flags;
-
-       pr_debug("gsc%d: state: 0x%lx", gsc->id, gsc->state);
-
-       /* Do not resume if the device was idle before system suspend */
-       spin_lock_irqsave(&gsc->slock, flags);
-       if (!test_and_clear_bit(ST_SUSPEND, &gsc->state) ||
-           !gsc_m2m_opened(gsc)) {
-               spin_unlock_irqrestore(&gsc->slock, flags);
-               return 0;
-       }
-       spin_unlock_irqrestore(&gsc->slock, flags);
-
        if (!pm_runtime_suspended(dev))
                return gsc_runtime_resume(dev);
 
@@ -1204,13 +1190,6 @@ static int gsc_resume(struct device *dev)
 
 static int gsc_suspend(struct device *dev)
 {
-       struct gsc_dev *gsc = dev_get_drvdata(dev);
-
-       pr_debug("gsc%d: state: 0x%lx", gsc->id, gsc->state);
-
-       if (test_and_set_bit(ST_SUSPEND, &gsc->state))
-               return 0;
-
        if (!pm_runtime_suspended(dev))
                return gsc_runtime_suspend(dev);
 
index 7ad7b9d..8480aec 100644 (file)
@@ -48,9 +48,6 @@
 #define        GSC_CTX_ABORT                   (1 << 7)
 
 enum gsc_dev_flags {
-       /* for global */
-       ST_SUSPEND,
-
        /* for m2m node */
        ST_M2M_OPEN,
        ST_M2M_RUN,