diff options
author | Valdis Kletnieks <valdis.kletnieks@vt.edu> | 2019-03-08 03:29:06 +0300 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2019-03-08 05:32:01 +0300 |
commit | 8496ecd0bed4c70b43f39cecf0872b84360f0d14 (patch) | |
tree | 2996f291eb75fe89e3e1a13957529497b2661010 | |
parent | 49ac981965e0032c22e44791a694a83511ebd8fe (diff) | |
download | linux-8496ecd0bed4c70b43f39cecf0872b84360f0d14.tar.xz |
init/calibrate.c: provide proper prototype
Sparse issues a warning:
CHECK init/calibrate.c
init/calibrate.c:271:28: warning: symbol 'calibration_delay_done' was not declared. Should it be static?
The actual issue is that it's a __weak symbol that archs can override
(in fact, ARM does so), but no prototype is provided. Let's provide one
to prevent surprises.
Link: http://lkml.kernel.org/r/18827.1548750938@turing-police.cc.vt.edu
Signed-off-by: Valdis Kletnieks <valdis.kletnieks@vt.edu>
Reviewed-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | include/linux/delay.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/linux/delay.h b/include/linux/delay.h index b78bab4395d8..8e6828094c1e 100644 --- a/include/linux/delay.h +++ b/include/linux/delay.h @@ -55,6 +55,7 @@ static inline void ndelay(unsigned long x) extern unsigned long lpj_fine; void calibrate_delay(void); +void __attribute__((weak)) calibration_delay_done(void); void msleep(unsigned int msecs); unsigned long msleep_interruptible(unsigned int msecs); void usleep_range(unsigned long min, unsigned long max); |