Patchwork [U-Boot,v2,10/10] tegra20: Remove armv4t build flags

login
register
mail settings
Submitter Allen Martin
Date June 5, 2012, 9:20 p.m.
Message ID <1338931225-12246-11-git-send-email-amartin@nvidia.com>
Download mbox | patch
Permalink /patch/163174/
State Superseded
Headers show

Comments

Allen Martin - June 5, 2012, 9:20 p.m.
These flags were necessary when building tegra20 as a single binary
that supported ARM7TDMI and Cortex A9.  Now that the ARM7TDMI support
is split into a separate SPL, this is no longer necessary.

Signed-off-by: Allen Martin <amartin@nvidia.com>
---
 arch/arm/cpu/armv7/tegra20/config.mk |    7 -------
 1 file changed, 7 deletions(-)
Stephen Warren - June 6, 2012, 4:52 p.m.
On 06/05/2012 03:20 PM, Allen Martin wrote:
> These flags were necessary when building tegra20 as a single binary
> that supported ARM7TDMI and Cortex A9.  Now that the ARM7TDMI support
> is split into a separate SPL, this is no longer necessary.

Can we also (probably in a separate patch) remove USE_PRIVATE_LIBGCC
from the makefiles/... wherever we define it for Tegra? At least for the
non-SPL build; presumably the SPL build will still need it.

Patch

diff --git a/arch/arm/cpu/armv7/tegra20/config.mk b/arch/arm/cpu/armv7/tegra20/config.mk
index fe9ef5b..e666f6b 100644
--- a/arch/arm/cpu/armv7/tegra20/config.mk
+++ b/arch/arm/cpu/armv7/tegra20/config.mk
@@ -23,13 +23,6 @@ 
 # Foundation, Inc., 59 Temple Place, Suite 330, Boston,
 # MA 02111-1307 USA
 #
-
-# 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
-
 USE_PRIVATE_LIBGCC = yes
 
 CONFIG_ARCH_DEVICE_TREE := tegra20