Commit d5e136a2 authored by Sylwester Nawrocki's avatar Sylwester Nawrocki Committed by Tomasz Figa

clk: samsung: Register clk provider only after registering its all clocks

Ensure the clock provider is not registered until after all its related
clocks were created and are ready to use. Currently there are races
possible and any (of_)clk_get() call right after a clock provider's
clk_init_cb callback call may fail.
Signed-off-by: default avatarSylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: default avatarTomasz Figa <t.figa@samsung.com>
parent bdfcdf18
...@@ -776,5 +776,7 @@ static void __init exynos3250_cmu_init(struct device_node *np) ...@@ -776,5 +776,7 @@ static void __init exynos3250_cmu_init(struct device_node *np)
samsung_clk_register_gate(ctx, gate_clks, ARRAY_SIZE(gate_clks)); samsung_clk_register_gate(ctx, gate_clks, ARRAY_SIZE(gate_clks));
exynos3250_clk_sleep_init(); exynos3250_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
} }
CLK_OF_DECLARE(exynos3250_cmu, "samsung,exynos3250-cmu", exynos3250_cmu_init); CLK_OF_DECLARE(exynos3250_cmu, "samsung,exynos3250-cmu", exynos3250_cmu_init);
...@@ -1252,6 +1252,8 @@ static void __init exynos4_clk_init(struct device_node *np, ...@@ -1252,6 +1252,8 @@ static void __init exynos4_clk_init(struct device_node *np,
exynos4_clk_sleep_init(); exynos4_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
pr_info("%s clocks: sclk_apll = %ld, sclk_mpll = %ld\n" pr_info("%s clocks: sclk_apll = %ld, sclk_mpll = %ld\n"
"\tsclk_epll = %ld, sclk_vpll = %ld, arm_clk = %ld\n", "\tsclk_epll = %ld, sclk_vpll = %ld, arm_clk = %ld\n",
exynos4_soc == EXYNOS4210 ? "Exynos4210" : "Exynos4x12", exynos4_soc == EXYNOS4210 ? "Exynos4210" : "Exynos4x12",
......
...@@ -820,6 +820,8 @@ static void __init exynos5250_clk_init(struct device_node *np) ...@@ -820,6 +820,8 @@ static void __init exynos5250_clk_init(struct device_node *np)
exynos5250_clk_sleep_init(); exynos5250_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
pr_info("Exynos5250: clock setup completed, armclk=%ld\n", pr_info("Exynos5250: clock setup completed, armclk=%ld\n",
_get_rate("div_arm2")); _get_rate("div_arm2"));
} }
......
...@@ -206,6 +206,8 @@ void __init exynos5260_cmu_register_one(struct device_node *np, ...@@ -206,6 +206,8 @@ void __init exynos5260_cmu_register_one(struct device_node *np,
if (cmu->clk_regs) if (cmu->clk_regs)
exynos5260_clk_sleep_init(reg_base, cmu->clk_regs, exynos5260_clk_sleep_init(reg_base, cmu->clk_regs,
cmu->nr_clk_regs); cmu->nr_clk_regs);
samsung_clk_of_add_provider(np, ctx);
} }
......
...@@ -204,6 +204,8 @@ static void __init exynos5410_clk_init(struct device_node *np) ...@@ -204,6 +204,8 @@ static void __init exynos5410_clk_init(struct device_node *np)
samsung_clk_register_gate(ctx, exynos5410_gate_clks, samsung_clk_register_gate(ctx, exynos5410_gate_clks,
ARRAY_SIZE(exynos5410_gate_clks)); ARRAY_SIZE(exynos5410_gate_clks));
samsung_clk_of_add_provider(np, ctx);
pr_debug("Exynos5410: clock setup completed.\n"); pr_debug("Exynos5410: clock setup completed.\n");
} }
CLK_OF_DECLARE(exynos5410_clk, "samsung,exynos5410-clock", exynos5410_clk_init); CLK_OF_DECLARE(exynos5410_clk, "samsung,exynos5410-clock", exynos5410_clk_init);
...@@ -1251,6 +1251,8 @@ static void __init exynos5x_clk_init(struct device_node *np, ...@@ -1251,6 +1251,8 @@ static void __init exynos5x_clk_init(struct device_node *np,
} }
exynos5420_clk_sleep_init(); exynos5420_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
} }
static void __init exynos5420_clk_init(struct device_node *np) static void __init exynos5420_clk_init(struct device_node *np)
......
...@@ -123,6 +123,8 @@ static void __init exynos5440_clk_init(struct device_node *np) ...@@ -123,6 +123,8 @@ static void __init exynos5440_clk_init(struct device_node *np)
samsung_clk_register_gate(ctx, exynos5440_gate_clks, samsung_clk_register_gate(ctx, exynos5440_gate_clks,
ARRAY_SIZE(exynos5440_gate_clks)); ARRAY_SIZE(exynos5440_gate_clks));
samsung_clk_of_add_provider(np, ctx);
pr_info("Exynos5440: arm_clk = %ldHz\n", _get_rate("arm_clk")); pr_info("Exynos5440: arm_clk = %ldHz\n", _get_rate("arm_clk"));
pr_info("exynos5440 clock initialization complete\n"); pr_info("exynos5440 clock initialization complete\n");
} }
......
...@@ -466,6 +466,8 @@ void __init s3c2410_common_clk_init(struct device_node *np, unsigned long xti_f, ...@@ -466,6 +466,8 @@ void __init s3c2410_common_clk_init(struct device_node *np, unsigned long xti_f,
} }
s3c2410_clk_sleep_init(); s3c2410_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
} }
static void __init s3c2410_clk_init(struct device_node *np) static void __init s3c2410_clk_init(struct device_node *np)
......
...@@ -265,6 +265,8 @@ void __init s3c2412_common_clk_init(struct device_node *np, unsigned long xti_f, ...@@ -265,6 +265,8 @@ void __init s3c2412_common_clk_init(struct device_node *np, unsigned long xti_f,
ARRAY_SIZE(s3c2412_aliases)); ARRAY_SIZE(s3c2412_aliases));
s3c2412_clk_sleep_init(); s3c2412_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
} }
static void __init s3c2412_clk_init(struct device_node *np) static void __init s3c2412_clk_init(struct device_node *np)
......
...@@ -445,6 +445,8 @@ void __init s3c2443_common_clk_init(struct device_node *np, unsigned long xti_f, ...@@ -445,6 +445,8 @@ void __init s3c2443_common_clk_init(struct device_node *np, unsigned long xti_f,
} }
s3c2443_clk_sleep_init(); s3c2443_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
} }
static void __init s3c2416_clk_init(struct device_node *np) static void __init s3c2416_clk_init(struct device_node *np)
......
...@@ -518,6 +518,8 @@ void __init s3c64xx_clk_init(struct device_node *np, unsigned long xtal_f, ...@@ -518,6 +518,8 @@ void __init s3c64xx_clk_init(struct device_node *np, unsigned long xtal_f,
ARRAY_SIZE(s3c64xx_clock_aliases)); ARRAY_SIZE(s3c64xx_clock_aliases));
s3c64xx_clk_sleep_init(); s3c64xx_clk_sleep_init();
samsung_clk_of_add_provider(np, ctx);
pr_info("%s clocks: apll = %lu, mpll = %lu\n" pr_info("%s clocks: apll = %lu, mpll = %lu\n"
"\tepll = %lu, arm_clk = %lu\n", "\tepll = %lu, arm_clk = %lu\n",
is_s3c6400 ? "S3C6400" : "S3C6410", is_s3c6400 ? "S3C6400" : "S3C6410",
......
...@@ -53,7 +53,6 @@ struct samsung_clk_provider *__init samsung_clk_init(struct device_node *np, ...@@ -53,7 +53,6 @@ struct samsung_clk_provider *__init samsung_clk_init(struct device_node *np,
{ {
struct samsung_clk_provider *ctx; struct samsung_clk_provider *ctx;
struct clk **clk_table; struct clk **clk_table;
int ret;
int i; int i;
ctx = kzalloc(sizeof(struct samsung_clk_provider), GFP_KERNEL); ctx = kzalloc(sizeof(struct samsung_clk_provider), GFP_KERNEL);
...@@ -72,17 +71,19 @@ struct samsung_clk_provider *__init samsung_clk_init(struct device_node *np, ...@@ -72,17 +71,19 @@ struct samsung_clk_provider *__init samsung_clk_init(struct device_node *np,
ctx->clk_data.clk_num = nr_clks; ctx->clk_data.clk_num = nr_clks;
spin_lock_init(&ctx->lock); spin_lock_init(&ctx->lock);
if (!np)
return ctx;
ret = of_clk_add_provider(np, of_clk_src_onecell_get,
&ctx->clk_data);
if (ret)
panic("could not register clock provide\n");
return ctx; return ctx;
} }
void __init samsung_clk_of_add_provider(struct device_node *np,
struct samsung_clk_provider *ctx)
{
if (np) {
if (of_clk_add_provider(np, of_clk_src_onecell_get,
&ctx->clk_data))
panic("could not register clk provider\n");
}
}
/* add a clock instance to the clock lookup table used for dt based lookup */ /* add a clock instance to the clock lookup table used for dt based lookup */
void samsung_clk_add_lookup(struct samsung_clk_provider *ctx, struct clk *clk, void samsung_clk_add_lookup(struct samsung_clk_provider *ctx, struct clk *clk,
unsigned int id) unsigned int id)
......
...@@ -327,6 +327,8 @@ struct samsung_pll_clock { ...@@ -327,6 +327,8 @@ struct samsung_pll_clock {
extern struct samsung_clk_provider *__init samsung_clk_init( extern struct samsung_clk_provider *__init samsung_clk_init(
struct device_node *np, void __iomem *base, struct device_node *np, void __iomem *base,
unsigned long nr_clks); unsigned long nr_clks);
extern void __init samsung_clk_of_add_provider(struct device_node *np,
struct samsung_clk_provider *ctx);
extern void __init samsung_clk_of_register_fixed_ext( extern void __init samsung_clk_of_register_fixed_ext(
struct samsung_clk_provider *ctx, struct samsung_clk_provider *ctx,
struct samsung_fixed_rate_clock *fixed_rate_clk, struct samsung_fixed_rate_clock *fixed_rate_clk,
......
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