From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ulf Samuelsson <ulf@atmel.com>
Date: Mon, 30 Jul 2007 16:25:44 +0200
Subject: [Buildroot] svn commit: trunk/buildroot/toolchain/gcc
References: <20070729174350.EF22BA4508@busybox.net>
	<20070730072137.GB21479@aon.at>
Message-ID: <025e01c7d2c2$c4a9a8d0$dcc4af0a@atmel.com>
List-Id: <buildroot.busybox.net>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: buildroot@busybox.net

----- Original Message ----- 
From: "Bernhard Fischer" <rep.dot.nop@gmail.com>
To: <buildroot@uclibc.org>
Cc: <ulf@atmel.com>
Sent: Monday, July 30, 2007 9:21 AM
Subject: Re: [Buildroot] svn commit: trunk/buildroot/toolchain/gcc


> On Sun, Jul 29, 2007 at 10:43:50AM -0700, ulf at uclibc.org wrote:
>>Author: ulf
>>Date: 2007-07-29 10:43:50 -0700 (Sun, 29 Jul 2007)
>>New Revision: 19332
>>
>>Log:
>>gcc target config for AVR32
>>
>>Modified:
>>   trunk/buildroot/toolchain/gcc/Config.in.2
>>
>>
>>Changeset:
>>Modified: trunk/buildroot/toolchain/gcc/Config.in.2
>>===================================================================
>>--- trunk/buildroot/toolchain/gcc/Config.in.2 2007-07-29 17:40:42 UTC (rev 19331)
>>+++ trunk/buildroot/toolchain/gcc/Config.in.2 2007-07-29 17:43:50 UTC (rev 19332)
>>@@ -5,9 +5,20 @@
>>   If you want the target system to be able to run 
>>   binutils/gcc and compile native code, say Y here.
>> 
>>+config BR2_GCC_TARGET_AVR32_CONFIGURATION
>>+ bool "Enable default AVR32 GCC configuration"
>>+ default y
>>+ depends BR2_avr32 && BR2_PACKAGE_GCC_TARGET
>>+ help
>>+   For the AVR32 platform there is some configuration which are not
>>+   supported.
>>+
>>+   Currently this will add "--disable-libmudflap" to configure.
>>+
> 
> I object to this change.
> Please explain why you don't just add disable-mudflap if the target arch
> is BR2_avr32
> 
>> config BR2_EXTRA_TARGET_GCC_CONFIG_OPTIONS
>> string "Additional target gcc options"
>> default ""
>>+ depends BR2_PACKAGE_GCC_TARGET
>> help
>>   Any additional target gcc options you may want to include....
>>   Including, but not limited to --disable-checking etc.
>

OK, no problem with me.
Guessed there would be comments :-)
Will read your comments for the rest of the patches
and try to apply within the next few days.
.

Best Regards
Ulf Samuelsson