From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sascha Hauer Date: Mon, 12 May 2008 11:47:55 +0200 Subject: [U-Boot-Users] [Patch] U-Boot-V2: Sandbox: remove unwanted compiler flags In-Reply-To: <4823A49D.9070301@gmail.com> References: <4823A49D.9070301@gmail.com> Message-ID: <20080512094755.GC4464@pengutronix.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, May 08, 2008 at 08:10:53PM -0500, Nishanth Menon wrote: > The following patch removes global compilation flags duplicated in > sandbox makefile. > This also works-around a probable compiler issue noted in my mail here: > http://sourceforge.net/mailarchive/forum.php?thread_name=48193193.30508%40gmail.com&forum_name=u-boot-users > > Signed-off-by: Nishanth Menon This patch does the right thing generally, I did not apply it for one reason: There are many compiler options which should be in the toplevel Makefile instead of the architecture Makefiles. I also realized that CFLAGS get overwritten in the architecture Makefiles instead of appended, so there are some more things to fix. May I ask you to keep your patch for another three weeks? Then we have our Pengutronix-hacking-week (tm) and I expect to do some general U-Boot work then. Sascha > > Index: u-boot.v2/arch/sandbox/Makefile > =================================================================== > --- u-boot.v2.orig/arch/sandbox/Makefile 2008-05-08 > 19:48:10.000000000 -0500 > +++ u-boot.v2/arch/sandbox/Makefile 2008-05-08 19:49:58.000000000 -0500 > @@ -1,5 +1,5 @@ > > -CPPFLAGS += -fno-builtin -ffreestanding -nostdinc -Wall \ > +CPPFLAGS += -fno-builtin -ffreestanding\ > -pipe -fno-strict-aliasing > > machine-y := sandbox > @@ -11,7 +11,7 @@ > TEXT_BASE = $(CONFIG_TEXT_BASE) > > CPPFLAGS += -P > -CFLAGS := -fno-common -Os -Dmalloc=u_boot_malloc \ > +CFLAGS := -Os -Dmalloc=u_boot_malloc \ > -Dfree=u_boot_free -Drealloc=u_boot_realloc \ > -Dread=u_boot_read -Dwrite=u_boot_write \ > -Dopen=u_boot_open -Dclose=u_boot_close \ > > -- Pengutronix e.K. - Linux Solutions for Science and Industry ----------------------------------------------------------- Kontakt-Informationen finden Sie im Header dieser Mail oder auf der Webseite -> http://www.pengutronix.de/impressum/ <-