Commit 412d6b47 authored by Stephen Boyd's avatar Stephen Boyd
Browse files

clk: ti: Silence sparse warnings 



drivers/clk/ti/clk.c:125:31: warning: incorrect type in return expression (different address spaces)
drivers/clk/ti/clk.c:125:31:    expected void [noderef] <asn:2>*
drivers/clk/ti/clk.c:125:31:    got void *
drivers/clk/ti/clk.c:132:31: warning: incorrect type in return expression (different address spaces)
drivers/clk/ti/clk.c:132:31:    expected void [noderef] <asn:2>*
drivers/clk/ti/clk.c:132:31:    got void *
drivers/clk/ti/dpll.c:180:14: warning: symbol '_get_reg' was not declared. Should it be static?
drivers/clk/ti/fapll.c:624:32: warning: Using plain integer as NULL pointer
drivers/clk/ti/fapll.c:625:31: warning: Using plain integer as NULL pointer
drivers/clk/ti/fapll.c:630:40: warning: Using plain integer as NULL pointer
drivers/clk/ti/clk-dra7-atl.c:158:22: warning: symbol 'atl_clk_ops' was not declared. Should it be static?
drivers/clk/ti/clk-dra7-atl.c:170:39: warning: Using plain integer as NULL pointer

Acked-by: default avatarPeter Ujfalusi <peter.ujfalusi@ti.com>
Acked-by: default avatarTero Kristo <t-kristo@ti.com>
Signed-off-by: default avatarStephen Boyd <sboyd@codeaurora.org>
parent 8e6dd77c

drivers/clk/ti/clk-dra7-atl.c

+2 −2
Original line number Diff line number Diff line
@@ -155,7 +155,7 @@ static int atl_clk_set_rate(struct clk_hw *hw, unsigned long rate, 
	return 0;

}



const struct clk_ops atl_clk_ops = {

static const struct clk_ops atl_clk_ops = {

	.enable		= atl_clk_enable,

	.disable	= atl_clk_disable,

	.is_enabled	= atl_clk_is_enabled,
@@ -167,7 +167,7 @@ const struct clk_ops atl_clk_ops = { 
static void __init of_dra7_atl_clock_setup(struct device_node *node)

{

	struct dra7_atl_desc *clk_hw = NULL;

	struct clk_init_data init = { 0 };

	struct clk_init_data init = { NULL };

	const char **parent_names = NULL;

	struct clk *clk;

drivers/clk/ti/clk.c

+2 −2
Original line number Diff line number Diff line
@@ -122,14 +122,14 @@ void __iomem *ti_clk_get_reg_addr(struct device_node *node, int index) 


	if (i == CLK_MAX_MEMMAPS) {

		pr_err("clk-provider not found for %s!\n", node->name);

		return ERR_PTR(-ENOENT);

		return IOMEM_ERR_PTR(-ENOENT);

	}



	reg->index = i;



	if (of_property_read_u32_index(node, "reg", index, &val)) {

		pr_err("%s must have reg[%d]!\n", node->name, index);

		return ERR_PTR(-EINVAL);

		return IOMEM_ERR_PTR(-EINVAL);

	}



	reg->offset = val;

drivers/clk/ti/dpll.c

+1 −1
Original line number Diff line number Diff line
@@ -177,7 +177,7 @@ cleanup: 
}



#if defined(CONFIG_ARCH_OMAP3) && defined(CONFIG_ATAGS)

void __iomem *_get_reg(u8 module, u16 offset)

static void __iomem *_get_reg(u8 module, u16 offset)

{

	u32 reg;

	struct clk_omap_reg *reg_setup;

drivers/clk/ti/fapll.c

+3 −3
Original line number Diff line number Diff line
@@ -621,13 +621,13 @@ static void __init ti_fapll_setup(struct device_node *node) 


		/* Check for hardwired audio_pll_clk1 */

		if (is_audio_pll_clk1(freq)) {

			freq = 0;

			div = 0;

			freq = NULL;

			div = NULL;

		} else {

			/* Does the synthesizer have a FREQ register? */

			v = readl_relaxed(freq);

			if (!v)

				freq = 0;

				freq = NULL;

		}

		synth_clk = ti_fapll_synth_setup(fd, freq, div, output_instance,

						 output_name, node->name,

CRA Git | Maintained and supported by SUSTech CRA and CCSE