diff options
author | Paul Gortmaker <paul.gortmaker@windriver.com> | 2014-04-29 15:25:16 -0400 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2014-05-19 20:54:05 -0400 |
commit | 04c32a516806ec74b62048baf4cddcbb840927db (patch) | |
tree | 3d4a71b3c9cf8ad58be643510d475b30792d0493 /arch | |
parent | dade934a5e62f99d3bd8377b2f401f559d827f6f (diff) |
powerpc: Drop return value from set_breakpoint as it is unused
None of the callers check the return value, so it might as
well not have one at all.
Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/powerpc/include/asm/debug.h | 2 | ||||
-rw-r--r-- | arch/powerpc/kernel/process.c | 8 |
2 files changed, 5 insertions, 5 deletions
diff --git a/arch/powerpc/include/asm/debug.h b/arch/powerpc/include/asm/debug.h index d2516308ed1e..1d7f966d3b18 100644 --- a/arch/powerpc/include/asm/debug.h +++ b/arch/powerpc/include/asm/debug.h | |||
@@ -46,7 +46,7 @@ static inline int debugger_break_match(struct pt_regs *regs) { return 0; } | |||
46 | static inline int debugger_fault_handler(struct pt_regs *regs) { return 0; } | 46 | static inline int debugger_fault_handler(struct pt_regs *regs) { return 0; } |
47 | #endif | 47 | #endif |
48 | 48 | ||
49 | int set_breakpoint(struct arch_hw_breakpoint *brk); | 49 | void set_breakpoint(struct arch_hw_breakpoint *brk); |
50 | #ifdef CONFIG_PPC_ADV_DEBUG_REGS | 50 | #ifdef CONFIG_PPC_ADV_DEBUG_REGS |
51 | extern void do_send_trap(struct pt_regs *regs, unsigned long address, | 51 | extern void do_send_trap(struct pt_regs *regs, unsigned long address, |
52 | unsigned long error_code, int signal_code, int brkpt); | 52 | unsigned long error_code, int signal_code, int brkpt); |
diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index 2ae1b99166c6..f895a5062287 100644 --- a/arch/powerpc/kernel/process.c +++ b/arch/powerpc/kernel/process.c | |||
@@ -496,14 +496,14 @@ static inline int set_dawr(struct arch_hw_breakpoint *brk) | |||
496 | return 0; | 496 | return 0; |
497 | } | 497 | } |
498 | 498 | ||
499 | int set_breakpoint(struct arch_hw_breakpoint *brk) | 499 | void set_breakpoint(struct arch_hw_breakpoint *brk) |
500 | { | 500 | { |
501 | __get_cpu_var(current_brk) = *brk; | 501 | __get_cpu_var(current_brk) = *brk; |
502 | 502 | ||
503 | if (cpu_has_feature(CPU_FTR_DAWR)) | 503 | if (cpu_has_feature(CPU_FTR_DAWR)) |
504 | return set_dawr(brk); | 504 | set_dawr(brk); |
505 | 505 | else | |
506 | return set_dabr(brk); | 506 | set_dabr(brk); |
507 | } | 507 | } |
508 | 508 | ||
509 | #ifdef CONFIG_PPC64 | 509 | #ifdef CONFIG_PPC64 |