diff options
author | Sam Ravnborg <sam@ravnborg.org> | 2014-04-21 15:39:32 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-04-29 01:12:26 -0400 |
commit | 6b5f02f31b5eab6e92060a7396d2773b074e3288 (patch) | |
tree | 9653687fd9b046e1b264674ad2c931ca013b9355 | |
parent | 8885ec7ca9db1bef558c7bb913829b8fa2d7e1d3 (diff) |
sparc32: fix sparse warnings in tadpole.c
Fix following warnings:
tadpole.c:21:6: warning: symbol 'cpu_pwr_save' was not declared. Should it be static?
tadpole.c:101:13: warning: symbol 'clock_stop_probe' was not declared. Should it be static?
This actually revealed that cpu_pwr_save is only assigned.
It was left static with a TODO comment for now - this should be cleaned up later.
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | arch/sparc/kernel/devices.c | 2 | ||||
-rw-r--r-- | arch/sparc/kernel/kernel.h | 3 | ||||
-rw-r--r-- | arch/sparc/kernel/tadpole.c | 6 |
3 files changed, 9 insertions, 2 deletions
diff --git a/arch/sparc/kernel/devices.c b/arch/sparc/kernel/devices.c index b9e9d8c1bd3c..086faf6e5c38 100644 --- a/arch/sparc/kernel/devices.c +++ b/arch/sparc/kernel/devices.c | |||
@@ -21,7 +21,7 @@ | |||
21 | #include <asm/cpu_type.h> | 21 | #include <asm/cpu_type.h> |
22 | #include <asm/setup.h> | 22 | #include <asm/setup.h> |
23 | 23 | ||
24 | extern void clock_stop_probe(void); /* tadpole.c */ | 24 | #include "kernel.h" |
25 | 25 | ||
26 | static char *cpu_mid_prop(void) | 26 | static char *cpu_mid_prop(void) |
27 | { | 27 | { |
diff --git a/arch/sparc/kernel/kernel.h b/arch/sparc/kernel/kernel.h index 7f05d3f65844..487762612224 100644 --- a/arch/sparc/kernel/kernel.h +++ b/arch/sparc/kernel/kernel.h | |||
@@ -113,6 +113,9 @@ asmlinkage int do_sys_sigstack(struct sigstack __user *ssptr, | |||
113 | /* windows.c */ | 113 | /* windows.c */ |
114 | void try_to_clear_window_buffer(struct pt_regs *regs, int who); | 114 | void try_to_clear_window_buffer(struct pt_regs *regs, int who); |
115 | 115 | ||
116 | /* tadpole.c */ | ||
117 | void __init clock_stop_probe(void); | ||
118 | |||
116 | #else /* CONFIG_SPARC32 */ | 119 | #else /* CONFIG_SPARC32 */ |
117 | #endif /* CONFIG_SPARC32 */ | 120 | #endif /* CONFIG_SPARC32 */ |
118 | #endif /* !(__SPARC_KERNEL_H) */ | 121 | #endif /* !(__SPARC_KERNEL_H) */ |
diff --git a/arch/sparc/kernel/tadpole.c b/arch/sparc/kernel/tadpole.c index 9aba8bd5a78b..4bbfe4e2676d 100644 --- a/arch/sparc/kernel/tadpole.c +++ b/arch/sparc/kernel/tadpole.c | |||
@@ -12,13 +12,17 @@ | |||
12 | #include <asm/oplib.h> | 12 | #include <asm/oplib.h> |
13 | #include <asm/io.h> | 13 | #include <asm/io.h> |
14 | 14 | ||
15 | #include "kernel.h" | ||
16 | |||
15 | #define MACIO_SCSI_CSR_ADDR 0x78400000 | 17 | #define MACIO_SCSI_CSR_ADDR 0x78400000 |
16 | #define MACIO_EN_DMA 0x00000200 | 18 | #define MACIO_EN_DMA 0x00000200 |
17 | #define CLOCK_INIT_DONE 1 | 19 | #define CLOCK_INIT_DONE 1 |
18 | 20 | ||
19 | static int clk_state; | 21 | static int clk_state; |
20 | static volatile unsigned char *clk_ctrl; | 22 | static volatile unsigned char *clk_ctrl; |
21 | void (*cpu_pwr_save)(void); | 23 | |
24 | /* TODO - cpu_pwr_save is only assigned - cleanup potential. */ | ||
25 | static void (*cpu_pwr_save)(void); | ||
22 | 26 | ||
23 | static inline unsigned int ldphys(unsigned int addr) | 27 | static inline unsigned int ldphys(unsigned int addr) |
24 | { | 28 | { |