| Message ID | 20180225172236.29650-9-malat@debian.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 1cdf039bf82a39d816ca4b8161b01c0acfca3e62 |
| Headers | show |
| Series | powerpc/gamecube: make W=1 compilation errors free | expand |
On Sun, 2018-02-25 at 17:22:23 UTC, Mathieu Malaterre wrote: > When CONFIG_PPC_FPU is defined the prototype exposed in asm/switch_to.h is: > > extern void giveup_fpu(struct task_struct *); > > Change the function prototype of __giveup_fpu to static. Fix warning > (treated as error in W=1): > > CC arch/powerpc/kernel/process.o > arch/powerpc/kernel/process.c:176:6: error: no previous prototype for ‘__giveup_fpu’ [-Werror=missing-prototypes] > void __giveup_fpu(struct task_struct *tsk) > ^~~~~~~~~~~~ > cc1: all warnings being treated as errors > > Signed-off-by: Mathieu Malaterre <malat@debian.org> Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/1cdf039bf82a39d816ca4b8161b01c cheers
diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index 1738c4127b32..3ac98c0463d6 100644 --- a/arch/powerpc/kernel/process.c +++ b/arch/powerpc/kernel/process.c @@ -173,7 +173,7 @@ void __msr_check_and_clear(unsigned long bits) EXPORT_SYMBOL(__msr_check_and_clear); #ifdef CONFIG_PPC_FPU -void __giveup_fpu(struct task_struct *tsk) +static void __giveup_fpu(struct task_struct *tsk) { unsigned long msr;
When CONFIG_PPC_FPU is defined the prototype exposed in asm/switch_to.h is: extern void giveup_fpu(struct task_struct *); Change the function prototype of __giveup_fpu to static. Fix warning (treated as error in W=1): CC arch/powerpc/kernel/process.o arch/powerpc/kernel/process.c:176:6: error: no previous prototype for ‘__giveup_fpu’ [-Werror=missing-prototypes] void __giveup_fpu(struct task_struct *tsk) ^~~~~~~~~~~~ cc1: all warnings being treated as errors Signed-off-by: Mathieu Malaterre <malat@debian.org> --- arch/powerpc/kernel/process.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)