Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753287AbbK3IRS (ORCPT ); Mon, 30 Nov 2015 03:17:18 -0500 Received: from e06smtp11.uk.ibm.com ([195.75.94.107]:37223 "EHLO e06smtp11.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752886AbbK3IRP (ORCPT ); Mon, 30 Nov 2015 03:17:15 -0500 X-IBM-Helo: d06dlp02.portsmouth.uk.ibm.com X-IBM-MailFrom: heiko.carstens@de.ibm.com X-IBM-RcptTo: linux-kbuild@vger.kernel.org;linux-kernel@vger.kernel.org Date: Mon, 30 Nov 2015 09:17:08 +0100 From: Heiko Carstens To: Michal Marek , linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, Martin Schwidefsky Subject: Re: [PATCH 1/2] kbuild: add AFLAGS_REMOVE_(basename).o option Message-ID: <20151130081708.GA4045@osiris> References: <1448103599-6660-1-git-send-email-heiko.carstens@de.ibm.com> <1448103599-6660-2-git-send-email-heiko.carstens@de.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1448103599-6660-2-git-send-email-heiko.carstens@de.ibm.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15113008-0041-0000-0000-000006B33C92 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1840 Lines: 43 On Sat, Nov 21, 2015 at 11:59:58AM +0100, Heiko Carstens wrote: > It is already possible to remove CFLAGS with the CFLAGS_REMOVE option > that was introduced with commit 656ee82cc855 ("kbuild: create new > CFLAGS_REMOVE_(basename).o option"). However it is not possible to > remove AFLAGS for assembler files. > > So this patch just adds the AFLAGS_REMOVE option which works the same > like CFLAGS_REMOVE. > > Signed-off-by: Heiko Carstens > --- > scripts/Makefile.lib | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > index 79e86613712f..26a48d76eb9d 100644 > --- a/scripts/Makefile.lib > +++ b/scripts/Makefile.lib > @@ -104,8 +104,9 @@ modname_flags = $(if $(filter 1,$(words $(modname))),\ > orig_c_flags = $(KBUILD_CPPFLAGS) $(KBUILD_CFLAGS) $(KBUILD_SUBDIR_CCFLAGS) \ > $(ccflags-y) $(CFLAGS_$(basetarget).o) > _c_flags = $(filter-out $(CFLAGS_REMOVE_$(basetarget).o), $(orig_c_flags)) > -_a_flags = $(KBUILD_CPPFLAGS) $(KBUILD_AFLAGS) $(KBUILD_SUBDIR_ASFLAGS) \ > +orig_a_flags = $(KBUILD_CPPFLAGS) $(KBUILD_AFLAGS) $(KBUILD_SUBDIR_ASFLAGS) \ > $(asflags-y) $(AFLAGS_$(basetarget).o) > +_a_flags = $(filter-out $(AFLAGS_REMOVE_$(basetarget).o), $(orig_a_flags)) > _cpp_flags = $(KBUILD_CPPFLAGS) $(cppflags-y) $(CPPFLAGS_$(@F)) > > # Hello Michal, since I didn't hear anything from you, I'll simply add this to the s390 tree. The patch should be trivial enough and is certainly not controversial. Thanks, Heiko -- 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/