Commit 11bee5e1 authored May 01, 2015 by Stephen Boyd

clk: versatile: Silence sparse warnings

drivers/clk/versatile/clk-sp810.c:159:29: error: incompatible types for operation (<=)
drivers/clk/versatile/clk-sp810.c:159:29: left side has type char const *<noident>
drivers/clk/versatile/clk-sp810.c:159:29: right side has type int
drivers/clk/versatile/clk-sp810.c:159:53: error: incompatible types for operation (<=)
drivers/clk/versatile/clk-sp810.c:159:53: left side has type char const *<noident>
drivers/clk/versatile/clk-sp810.c:159:53: right side has type int
drivers/clk/versatile/clk-sp810.c:138:13: warning: symbol 'clk_sp810_of_setup' was not declared. Should it be static?

Acked: Pawel Moll <pawel.moll@arm.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>

parent 412d6b47

drivers/clk/versatile/clk-sp810.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -135,7 +135,7 @@ static struct clk clk_sp810_timerclken_of_get(struct of_phandle_args clkspec,
		return sp810->timerclken[clkspec->args[0]].clk;
		}

		void __init clk_sp810_of_setup(struct device_node *node)
		static void __init clk_sp810_of_setup(struct device_node *node)
		{
		struct clk_sp810 sp810 = kzalloc(sizeof(sp810), GFP_KERNEL);
		const char *parent_names[2];
		@@ -156,7 +156,7 @@ void __init clk_sp810_of_setup(struct device_node *node)
		"timclk");
		parent_names[1] = of_clk_get_parent_name(node, sp810->timclk_index);

		if (parent_names[0] <= 0 \|\| parent_names[1] <= 0) {
		if (!parent_names[0] \|\| !parent_names[1]) {
		pr_warn("Failed to obtain parent clocks for SP810!\n");
		return;
		}