Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756088Ab0G1Sa2 (ORCPT ); Wed, 28 Jul 2010 14:30:28 -0400 Received: from pfepb.post.tele.dk ([195.41.46.236]:59315 "EHLO pfepb.post.tele.dk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755343Ab0G1Sa0 (ORCPT ); Wed, 28 Jul 2010 14:30:26 -0400 Date: Wed, 28 Jul 2010 20:30:24 +0200 From: Sam Ravnborg To: Michal Marek , linux-kbuild , lkml , Denys Vlasenko Cc: Benjamin Herrenschmidt , Chen Liqin , David Howells , Geert Uytterhoeven , Haavard Skinnemoen , Hirokazu Takata , Martin Schwidefsky , Mike Frysinger , Ralf Baechle , Tony Luck Subject: Re: [PATCH v2 0/3] update of various kbuild flags to allow user settings Message-ID: <20100728183024.GD29232@merkur.ravnborg.org> References: <20100728182523.GA29198@merkur.ravnborg.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100728182523.GA29198@merkur.ravnborg.org> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2280 Lines: 57 On Wed, Jul 28, 2010 at 08:25:23PM +0200, Sam Ravnborg wrote: > Denys Vlasenko complained that he could not set > a moduel specific linker flags on the command line. > > It turned out that the cause of this was kbuild > that assigned a value to a variables that > may be modified by the user. > So any user supplied value would overwrite > the kbuild assigned value. > > Fix this so we use internal variables for kbuild > feering up the official variables for use by > the user. > > Changes in v2: > Fix so we can now specify both module and built-in > options on command line. > Combine all *_MODULE changes in one patch. > Add a new patch for *_KERNEL changes. > Document the new variables in makefiles.txt > Document the variables that may be set on the command line > in kbuild.txt > > The frv Makefile saw a bigger cleanup this time. > I suggest to merge it through kbuild.git because the > update is dependent on the first patch in this serie. Forgot the stats: Sam Ravnborg (3): kbuild: allow assignment to {A,C,LD}FLAGS_MODULE on the command line kbuild: allow assignment to {A,C}FLAGS_KERNEL on the command line frv: clean up arch/frv/Makefile Documentation/kbuild/kbuild.txt | 26 ++++++++++++++++++++++++-- Documentation/kbuild/makefiles.txt | 27 ++++++++++++++++++++++----- Makefile | 17 ++++++++++++----- arch/avr32/Makefile | 2 +- arch/blackfin/Makefile | 4 ++-- arch/frv/Makefile | 22 ++-------------------- arch/ia64/Makefile | 6 +++--- arch/m32r/Makefile | 4 ++-- arch/m68k/Makefile | 2 +- arch/mips/Makefile | 6 ++++-- arch/powerpc/Makefile | 2 +- arch/s390/Makefile | 3 ++- arch/score/Makefile | 3 ++- scripts/Makefile.build | 11 +++++++---- scripts/Makefile.modpost | 7 ++++--- 15 files changed, 89 insertions(+), 53 deletions(-) -- 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/