[media] s5p-mfc: Remove dead conditional code
authorMarek Szyprowski <m.szyprowski@samsung.com>
Wed, 16 Nov 2016 09:04:54 +0000 (07:04 -0200)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Wed, 30 Nov 2016 11:19:17 +0000 (09:19 -0200)
CONFIG_PM is always enabled on Exynos platforms, so remove dead code
related to early development of MFC driver on platform without PM support.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/platform/s5p-mfc/s5p_mfc.c
drivers/media/platform/s5p-mfc/s5p_mfc_common.h
drivers/media/platform/s5p-mfc/s5p_mfc_pm.c

index 991649111e77f868b3b2926b88d35b9b60f5fd8a..b34ccbefbb022658cf4637169878e86946cd15b9 100644 (file)
@@ -1405,31 +1405,9 @@ static int s5p_mfc_resume(struct device *dev)
 }
 #endif
 
-#ifdef CONFIG_PM
-static int s5p_mfc_runtime_suspend(struct device *dev)
-{
-       struct platform_device *pdev = to_platform_device(dev);
-       struct s5p_mfc_dev *m_dev = platform_get_drvdata(pdev);
-
-       atomic_set(&m_dev->pm.power, 0);
-       return 0;
-}
-
-static int s5p_mfc_runtime_resume(struct device *dev)
-{
-       struct platform_device *pdev = to_platform_device(dev);
-       struct s5p_mfc_dev *m_dev = platform_get_drvdata(pdev);
-
-       atomic_set(&m_dev->pm.power, 1);
-       return 0;
-}
-#endif
-
 /* Power management */
 static const struct dev_pm_ops s5p_mfc_pm_ops = {
        SET_SYSTEM_SLEEP_PM_OPS(s5p_mfc_suspend, s5p_mfc_resume)
-       SET_RUNTIME_PM_OPS(s5p_mfc_runtime_suspend, s5p_mfc_runtime_resume,
-                          NULL)
 };
 
 static struct s5p_mfc_buf_size_v5 mfc_buf_size_v5 = {
index c068ee3ece6ef48ea6a4ed1a9a0dd0c86283e38f..58b15c212dd2edf8d11344d206538875c27198a2 100644 (file)
@@ -200,7 +200,6 @@ struct s5p_mfc_pm {
        struct clk      *clock;
        struct clk      *clock_gate;
        bool            use_clock_gating;
-       atomic_t        power;
        struct device   *device;
 };
 
index 818c04646061823dcaaabe141ebb46a8cc97c121..11a918eb7564c5ee1b17fdd21cb36adb29934a2c 100644 (file)
 #define MFC_GATE_CLK_NAME      "mfc"
 #define MFC_SCLK_NAME          "sclk_mfc"
 
-#define CLK_DEBUG
-
 static struct s5p_mfc_pm *pm;
 static struct s5p_mfc_dev *p_dev;
-
-#ifdef CLK_DEBUG
 static atomic_t clk_ref;
-#endif
 
 int s5p_mfc_init_pm(struct s5p_mfc_dev *dev)
 {
@@ -64,14 +59,10 @@ int s5p_mfc_init_pm(struct s5p_mfc_dev *dev)
                }
        }
 
-       atomic_set(&pm->power, 0);
-#ifdef CONFIG_PM
        pm->device = &dev->plat_dev->dev;
        pm_runtime_enable(pm->device);
-#endif
-#ifdef CLK_DEBUG
        atomic_set(&clk_ref, 0);
-#endif
+
        return 0;
 
 err_s_clk:
@@ -95,18 +86,16 @@ void s5p_mfc_final_pm(struct s5p_mfc_dev *dev)
        clk_unprepare(pm->clock_gate);
        clk_put(pm->clock_gate);
        pm->clock_gate = NULL;
-#ifdef CONFIG_PM
        pm_runtime_disable(pm->device);
-#endif
 }
 
 int s5p_mfc_clock_on(void)
 {
        int ret = 0;
-#ifdef CLK_DEBUG
+
        atomic_inc(&clk_ref);
        mfc_debug(3, "+ %d\n", atomic_read(&clk_ref));
-#endif
+
        if (!pm->use_clock_gating)
                return 0;
        if (!IS_ERR_OR_NULL(pm->clock_gate))
@@ -116,10 +105,9 @@ int s5p_mfc_clock_on(void)
 
 void s5p_mfc_clock_off(void)
 {
-#ifdef CLK_DEBUG
        atomic_dec(&clk_ref);
        mfc_debug(3, "- %d\n", atomic_read(&clk_ref));
-#endif
+
        if (!pm->use_clock_gating)
                return;
        if (!IS_ERR_OR_NULL(pm->clock_gate))
@@ -130,13 +118,10 @@ int s5p_mfc_power_on(void)
 {
        int ret = 0;
 
-#ifdef CONFIG_PM
        ret = pm_runtime_get_sync(pm->device);
        if (ret)
                return ret;
-#else
-       atomic_set(&pm->power, 1);
-#endif
+
        if (!pm->use_clock_gating && !IS_ERR_OR_NULL(pm->clock_gate))
                ret = clk_enable(pm->clock_gate);
        return ret;
@@ -146,12 +131,7 @@ int s5p_mfc_power_off(void)
 {
        if (!pm->use_clock_gating && !IS_ERR_OR_NULL(pm->clock_gate))
                clk_disable(pm->clock_gate);
-#ifdef CONFIG_PM
        return pm_runtime_put_sync(pm->device);
-#else
-       atomic_set(&pm->power, 0);
-       return 0;
-#endif
 }