diff options
author | Jason Wessel <jason.wessel@windriver.com> | 2010-08-23 10:20:14 -0400 |
---|---|---|
committer | Jason Wessel <jason.wessel@windriver.com> | 2010-10-22 16:34:12 -0400 |
commit | 91b152aa85bbcf076e269565394c31964f940371 (patch) | |
tree | 32b2a8684877a15b050ef21c8ddcd583b695538d /drivers/serial | |
parent | 75d14edee5689716b55afe467acfc13206a31f95 (diff) |
kdb,kgdb: fix sparse fixups
Fix the following sparse warnings:
kdb_main.c:328:5: warning: symbol 'kdbgetu64arg' was not declared. Should it be static?
kgdboc.c:246:12: warning: symbol 'kgdboc_early_init' was not declared. Should it be static?
kgdb.c:652:26: warning: incorrect type in argument 1 (different address spaces)
kgdb.c:652:26: expected void const *ptr
kgdb.c:652:26: got struct perf_event *[noderef] <asn:3>*pev
The one in kgdb.c required the (void * __force) because of the return
code from register_wide_hw_breakpoint looking like:
return (void __percpu __force *)ERR_PTR(err);
Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Diffstat (limited to 'drivers/serial')
-rw-r--r-- | drivers/serial/kgdboc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/serial/kgdboc.c b/drivers/serial/kgdboc.c index 39f9a1adaa75..d4b711c9a416 100644 --- a/drivers/serial/kgdboc.c +++ b/drivers/serial/kgdboc.c | |||
@@ -243,7 +243,7 @@ static struct kgdb_io kgdboc_io_ops = { | |||
243 | 243 | ||
244 | #ifdef CONFIG_KGDB_SERIAL_CONSOLE | 244 | #ifdef CONFIG_KGDB_SERIAL_CONSOLE |
245 | /* This is only available if kgdboc is a built in for early debugging */ | 245 | /* This is only available if kgdboc is a built in for early debugging */ |
246 | int __init kgdboc_early_init(char *opt) | 246 | static int __init kgdboc_early_init(char *opt) |
247 | { | 247 | { |
248 | /* save the first character of the config string because the | 248 | /* save the first character of the config string because the |
249 | * init routine can destroy it. | 249 | * init routine can destroy it. |