From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e06smtp11.uk.ibm.com ([195.75.94.107]:37222 "EHLO e06smtp11.uk.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753085AbbK3IRP (ORCPT ); Mon, 30 Nov 2015 03:17:15 -0500 Received: from localhost by e06smtp11.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 30 Nov 2015 08:17:13 -0000 Date: Mon, 30 Nov 2015 09:17:08 +0100 From: Heiko Carstens 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> Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Michal Marek , linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, Martin Schwidefsky 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