Commit d19f405a authored by Jacek Anaszewski's avatar Jacek Anaszewski Committed by Mauro Carvalho Chehab

[media] s5p-mfc: Fix selective sclk_mfc init

fc906b6d "Remove special clock usage in driver" removed
initialization of MFC special clock, arguing that there's
no need to do it explicitly, since it's one of MFC gate clock's
dependencies and gets enabled along with it. However, there's
no promise of keeping this hierarchy across Exynos SoC
releases, therefore this approach fails to provide a stable,
portable solution.

Out of all MFC versions, only v6 doesn't use special clock at all.
For other versions log a message only in case clk_get fails,
as not all the devices with the same MFC version require
initializing the clock explicitly.
Signed-off-by: default avatarMateusz Zalega <m.zalega@samsung.com>
Signed-off-by: default avatarSeung-Woo Kim <sw0312.kim@samsung.com>
Signed-off-by: default avatarJacek Anaszewski <j.anaszewski@samsung.com>
Signed-off-by: default avatarKyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: default avatarKamil Debski <k.debski@samsung.com>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent 69b9fe22
...@@ -21,6 +21,8 @@ ...@@ -21,6 +21,8 @@
#include "s5p_mfc_pm.h" #include "s5p_mfc_pm.h"
#define MFC_GATE_CLK_NAME "mfc" #define MFC_GATE_CLK_NAME "mfc"
#define MFC_SCLK_NAME "sclk-mfc"
#define MFC_SCLK_RATE (200 * 1000000)
#define CLK_DEBUG #define CLK_DEBUG
...@@ -50,6 +52,20 @@ int s5p_mfc_init_pm(struct s5p_mfc_dev *dev) ...@@ -50,6 +52,20 @@ int s5p_mfc_init_pm(struct s5p_mfc_dev *dev)
goto err_p_ip_clk; goto err_p_ip_clk;
} }
if (dev->variant->version != MFC_VERSION_V6) {
pm->clock = clk_get(&dev->plat_dev->dev, MFC_SCLK_NAME);
if (IS_ERR(pm->clock)) {
mfc_info("Failed to get MFC special clock control\n");
} else {
clk_set_rate(pm->clock, MFC_SCLK_RATE);
ret = clk_prepare_enable(pm->clock);
if (ret) {
mfc_err("Failed to enable MFC special clock\n");
goto err_s_clk;
}
}
}
atomic_set(&pm->power, 0); atomic_set(&pm->power, 0);
#ifdef CONFIG_PM_RUNTIME #ifdef CONFIG_PM_RUNTIME
pm->device = &dev->plat_dev->dev; pm->device = &dev->plat_dev->dev;
...@@ -59,6 +75,9 @@ int s5p_mfc_init_pm(struct s5p_mfc_dev *dev) ...@@ -59,6 +75,9 @@ int s5p_mfc_init_pm(struct s5p_mfc_dev *dev)
atomic_set(&clk_ref, 0); atomic_set(&clk_ref, 0);
#endif #endif
return 0; return 0;
err_s_clk:
clk_put(pm->clock);
err_p_ip_clk: err_p_ip_clk:
clk_put(pm->clock_gate); clk_put(pm->clock_gate);
err_g_ip_clk: err_g_ip_clk:
...@@ -67,6 +86,11 @@ int s5p_mfc_init_pm(struct s5p_mfc_dev *dev) ...@@ -67,6 +86,11 @@ int s5p_mfc_init_pm(struct s5p_mfc_dev *dev)
void s5p_mfc_final_pm(struct s5p_mfc_dev *dev) void s5p_mfc_final_pm(struct s5p_mfc_dev *dev)
{ {
if (dev->variant->version != MFC_VERSION_V6 &&
pm->clock) {
clk_disable_unprepare(pm->clock);
clk_put(pm->clock);
}
clk_unprepare(pm->clock_gate); clk_unprepare(pm->clock_gate);
clk_put(pm->clock_gate); clk_put(pm->clock_gate);
#ifdef CONFIG_PM_RUNTIME #ifdef CONFIG_PM_RUNTIME
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment