diff options
author | Stephen Boyd <sboyd@codeaurora.org> | 2015-05-01 15:59:32 -0400 |
---|---|---|
committer | Stephen Boyd <sboyd@codeaurora.org> | 2015-05-14 20:11:26 -0400 |
commit | 412d6b47cc6e33a04fe03be1db725202417f4d72 (patch) | |
tree | e33059e1bc58e1b33173a3aa1e4274aa07273fea | |
parent | 8e6dd77ccc324a57d5f37c7f8b1e37319fb2e2b3 (diff) |
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: Peter Ujfalusi <peter.ujfalusi@ti.com>
Acked-by: Tero Kristo <t-kristo@ti.com>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
-rw-r--r-- | drivers/clk/ti/clk-dra7-atl.c | 4 | ||||
-rw-r--r-- | drivers/clk/ti/clk.c | 4 | ||||
-rw-r--r-- | drivers/clk/ti/dpll.c | 2 | ||||
-rw-r--r-- | drivers/clk/ti/fapll.c | 6 |
4 files changed, 8 insertions, 8 deletions
diff --git a/drivers/clk/ti/clk-dra7-atl.c b/drivers/clk/ti/clk-dra7-atl.c index 0a1df821860f..19e543a32e2b 100644 --- a/drivers/clk/ti/clk-dra7-atl.c +++ b/drivers/clk/ti/clk-dra7-atl.c | |||
@@ -155,7 +155,7 @@ static int atl_clk_set_rate(struct clk_hw *hw, unsigned long rate, | |||
155 | return 0; | 155 | return 0; |
156 | } | 156 | } |
157 | 157 | ||
158 | const struct clk_ops atl_clk_ops = { | 158 | static const struct clk_ops atl_clk_ops = { |
159 | .enable = atl_clk_enable, | 159 | .enable = atl_clk_enable, |
160 | .disable = atl_clk_disable, | 160 | .disable = atl_clk_disable, |
161 | .is_enabled = atl_clk_is_enabled, | 161 | .is_enabled = atl_clk_is_enabled, |
@@ -167,7 +167,7 @@ const struct clk_ops atl_clk_ops = { | |||
167 | static void __init of_dra7_atl_clock_setup(struct device_node *node) | 167 | static void __init of_dra7_atl_clock_setup(struct device_node *node) |
168 | { | 168 | { |
169 | struct dra7_atl_desc *clk_hw = NULL; | 169 | struct dra7_atl_desc *clk_hw = NULL; |
170 | struct clk_init_data init = { 0 }; | 170 | struct clk_init_data init = { NULL }; |
171 | const char **parent_names = NULL; | 171 | const char **parent_names = NULL; |
172 | struct clk *clk; | 172 | struct clk *clk; |
173 | 173 | ||
diff --git a/drivers/clk/ti/clk.c b/drivers/clk/ti/clk.c index 0ebe5c51062b..64bb5e8a3b8c 100644 --- a/drivers/clk/ti/clk.c +++ b/drivers/clk/ti/clk.c | |||
@@ -122,14 +122,14 @@ void __iomem *ti_clk_get_reg_addr(struct device_node *node, int index) | |||
122 | 122 | ||
123 | if (i == CLK_MAX_MEMMAPS) { | 123 | if (i == CLK_MAX_MEMMAPS) { |
124 | pr_err("clk-provider not found for %s!\n", node->name); | 124 | pr_err("clk-provider not found for %s!\n", node->name); |
125 | return ERR_PTR(-ENOENT); | 125 | return IOMEM_ERR_PTR(-ENOENT); |
126 | } | 126 | } |
127 | 127 | ||
128 | reg->index = i; | 128 | reg->index = i; |
129 | 129 | ||
130 | if (of_property_read_u32_index(node, "reg", index, &val)) { | 130 | if (of_property_read_u32_index(node, "reg", index, &val)) { |
131 | pr_err("%s must have reg[%d]!\n", node->name, index); | 131 | pr_err("%s must have reg[%d]!\n", node->name, index); |
132 | return ERR_PTR(-EINVAL); | 132 | return IOMEM_ERR_PTR(-EINVAL); |
133 | } | 133 | } |
134 | 134 | ||
135 | reg->offset = val; | 135 | reg->offset = val; |
diff --git a/drivers/clk/ti/dpll.c b/drivers/clk/ti/dpll.c index 11478a501c30..2aacf7a3bcae 100644 --- a/drivers/clk/ti/dpll.c +++ b/drivers/clk/ti/dpll.c | |||
@@ -177,7 +177,7 @@ cleanup: | |||
177 | } | 177 | } |
178 | 178 | ||
179 | #if defined(CONFIG_ARCH_OMAP3) && defined(CONFIG_ATAGS) | 179 | #if defined(CONFIG_ARCH_OMAP3) && defined(CONFIG_ATAGS) |
180 | void __iomem *_get_reg(u8 module, u16 offset) | 180 | static void __iomem *_get_reg(u8 module, u16 offset) |
181 | { | 181 | { |
182 | u32 reg; | 182 | u32 reg; |
183 | struct clk_omap_reg *reg_setup; | 183 | struct clk_omap_reg *reg_setup; |
diff --git a/drivers/clk/ti/fapll.c b/drivers/clk/ti/fapll.c index ffcd8e09e85b..730aa62454a2 100644 --- a/drivers/clk/ti/fapll.c +++ b/drivers/clk/ti/fapll.c | |||
@@ -621,13 +621,13 @@ static void __init ti_fapll_setup(struct device_node *node) | |||
621 | 621 | ||
622 | /* Check for hardwired audio_pll_clk1 */ | 622 | /* Check for hardwired audio_pll_clk1 */ |
623 | if (is_audio_pll_clk1(freq)) { | 623 | if (is_audio_pll_clk1(freq)) { |
624 | freq = 0; | 624 | freq = NULL; |
625 | div = 0; | 625 | div = NULL; |
626 | } else { | 626 | } else { |
627 | /* Does the synthesizer have a FREQ register? */ | 627 | /* Does the synthesizer have a FREQ register? */ |
628 | v = readl_relaxed(freq); | 628 | v = readl_relaxed(freq); |
629 | if (!v) | 629 | if (!v) |
630 | freq = 0; | 630 | freq = NULL; |
631 | } | 631 | } |
632 | synth_clk = ti_fapll_synth_setup(fd, freq, div, output_instance, | 632 | synth_clk = ti_fapll_synth_setup(fd, freq, div, output_instance, |
633 | output_name, node->name, | 633 | output_name, node->name, |