From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH] include/asm-generic/io.h: add dummy fuctions to support 'COMPILE_TEST' in 'asm-generic'. Date: Wed, 3 Jul 2013 10:14:12 +0200 Message-ID: <201307031014.12583.arnd@arndb.de> References: <51CA8AB8.7080504@asianux.com> <51D375AA.5060300@asianux.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset="gb2312" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51D375AA.5060300@asianux.com> Sender: linux-kernel-owner@vger.kernel.org To: Chen Gang Cc: Geert Uytterhoeven , Richard Weinberger , Jeff Dike , David Sharp , "sfr@canb.auug.org.au" , Steven Rostedt , Ingo Molnar , uml-devel , uml-user , "linux-kernel@vger.kernel.org" , Linux-Arch , Mark Brown , David Miller , Andrew Morton , Jiri Kosina List-Id: linux-arch.vger.kernel.org On Wednesday 03 July 2013, Chen Gang wrote: > On 07/02/2013 06:57 PM, Geert Uytterhoeven wrote: > > On Tue, Jul 2, 2013 at 10:00 AM, Chen Gang wrote: > >> > On 07/02/2013 03:19 PM, Geert Uytterhoeven wrote: > >>> >> On Tue, Jul 2, 2013 at 4:13 AM, Chen Gang wrote: > > I mean that COMPILE_TEST should exist in Kconfig files only. > > It's only meant to have more compile coverage, not to "fix" (through #ifdef) > > more code to make it compile. > > If so, can we allow the module to 'COMPILE_TEST' under one platform > which not support the related HW ? > > e.g. "...Despite they cannot be loaded there (or even when they load > they cannot be used due to missing HW support)...". There is a reason why ioremap and the associated functions make no sense on UML, and it remains important to not provide them here so we can find drivers that accidentally use them and are missing a dependency on HAS_IOMEM. > 'asm-generic' need provide generic layer to users (both architecture > guys and module guys). No. It's a set of examples for the architectures to look at and include if they want to. > So for 'default', it can depend on some conditions (e.g. HW support); > but for 'generic', it need try to be independent from any conditions. > > And it is also necessary for 'generic' to provide the configuration > checking features, but this checking must be no negative effect (or > consistent) with its 'generic' services. > > So it is necessary to check 'NOMMU', 'CONFIG_HAS_IOMEM' ..., but it > also necessary to consider about 'COMPILE_TEST' to be consistent with > its 'generic' services. The important distinction is between drivers we want to enable in COMPILE_TEST because they are written in a portable way but are just useless if you don't have the hardware, and other drivers that rely on an interface and that should not be built when that interface is not available. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from moutng.kundenserver.de ([212.227.17.9]:52904 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754838Ab3GCIOp (ORCPT ); Wed, 3 Jul 2013 04:14:45 -0400 From: Arnd Bergmann Subject: Re: [PATCH] include/asm-generic/io.h: add dummy fuctions to support 'COMPILE_TEST' in 'asm-generic'. Date: Wed, 3 Jul 2013 10:14:12 +0200 References: <51CA8AB8.7080504@asianux.com> <51D375AA.5060300@asianux.com> In-Reply-To: <51D375AA.5060300@asianux.com> MIME-Version: 1.0 Content-Type: Text/Plain; charset="gb2312" Content-Transfer-Encoding: 7bit Message-ID: <201307031014.12583.arnd@arndb.de> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Chen Gang Cc: Geert Uytterhoeven , Richard Weinberger , Jeff Dike , David Sharp , "sfr@canb.auug.org.au" , Steven Rostedt , Ingo Molnar , uml-devel , uml-user , "linux-kernel@vger.kernel.org" , Linux-Arch , Mark Brown , David Miller , Andrew Morton , Jiri Kosina Message-ID: <20130703081412.vGQMHetwvQMbIUL7E-_jVsMO5ssSaUMuiLG2qIPZcB4@z> On Wednesday 03 July 2013, Chen Gang wrote: > On 07/02/2013 06:57 PM, Geert Uytterhoeven wrote: > > On Tue, Jul 2, 2013 at 10:00 AM, Chen Gang wrote: > >> > On 07/02/2013 03:19 PM, Geert Uytterhoeven wrote: > >>> >> On Tue, Jul 2, 2013 at 4:13 AM, Chen Gang wrote: > > I mean that COMPILE_TEST should exist in Kconfig files only. > > It's only meant to have more compile coverage, not to "fix" (through #ifdef) > > more code to make it compile. > > If so, can we allow the module to 'COMPILE_TEST' under one platform > which not support the related HW ? > > e.g. "...Despite they cannot be loaded there (or even when they load > they cannot be used due to missing HW support)...". There is a reason why ioremap and the associated functions make no sense on UML, and it remains important to not provide them here so we can find drivers that accidentally use them and are missing a dependency on HAS_IOMEM. > 'asm-generic' need provide generic layer to users (both architecture > guys and module guys). No. It's a set of examples for the architectures to look at and include if they want to. > So for 'default', it can depend on some conditions (e.g. HW support); > but for 'generic', it need try to be independent from any conditions. > > And it is also necessary for 'generic' to provide the configuration > checking features, but this checking must be no negative effect (or > consistent) with its 'generic' services. > > So it is necessary to check 'NOMMU', 'CONFIG_HAS_IOMEM' ..., but it > also necessary to consider about 'COMPILE_TEST' to be consistent with > its 'generic' services. The important distinction is between drivers we want to enable in COMPILE_TEST because they are written in a portable way but are just useless if you don't have the hardware, and other drivers that rely on an interface and that should not be built when that interface is not available. Arnd