diff options
author | Russell King <rmk@dyn-67.arm.linux.org.uk> | 2008-09-05 09:08:44 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2008-09-05 09:11:24 -0400 |
commit | 09d9bae064724635df3920bcca47e077cfb23e76 (patch) | |
tree | a42fa183dbdbd69854866c0b1f0169f0286b45d4 /arch/arm/kernel/traps.c | |
parent | 22acc4e65043d436bc286e6059960fa533e37356 (diff) |
[ARM] sparse: fix several warnings
arch/arm/kernel/process.c:270:6: warning: symbol 'show_fpregs' was not declared. Should it be static?
This function isn't used, so can be removed.
arch/arm/kernel/setup.c:532:9: warning: symbol 'len' shadows an earlier one
arch/arm/kernel/setup.c:524:6: originally declared here
A function containing two 'len's.
arch/arm/mm/fault-armv.c:188:13: warning: symbol 'check_writebuffer_bugs' was not declared. Should it be static?
arch/arm/mm/mmap.c:122:5: warning: symbol 'valid_phys_addr_range' was not declared. Should it be static?
arch/arm/mm/mmap.c:137:5: warning: symbol 'valid_mmap_phys_addr_range' was not declared. Should it be static?
Missing includes.
arch/arm/kernel/traps.c:71:77: warning: Using plain integer as NULL pointer
arch/arm/mm/ioremap.c:355:46: error: incompatible types in comparison expression (different address spaces)
Sillies.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'arch/arm/kernel/traps.c')
-rw-r--r-- | arch/arm/kernel/traps.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/arm/kernel/traps.c b/arch/arm/kernel/traps.c index 46e2c8315a33..13f8f1860f75 100644 --- a/arch/arm/kernel/traps.c +++ b/arch/arm/kernel/traps.c | |||
@@ -68,7 +68,8 @@ void dump_backtrace_entry(unsigned long where, unsigned long from, unsigned long | |||
68 | */ | 68 | */ |
69 | static int verify_stack(unsigned long sp) | 69 | static int verify_stack(unsigned long sp) |
70 | { | 70 | { |
71 | if (sp < PAGE_OFFSET || (sp > (unsigned long)high_memory && high_memory != 0)) | 71 | if (sp < PAGE_OFFSET || |
72 | (sp > (unsigned long)high_memory && high_memory != NULL)) | ||
72 | return -EFAULT; | 73 | return -EFAULT; |
73 | 74 | ||
74 | return 0; | 75 | return 0; |