From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751179AbVKKUgG (ORCPT ); Fri, 11 Nov 2005 15:36:06 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751181AbVKKUgG (ORCPT ); Fri, 11 Nov 2005 15:36:06 -0500 Received: from emailhub.stusta.mhn.de ([141.84.69.5]:29445 "HELO mailout.stusta.mhn.de") by vger.kernel.org with SMTP id S1751179AbVKKUgF (ORCPT ); Fri, 11 Nov 2005 15:36:05 -0500 Date: Fri, 11 Nov 2005 21:36:01 +0100 From: Adrian Bunk To: Andrew Morton Cc: linux-kernel@vger.kernel.org, paulus@samba.org, anton@samba.org, linuxppc64-dev@ozlabs.org Subject: Re: [2.6 patch] add -Werror-implicit-function-declaration to CFLAGS Message-ID: <20051111203601.GR5376@stusta.de> References: <20051107200336.GH3847@stusta.de> <20051110042857.38b4635b.akpm@osdl.org> <20051111021258.GK5376@stusta.de> <20051110182443.514622ed.akpm@osdl.org> <20051111201849.GP5376@stusta.de> <20051111202005.GQ5376@stusta.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20051111202005.GQ5376@stusta.de> User-Agent: Mutt/1.5.11 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Nov 11, 2005 at 09:20:05PM +0100, Adrian Bunk wrote: > On Fri, Nov 11, 2005 at 09:18:49PM +0100, Adrian Bunk wrote: > >... > > 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? > >... > > 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? 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 -- "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