From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH 1/2] asm-generic/io.h: provide default ioremap/iounmap for !HAS_IOMEM References: <1459275781-3863-1-git-send-email-robh@kernel.org> <3074778.3TElYrXjM1@wuerfel> From: Richard Weinberger Message-ID: <56FC3899.4000100@nod.at> Date: Wed, 30 Mar 2016 22:35:37 +0200 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven , Rob Herring Cc: Arnd Bergmann , Lee Jones , "linux-kernel@vger.kernel.org" , "linux-arch@vger.kernel.org" , uml-devel List-ID: Am 30.03.2016 um 22:08 schrieb Geert Uytterhoeven: >> I actually started a patch to remove the HAS_IOMEM dependency >> everywhere (or just the per driver cases). It didn't break as bad as I >> expected, but became more than I wanted to fix. Mainly, all the devm_ >> variants also need empty versions or to be always enabled. > > Should these dependencies on HAS_IOMEM be changed to "HAS_IOMEM || > COMPILE_TEST"? Another idea, the issue with COMPILE_TEST is that it is automatically selected by make targets such as allyesconfig or allmodconfig. Couldn't we make COMPILE_TEST just harder to select? i.e. that with allyesconfig it is still disabled and if someone really wants to do a compile test of anything we could issue something like "make allyesconfig COMPILE_TEST=y" to have COMPILE_TEST selected? Thanks, //richard