Hi Mike,

On Thu, Oct 13, 2011 at 3:52 PM, Mike Frysinger <vap...@gentoo.org> wrote:
> On Thursday 13 October 2011 17:05:55 Simon Glass wrote:
>> --- a/arch/arm/lib/Makefile
>> +++ b/arch/arm/lib/Makefile
>>
>> +# Tegra has an ARMv4T CPU which runs board_init_f(), so we must build this
>> +# file with compatible flags
>> +ifdef CONFIG_TEGRA2
>> +CFLAGS_arch/arm/lib/board.o += -march=armv4t
>> +endif
>
> all Tegra SoC's have an ARMv4T inside of the package ?  that's a bit crazy.
> isn't there a tegra-specific config.mk somewhere we could add this instead ?
> -mike
>

Well that same line can't go into the tegra-specific config.mk as it
will not be seen when compiling arch/arm/lib.

The current code compiles the whole of U-Boot with ARMv4T which is bad
since only a few functions need it. This is one thing that the patch
series cleans up.

Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to