Messages in this thread Patch in this message | | | Date | Wed, 22 Nov 2006 05:17:14 +0100 | From | Adrian Bunk <> | Subject | [2.6 patch] include/asm-h8300/: "extern inline" -> "static inline" |
| |
"extern inline" generates a warning with -Wmissing-prototypes and I'm currently working on getting the kernel cleaned up for adding this to the CFLAGS since it will help us to avoid a nasty class of runtime errors.
If there are places that really need a forced inline, __always_inline would be the correct solution.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
include/asm-h8300/delay.h | 4 ++-- include/asm-h8300/mmu_context.h | 4 ++-- include/asm-h8300/pci.h | 4 ++-- include/asm-h8300/tlbflush.h | 4 ++-- 4 files changed, 8 insertions(+), 8 deletions(-)
--- linux-2.6.19-rc5-mm2/include/asm-h8300/delay.h.old 2006-11-22 01:48:48.000000000 +0100 +++ linux-2.6.19-rc5-mm2/include/asm-h8300/delay.h 2006-11-22 01:48:52.000000000 +0100 @@ -9,7 +9,7 @@ * Delay routines, using a pre-computed "loops_per_second" value. */ -extern __inline__ void __delay(unsigned long loops) +static inline void __delay(unsigned long loops) { __asm__ __volatile__ ("1:\n\t" "dec.l #1,%0\n\t" @@ -27,7 +27,7 @@ extern __inline__ void __delay(unsigned extern unsigned long loops_per_jiffy; -extern __inline__ void udelay(unsigned long usecs) +static inline void udelay(unsigned long usecs) { usecs *= 4295; /* 2**32 / 1000000 */ usecs /= (loops_per_jiffy*HZ); --- linux-2.6.19-rc5-mm2/include/asm-h8300/mmu_context.h.old 2006-11-22 01:49:00.000000000 +0100 +++ linux-2.6.19-rc5-mm2/include/asm-h8300/mmu_context.h 2006-11-22 01:49:05.000000000 +0100 @@ -9,7 +9,7 @@ static inline void enter_lazy_tlb(struct { } -extern inline int +static inline int init_new_context(struct task_struct *tsk, struct mm_struct *mm) { // mm->context = virt_to_phys(mm->pgd); @@ -23,7 +23,7 @@ static inline void switch_mm(struct mm_s { } -extern inline void activate_mm(struct mm_struct *prev_mm, +static inline void activate_mm(struct mm_struct *prev_mm, struct mm_struct *next_mm) { } --- linux-2.6.19-rc5-mm2/include/asm-h8300/pci.h.old 2006-11-22 01:49:13.000000000 +0100 +++ linux-2.6.19-rc5-mm2/include/asm-h8300/pci.h 2006-11-22 01:49:19.000000000 +0100 @@ -10,12 +10,12 @@ #define pcibios_assign_all_busses() 0 #define pcibios_scan_all_fns(a, b) 0 -extern inline void pcibios_set_master(struct pci_dev *dev) +static inline void pcibios_set_master(struct pci_dev *dev) { /* No special bus mastering setup handling */ } -extern inline void pcibios_penalize_isa_irq(int irq, int active) +static inline void pcibios_penalize_isa_irq(int irq, int active) { /* We don't do dynamic PCI IRQ allocation */ } --- linux-2.6.19-rc5-mm2/include/asm-h8300/tlbflush.h.old 2006-11-22 01:49:29.000000000 +0100 +++ linux-2.6.19-rc5-mm2/include/asm-h8300/tlbflush.h 2006-11-22 01:49:34.000000000 +0100 @@ -47,12 +47,12 @@ static inline void flush_tlb_range(struc BUG(); } -extern inline void flush_tlb_kernel_page(unsigned long addr) +static inline void flush_tlb_kernel_page(unsigned long addr) { BUG(); } -extern inline void flush_tlb_pgtables(struct mm_struct *mm, +static inline void flush_tlb_pgtables(struct mm_struct *mm, unsigned long start, unsigned long end) { BUG(); - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |