[U-Boot] [PATCH 1/9] tegra2: Add arch_cpu_init() to fire up Cortex-A9

Simon Glass
2011-10-13T23:18:22Z

Hi Mike,

On Thu, Oct 13, 2011 at 3:52 PM, Mike Frysinger <vapier@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