add -Werror-implicit-function-declaration to CFLAGS
Subsystems (6)
PATCH 1/1 add -Werror-implicit-function-declaration to CFLAGS
Currently, using an undeclared function gives a compile warning, but it can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.
With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.
There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.
This patch also removes some unneeded spaces between two tabs in the following line.
---
This patch was already sent on:
- 30 Aug 2005
- 30 Jul 2005
Makefile renamed from Makefile.old +2 -1
@@ -351,7 +351,8 @@ CPPFLAGS := -D__KERNEL__ $(LINUXINCLUDE) CFLAGS := -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \ - -fno-strict-aliasing -fno-common \ + -Werror-implicit-function-declaration \ + -fno-strict-aliasing -fno-common \ -ffreestanding AFLAGS := -D__ASSEMBLY__
Quoted 5 lines · Adrian Bunk
Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.
Quoted 7 lines
With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.
There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.
Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.
Quoted 9 lines · Andrew Morton
Quoted 8 lines · Adrian Bunk
Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.
Quoted 7 lines
Quoted 4 lines
There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.
Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.
Can you send me the list of compile errors so that I can work on fixing them?
TIA
Adrian
Signature · 4 lines
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon SeedQuoted 16 lines · Adrian Bunk
Quoted 14 lines · Andrew Morton
Quoted 12 lines · Adrian Bunk
Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.
There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.
Quoted 6 lines
Quoted 2 lines
Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.
Can you send me the list of compile errors so that I can work on fixing them?
No handily, sorry. Missing virt_to_bus() is the typical problem.
The cross-tools at http://developer.osdl.org/dev/plm/cross_compile/ are quite simple to install.
Quoted 20 lines · On Thu, 10 Nov 2005 18:24:43 -0800 Andrew Morton wrote:
Quoted 19 lines · Adrian Bunk
Quoted 16 lines · Andrew Morton
Quoted 5 lines · Adrian Bunk
Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.Quoted 7 lines
With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.
There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.
Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.
Quoted 8 lines
Quoted 3 lines
Can you send me the list of compile errors so that I can work on fixing them?
No handily, sorry. Missing
virt_to_bus()is the typical problem.The cross-tools at http://developer.osdl.org/dev/plm/cross_compile/ are quite simple to install.
Ack that. Those are the ones I use.
---
~Randy
Quoted 17 lines · Andrew Morton
Quoted 16 lines · Adrian Bunk
Quoted 14 lines · Andrew Morton
Quoted 12 lines · Adrian Bunk
Currently, using an undeclared function gives a compile warning, but it
can lead to a nasty runtime error if the prototype of the function is different from what gcc guessed.With -Werror-implicit-function-declaration, we are getting an immediate compile error instead.
There will be some compile errors in cases where compilation previously worked because the undefined function wasn't called due to gcc dead code elimination, but in these cases a proper fix doesnt harm.
Quoted 9 lines
Quoted 6 lines
Quoted 2 lines
Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.
Can you send me the list of compile errors so that I can work on fixing them?
No handily, sorry. Missing
virt_to_bus()is the typical problem.
But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.
If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned long virt_to_bus_not_defined_use_pci_map(volatile void *addr); #define virt_to_bus virt_to_bus_not_defined_use_pci_map
extern unsigned long bus_to_virt_not_defined_use_pci_map(volatile void *addr); #define bus_to_virt bus_to_virt_not_defined_use_pci_map
Would a patch to mark the ISA legacy functions as __deprecated be OK?
This might give some motivation for people to convert drivers and would avoid new code like the recently introduced kexec to use this obsolete API.
Quoted 2 lines
The cross-tools at http://developer.osdl.org/dev/plm/cross_compile/ are quite simple to install.
Thanks, I've tried it.
Other problems I found until I gave up on compiling:
- a problem in sk98lin indirectly corrected by my SkPciWriteCfgDWord() patch
- drivers/net/wireless/tiacx/: missing #include <linux/vmalloc.h>'s (see my patch) - this seems to be a real bug
cu
Adrian
Signature · 4 lines
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon SeedQuoted 12 lines · Adrian Bunk
...
But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned longvirt_to_bus_not_defined_use_pci_map(volatile void *addr); #definevirt_to_busvirt_to_bus_not_defined_use_pci_map
extern unsigned longbus_to_virt_not_defined_use_pci_map(volatile void *addr); #definebus_to_virtbus_to_virt_not_defined_use_pci_map
Quoted 2 lines
Would a patch to mark the ISA legacy functions as
__deprecatedbe OK?
...
Sorry, this were two separate thoughts:
Would a patch to mark both virt_to_bus/bus_to_virt and the ISA legacy functions (that cause similar problems) as __deprecated be OK?
cu
Adrian
Signature · 4 lines
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon SeedQuoted 13 lines · Adrian Bunk
Quoted 12 lines · Adrian Bunk
...
But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned longvirt_to_bus_not_defined_use_pci_map(volatile void *addr); #definevirt_to_busvirt_to_bus_not_defined_use_pci_map
extern unsigned longbus_to_virt_not_defined_use_pci_map(volatile void *addr); #definebus_to_virtbus_to_virt_not_defined_use_pci_map
Quoted 7 lines
Quoted 2 lines
Would a patch to mark the ISA legacy functions as
__deprecatedbe OK?
...Sorry, this were two separate thoughts:
Would a patch to mark both virt_to_bus/bus_to_virt and the ISA legacy functions (that cause similar problems) as
__deprecatedbe OK?
Rethinking about this:
The ISA legacy functions were declared obsolete five years ago. Since there are not many drivers using them, we could simply mark the drivers still using them as BROKEN.
cu
Adrian
Signature · 4 lines
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon SeedRFC PATCH 1/1 remove ISA legacy functions
This patch removes the ISA legacy functions that are deprecated since kernel 2.4 and that aren't available on all architectures.
The 7 drivers that were still using this obsolete API are now marked as BROKEN.
---
Documentation/DocBook/deviceiobook.tmpl | 19 ----- arch/i386/Kconfig | 1 drivers/net/Kconfig | 6 - drivers/net/arcnet/Kconfig | 4 - drivers/scsi/Kconfig | 2 include/asm-alpha/io.h | 83 ------------------------ include/asm-arm/arch-aaec2000/io.h | 1 include/asm-arm/arch-clps711x/io.h | 1 include/asm-arm/arch-ebsa285/io.h | 8 -- include/asm-arm/arch-integrator/io.h | 1 include/asm-arm/arch-iop3xx/io.h | 1 include/asm-arm/arch-l7200/io.h | 1 include/asm-arm/arch-lh7a40x/io.h | 1 include/asm-arm/arch-omap/io.h | 1 include/asm-arm/arch-pxa/io.h | 1 include/asm-arm/arch-realview/io.h | 1 include/asm-arm/arch-sa1100/io.h | 1 include/asm-arm/arch-versatile/io.h | 1 include/asm-arm/io.h | 36 ---------- include/asm-i386/io.h | 11 --- include/asm-m68k/io.h | 21 ++---- include/asm-mips/io.h | 23 ------ include/asm-parisc/io.h | 18 ----- include/asm-sh/io.h | 13 --- include/asm-x86_64/io.h | 11 --- 25 files changed, 15 insertions(+), 252 deletions(-)
Documentation/DocBook/deviceiobook.tmpl renamed from Documentation/DocBook/deviceiobook.tmpl.old +0 -19
@@ -270,25 +270,6 @@ </para> </sect1> - <sect1> - <title>ISA legacy functions</title> - <para> - On older kernels (2.2 and earlier) the ISA bus could be read or - written with these functions and without ioremap being used. This is - no longer true in Linux 2.4. A set of equivalent functions exist for - easy legacy driver porting. The functions available are prefixed - with 'isa_' and are <function>isa_readb</function>, - <function>isa_writeb</function>, <function>isa_readw</function>, - <function>isa_writew</function>, <function>isa_readl</function>, - <function>isa_writel</function>, <function>isa_memcpy_fromio</function> - and <function>isa_memcpy_toio</function> - </para> - <para> - These functions should not be used in new drivers, and will - eventually be going away. - </para> - </sect1> - </chapter> <chapter>
include/asm-alpha/io.h renamed from include/asm-alpha/io.h.old +0 -83
95 lines not on this page · raw message
include/asm-arm/io.h renamed from include/asm-arm/io.h.old +0 -36
@@ -215,42 +215,6 @@#endif /* __mem_pci */ /* -* If this architecture has ISA IO, then define the isa_read/isa_write -* macros. -*/ -#ifdef __mem_isa - -#define isa_readb (addr) __raw_readb(__mem_isa(addr)) -#define isa_readw (addr) __raw_readw(__mem_isa(addr)) -#define isa_readl (addr) __raw_readl(__mem_isa(addr)) -#define isa_writeb (val,addr) __raw_writeb(val,__mem_isa(addr)) -#define isa_writew (val,addr) __raw_writew(val,__mem_isa(addr)) -#define isa_writel (val,addr) __raw_writel(val,__mem_isa(addr)) -#define isa_memset_io (a,b,c) _memset_io(__mem_isa(a),(b),(c)) -#define isa_memcpy_fromio (a,b,c) _memcpy_fromio((a),__mem_isa(b),(c)) -#define isa_memcpy_toio (a,b,c) _memcpy_toio(__mem_isa((a)),(b),(c)) - -#define isa_eth_io_copy_and_sum (a,b,c,d) \ - eth_copy_and_sum((a),__mem_isa(b),(c),(d)) - -#else /* __mem_isa */ - -#define isa_readb (addr) (__readwrite_bug("isa_readb"),0) -#define isa_readw (addr) (__readwrite_bug("isa_readw"),0) -#define isa_readl (addr) (__readwrite_bug("isa_readl"),0) -#define isa_writeb (val,addr) __readwrite_bug("isa_writeb") -#define isa_writew (val,addr) __readwrite_bug("isa_writew") -#define isa_writel (val,addr) __readwrite_bug("isa_writel") -#define isa_memset_io (a,b,c) __readwrite_bug("isa_memset_io") -#define isa_memcpy_fromio (a,b,c) __readwrite_bug("isa_memcpy_fromio") -#define isa_memcpy_toio (a,b,c) __readwrite_bug("isa_memcpy_toio") - -#define isa_eth_io_copy_and_sum (a,b,c,d) \ - __readwrite_bug("isa_eth_io_copy_and_sum") - -#endif /* __mem_isa */ - -/* * ioremap and friends. * * ioremap takes a PCI memory address, as specified in
include/asm-i386/io.h renamed from include/asm-i386/io.h.old +0 -11
@@ -214,23 +214,12 @@*/ #define __ISA_IO_base ((char __iomem *)(PAGE_OFFSET)) -#define isa_readb (a) readb(__ISA_IO_base + (a)) -#define isa_readw (a) readw(__ISA_IO_base + (a)) -#define isa_readl (a) readl(__ISA_IO_base + (a)) -#define isa_writeb (b,a) writeb(b,__ISA_IO_base + (a)) -#define isa_writew (w,a) writew(w,__ISA_IO_base + (a)) -#define isa_writel (l,a) writel(l,__ISA_IO_base + (a)) -#define isa_memset_io (a,b,c) memset_io(__ISA_IO_base + (a),(b),(c)) -#define isa_memcpy_fromio (a,b,c) memcpy_fromio((a),__ISA_IO_base + (b),(c)) -#define isa_memcpy_toio (a,b,c) memcpy_toio(__ISA_IO_base + (a),(b),(c)) -/* * Again, i386 does not require mem IO specific function. */ #define eth_io_copy_and_sum (a,b,c,d) eth_copy_and_sum((a),(void __force *)(b),(c),(d)) -#define isa_eth_io_copy_and_sum (a,b,c,d) eth_copy_and_sum((a),(void __force *)(__ISA_IO_base + (b)),(c),(d))/** * check_signature - find BIOS signatures
include/asm-mips/io.h renamed from include/asm-mips/io.h.old +0 -23
@@ -563,33 +563,10 @@extern void pci_iounmap (struct pci_dev *dev,void __iomem *);/* -* ISA space is 'always mapped' on currently supported MIPS systems, no need -* to explicitly ioremap() it. The fact that the ISA IO space is mapped -* to PAGE_OFFSET is pure coincidence - it does not mean ISA values -* are physical addresses. The following constant pointer can be -* used as the IO-area pointer (it can be iounmapped as well, so the -* analogy with PCI is quite large): -*/ -#define __ISA_IO_base ((char *)(isa_slot_offset)) - -#define isa_readb (a) readb(__ISA_IO_base + (a)) -#define isa_readw (a) readw(__ISA_IO_base + (a)) -#define isa_readl (a) readl(__ISA_IO_base + (a)) -#define isa_readq (a) readq(__ISA_IO_base + (a)) -#define isa_writeb (b,a) writeb(b,__ISA_IO_base + (a)) -#define isa_writew (w,a) writew(w,__ISA_IO_base + (a)) -#define isa_writel (l,a) writel(l,__ISA_IO_base + (a)) -#define isa_writeq (q,a) writeq(q,__ISA_IO_base + (a)) -#define isa_memset_io (a,b,c) memset_io(__ISA_IO_base + (a),(b),(c)) -#define isa_memcpy_fromio (a,b,c) memcpy_fromio((a),__ISA_IO_base + (b),(c)) -#define isa_memcpy_toio (a,b,c) memcpy_toio(__ISA_IO_base + (a),(b),(c)) - -/* * We don't have csum_partial_copy_fromio() yet, so we cheat here and * just copy it. The net code will then do the checksum later. */ #define eth_io_copy_and_sum (skb,src,len,unused) memcpy_fromio((skb)->data,(src),(len)) -#define isa_eth_io_copy_and_sum (a,b,c,d) eth_copy_and_sum((a),(b),(c),(d))/* * check_signature - find BIOS signatures
include/asm-parisc/io.h renamed from include/asm-parisc/io.h.old +0 -18
@@ -294,22 +294,6 @@void memcpy_fromio (void *dst,const volatile void __iomem *src,int count);void memcpy_toio (volatile void __iomem *dst,const void *src,int count); -/* Support old drivers which don't ioremap. -* NB this interface is scheduled to disappear in 2.5 -*/ - -#define __isa_addr (x) (void __iomem *)(F_EXTEND(0xfc000000) | (x)) -#define isa_readb (a) readb(__isa_addr(a)) -#define isa_readw (a) readw(__isa_addr(a)) -#define isa_readl (a) readl(__isa_addr(a)) -#define isa_writeb (b,a) writeb((b), __isa_addr(a)) -#define isa_writew (b,a) writew((b), __isa_addr(a)) -#define isa_writel (b,a) writel((b), __isa_addr(a)) -#define isa_memset_io (a,b,c) memset_io(__isa_addr(a), (b), (c)) -#define isa_memcpy_fromio (a,b,c) memcpy_fromio((a), __isa_addr(b), (c)) -#define isa_memcpy_toio (a,b,c) memcpy_toio(__isa_addr(a), (b), (c)) - -/* * XXX - We don't have csum_partial_copy_fromio() yet, so we cheat here and * just copy it. The net code will then do the checksum later. Presently
@@ -318,8 +302,6 @@#define eth_io_copy_and_sum (skb,src,len,unused) \ memcpy_fromio((skb)->data,(src),(len)) -#define isa_eth_io_copy_and_sum (skb,src,len,unused) \ - isa_memcpy_fromio((skb)->data,(src),(len))/* Port-space IO */
include/asm-x86_64/io.h renamed from include/asm-x86_64/io.h.old +0 -11
@@ -264,23 +264,12 @@*/ #define __ISA_IO_base ((char __iomem *)(PAGE_OFFSET)) -#define isa_readb (a) readb(__ISA_IO_base + (a)) -#define isa_readw (a) readw(__ISA_IO_base + (a)) -#define isa_readl (a) readl(__ISA_IO_base + (a)) -#define isa_writeb (b,a) writeb(b,__ISA_IO_base + (a)) -#define isa_writew (w,a) writew(w,__ISA_IO_base + (a)) -#define isa_writel (l,a) writel(l,__ISA_IO_base + (a)) -#define isa_memset_io (a,b,c) memset_io(__ISA_IO_base + (a),(b),(c)) -#define isa_memcpy_fromio (a,b,c) memcpy_fromio((a),__ISA_IO_base + (b),(c)) -#define isa_memcpy_toio (a,b,c) memcpy_toio(__ISA_IO_base + (a),(b),(c)) -/* * Again, x86-64 does not require mem IO specific function. */ #define eth_io_copy_and_sum (a,b,c,d) eth_copy_and_sum((a),(void *)(b),(c),(d)) -#define isa_eth_io_copy_and_sum (a,b,c,d) eth_copy_and_sum((a),(void *)(__ISA_IO_base + (b)),(c),(d))/** * check_signature - find BIOS signatures
include/asm-m68k/io.h renamed from include/asm-m68k/io.h.old +8 -13
@@ -191,15 +191,6 @@#define isa_outb (val,port) out_8(isa_itb(port),(val))#define isa_outw (val,port) (ISA_SEX ? out_be16(isa_itw(port),(val)) : out_le16(isa_itw(port),(val))) -#define isa_readb (p) in_8(isa_mtb((unsigned long)(p))) -#define isa_readw (p) \ - (ISA_SEX ? in_be16(isa_mtw((unsigned long)(p))) \ - : in_le16(isa_mtw((unsigned long)(p)))) -#define isa_writeb (val,p) out_8(isa_mtb((unsigned long)(p)),(val)) -#define isa_writew (val,p) \ - (ISA_SEX ? out_be16(isa_mtw((unsigned long)(p)),(val)) \ - : out_le16(isa_mtw((unsigned long)(p)),(val))) -static inline void isa_delay (void ) { switch(ISA_TYPE)
@@ -254,10 +245,14 @@#define insw isa_insw#define outsb isa_outsb#define outsw isa_outsw -#define readb isa_readb -#define readw isa_readw -#define writeb isa_writeb -#define writew isa_writew +#define readb (p) in_8(isa_mtb((unsigned long)(p))) +#define readw (p) \ + (ISA_SEX ? in_be16(isa_mtw((unsigned long)(p))) \ + : in_le16(isa_mtw((unsigned long)(p)))) +#define writeb (val,p) out_8(isa_mtb((unsigned long)(p)),(val)) +#define writew (val,p) \ + (ISA_SEX ? out_be16(isa_mtw((unsigned long)(p)),(val)) \ + : out_le16(isa_mtw((unsigned long)(p)),(val)))#endif /* CONFIG_ISA */ #if defined(CONFIG_PCI )
include/asm-sh/io.h renamed from include/asm-sh/io.h.old +0 -13
@@ -159,19 +159,6 @@ generic_io_base = pbase; } -#define isa_readb (a) readb(isa_port2addr(a)) -#define isa_readw (a) readw(isa_port2addr(a)) -#define isa_readl (a) readl(isa_port2addr(a)) -#define isa_writeb (b,a) writeb(b,isa_port2addr(a)) -#define isa_writew (w,a) writew(w,isa_port2addr(a)) -#define isa_writel (l,a) writel(l,isa_port2addr(a)) -#define isa_memset_io (a,b,c) \ - memset((void *)(isa_port2addr((unsigned long)a)),(b),(c)) -#define isa_memcpy_fromio (a,b,c) \ - memcpy((a),(void *)(isa_port2addr((unsigned long)(b))),(c)) -#define isa_memcpy_toio (a,b,c) \ - memcpy((void *)(isa_port2addr((unsigned long)(a))),(b),(c)) -/* We really want to try and get these to memcpy etc */ extern void memcpy_fromio (void *,unsigned long ,unsigned long );extern void memcpy_toio (unsigned long ,const void *,unsigned long );
arch/i386/Kconfig renamed from arch/i386/Kconfig.old +1 -0
@@ -307,6 +307,7 @@ config TOSHIBA tristate "Toshiba Laptop support" + depends on BROKEN ---help--- This adds a driver to safely access the System Management Mode of the CPU on Toshiba portables with a genuine Toshiba BIOS. It does
drivers/net/Kconfig renamed from drivers/net/Kconfig.old +3 -3
@@ -726,7 +726,7 @@ config LANCE tristate "AMD LANCE and PCnet (AT1500 and NE2100) support" - depends on NET_ETHERNET && ISA && ISA_DMA_API + depends on NET_ETHERNET && ISA && ISA_DMA_API && BROKEN help If you have a network (Ethernet) card of this type, say Y and read the Ethernet-HOWTO, available from
@@ -935,7 +935,7 @@ config HP100 tristate "HP 10/100VG PCLAN (ISA, EISA, PCI) support" - depends on NET_ETHERNET && (ISA || EISA || PCI) + depends on NET_ETHERNET && (ISA || EISA || PCI) && BROKEN help If you have a network (Ethernet) card of this type, say Y and read the Ethernet-HOWTO, available from
@@ -1021,7 +1021,7 @@ config HPLAN_PLUS tristate "HP PCLAN+ (27247B and 27252A) support" - depends on NET_ISA + depends on NET_ISA && BROKEN select CRC32 help If you have a network (Ethernet) card of this type, say Y and read
drivers/net/arcnet/Kconfig renamed from drivers/net/arcnet/Kconfig.old +2 -2
@@ -80,7 +80,7 @@ config ARCNET_COM90xx tristate "ARCnet COM90xx (normal) chipset driver" - depends on ARCNET + depends on ARCNET && BROKEN help This is the chipset driver for the standard COM90xx cards. If you have always used the old ARCnet driver without knowing what type of
@@ -105,7 +105,7 @@ config ARCNET_RIM_I tristate "ARCnet COM90xx (RIM I) chipset driver" - depends on ARCNET + depends on ARCNET && BROKEN ---help--- This is yet another chipset driver for the COM90xx cards, but this time only using memory-mapped mode, and no IO ports at all. This
drivers/scsi/Kconfig renamed from drivers/scsi/Kconfig.old +1 -1
@@ -474,7 +474,7 @@ config SCSI_IN2000 tristate "Always IN2000 SCSI support" - depends on ISA && SCSI + depends on ISA && SCSI && BROKEN help This is support for an ISA bus SCSI host adapter. You'll find more information in <file:Documentation/scsi/in2000.txt>. If it doesn't work
include/asm-arm/arch-aaec2000/io.h renamed from include/asm-arm/arch-aaec2000/io.h.old +0 -1
@@ -16,6 +16,5 @@*/ #define __io (a) ((void __iomem *)(a))#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#endif
include/asm-arm/arch-clps711x/io.h renamed from include/asm-arm/arch-clps711x/io.h.old +0 -1
@@ -26,7 +26,6 @@#define __io (a) ((void __iomem *)(a))#define __mem_pci (a) (a) -#define __mem_isa (a) (a)/* * We don't support ins[lb]/outs[lb]. Make them fault.
include/asm-arm/arch-ebsa285/io.h renamed from include/asm-arm/arch-ebsa285/io.h.old +0 -8
@@ -24,7 +24,6 @@#define __io (a) ((void __iomem *)(PCIO_BASE + (a)))#if 1 #define __mem_pci (a) (a) -#define __mem_isa (a) ((a) + PCIMEM_BASE)#else static inline void __iomem *___mem_pci (void __iomem *p)
@@ -34,14 +33,7 @@return p; } -static inline void __iomem *___mem_isa (void __iomem *p) -{ -unsigned long a = (unsigned long )p; -BUG_ON (a >=16 *1048576 ); -return p +PCIMEM_BASE ; -}#define __mem_pci (a) ___mem_pci(a) -#define __mem_isa (a) ___mem_isa(a)#endif #endif
include/asm-arm/arch-integrator/io.h renamed from include/asm-arm/arch-integrator/io.h.old +0 -1
@@ -32,6 +32,5 @@#define __io (a) ((void __iomem *)(PCI_IO_VADDR + (a)))#define __mem_pci (a) (a) -#define __mem_isa (a) ((a) + PCI_MEMORY_VADDR)#endif
include/asm-arm/arch-iop3xx/io.h renamed from include/asm-arm/arch-iop3xx/io.h.old +0 -1
@@ -17,6 +17,5 @@#define __io (p) ((void __iomem *)(p))#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#endif
include/asm-arm/arch-l7200/io.h renamed from include/asm-arm/arch-l7200/io.h.old +0 -1
@@ -19,7 +19,6 @@*/ #define __io_pci (a) ((void __iomem *)(PCIO_BASE + (a)))#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#define __ioaddr (p) __io_pci(p)
include/asm-arm/arch-lh7a40x/io.h renamed from include/asm-arm/arch-lh7a40x/io.h.old +0 -1
@@ -18,6 +18,5 @@/* No ISA or PCI bus on this machine. */ #define __io (a) ((void __iomem *)(a))#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#endif /* __ASM_ARCH_IO_H */
include/asm-arm/arch-omap/io.h renamed from include/asm-arm/arch-omap/io.h.old +0 -1
@@ -44,7 +44,6 @@*/ #define __io (a) ((void __iomem *)(PCIO_BASE + (a)))#define __mem_pci (a) (a) -#define __mem_isa (a) (a)/* * ----------------------------------------------------------------------------
include/asm-arm/arch-pxa/io.h renamed from include/asm-arm/arch-pxa/io.h.old +0 -1
@@ -16,6 +16,5 @@*/ #define __io (a) ((void __iomem *)(a))#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#endif
include/asm-arm/arch-sa1100/io.h renamed from include/asm-arm/arch-sa1100/io.h.old +0 -1
@@ -24,6 +24,5 @@ }#define __io (a) __io(a)#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#endif
include/asm-arm/arch-versatile/io.h renamed from include/asm-arm/arch-versatile/io.h.old +0 -1
@@ -28,6 +28,5 @@ }#define __io (a) __io(a)#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#endif
include/asm-arm/arch-realview/io.h renamed from include/asm-arm/arch-realview/io.h.old +0 -1
@@ -29,6 +29,5 @@#define __io (a) __io(a)#define __mem_pci (a) (a) -#define __mem_isa (a) (a)#endif
Re: [RFC: 2.6 patch] remove ISA legacy functions
Quoted 7 lines · Adrian Bunk
This patch removes the ISA legacy functions that are deprecated since kernel 2.4 and that aren't available on all architectures.
The 7 drivers that were still using this obsolete API are now marked as BROKEN.
Signed-off-by Adrian Bunk <[email protected]>
it's not valid to mark working drivers broken, IMO.
Mark them x86-only, perhaps.
Jeff
Re: [RFC: 2.6 patch] remove ISA legacy functions
Quoted 3 lines · Jeff Garzik
it's not valid to mark working drivers broken, IMO.
Mark them x86-only, perhaps.
hp100 works fine on parisc.
Re: [RFC: 2.6 patch] remove ISA legacy functions
Quoted 8 lines · Matthew Wilcox
Quoted 3 lines · Jeff Garzik
it's not valid to mark working drivers broken, IMO.
Mark them x86-only, perhaps.
hp100 works fine on parisc.
Certainly. The point was, don't mark them broken, limit them to the arches where they work.
Jeff
Re: [RFC: 2.6 patch] remove ISA legacy functions
Quoted 5 lines · Jeff Garzik
Quoted 4 lines · Matthew Wilcox
Quoted 2 lines · Jeff Garzik
it's not valid to mark working drivers broken, IMO.
Quoted 7 lines
Quoted 4 lines
Quoted 1 line
Mark them x86-only, perhaps.
hp100 works fine on parisc.
Certainly. The point was, don't mark them broken, limit them to the arches where they work.
I think they work fine everywhere. Adrian wants to remove the API they use.
I think this is a bad idea. The drivers should be converted.
Re: [RFC: 2.6 patch] remove ISA legacy functions
Quoted 4 lines · Matthew Wilcox
I think they work fine everywhere. Adrian wants to remove the API they use.
I think this is a bad idea. The drivers should be converted.
They are - I'll send patches later today...
Re: [RFC: 2.6 patch] remove ISA legacy functions
Quoted 7 lines · Al Viro
Quoted 4 lines · Matthew Wilcox
I think they work fine everywhere. Adrian wants to remove the API they use.
I think this is a bad idea. The drivers should be converted.
They are - I'll send patches later today...
NB: never say these words on a Friday night or you'll get a visit from Murphy.
Apologies for delay, patches sent.
Re: [RFC: 2.6 patch] remove ISA legacy functions
In reply to Adrian BunkQuoted 5 lines · Adrian Bunk
This patch removes the ISA legacy functions that are deprecated since kernel 2.4 and that aren't available on all architectures.
The 7 drivers that were still using this obsolete API are now marked as BROKEN.
NACK. Al has patches pending to convert them to proper APIs, please wait until after that patchseries has landed.
PATCH 1/1 mark virt_to_bus/bus_to_virt as __deprecated on i386
In reply to Adrian Bunkvirt_to_bus/bus_to_virt are long deprecated, mark them as __deprecated on i386.
include/asm-i386/io.h renamed from include/asm-i386/io.h.old +8 -2
@@ -144,8 +144,14 @@* * Allow them on x86 for legacy drivers, though. */ -#define virt_to_bus virt_to_phys -#define bus_to_virt phys_to_virt +static inline unsigned long __deprecated virt_to_bus (volatile void * address) +{ +return __pa (address); +} +static inline void *__deprecated bus_to_virt (unsigned long address) +{ +return __va (address); +}/* * readX/writeX() are used to access memory mapped devices. On some
Quoted 9 lines · Adrian Bunk
Quoted 9 lines
Quoted 8 lines
Quoted 4 lines
Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.
Can you send me the list of compile errors so that I can work on fixing them?
Quoted 6 lines
Quoted 2 lines
No handily, sorry. Missing
virt_to_bus()is the typical problem.But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.
It is a quite inconvenient change if you want to get full coverage with `make allmodconfig'.
Maybe one can do `make -i' and then weed through the noise - I haven't tried.
Quoted 6 lines
If you really want to keep the status quo, you can still steal the following from sparc64:
extern unsigned longvirt_to_bus_not_defined_use_pci_map(volatile void *addr); #definevirt_to_busvirt_to_bus_not_defined_use_pci_map
extern unsigned longbus_to_virt_not_defined_use_pci_map(volatile void *addr); #definebus_to_virtbus_to_virt_not_defined_use_pci_map
Maybe. There were some other failures.
Quoted 13 lines · Andrew Morton
Quoted 12 lines · Adrian Bunk
Quoted 11 lines
Quoted 8 lines
Quoted 4 lines
Sorry, I need to build allmodconfig kernels on wacky architectures (eg ppc64) and this patch is killing me.
Can you send me the list of compile errors so that I can work on fixing them?
No handily, sorry. Missing
virt_to_bus()is the typical problem.
Quoted 6 lines
Quoted 3 lines
But in this case -Werror-implicit-function-declaration doesn't create new compile errors, it only moves compile errors from compile time to link or depmod time - which is IMHO not a bad change.
It is a quite inconvenient change if you want to get full coverage with `make allmodconfig'.
It could be a Kconfig item if enabled or not.
Then you could use the new mechanishm in kconfig to disable it for your allmodconfig builds.
cat allmod.configCONFIG_CC_ERROR_IMPLICIT_FUNCTION_DECLARATION = 0
That should do the trick, but maybe too inconvinient??
Sam
Quoted 5 lines · Adrian Bunk
...
This might give some motivation for people to convert drivers and would avoid new code like the recently introduced kexec to use this obsolete API.
...
/me sits too long in front of the computer
s/kexec//
cu
Adrian
Signature · 4 lines
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed