From patchwork Sun Dec 12 08:25:34 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dirk Behme X-Patchwork-Id: 75220 X-Patchwork-Delegate: s-paulraj@ti.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 96DF9B70A3 for ; Sun, 12 Dec 2010 19:26:14 +1100 (EST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 1D63E280F5; Sun, 12 Dec 2010 09:26:11 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Awuc7KQ6hyfr; Sun, 12 Dec 2010 09:26:10 +0100 (CET) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 7BB82280ED; Sun, 12 Dec 2010 09:26:09 +0100 (CET) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id CF650280ED for ; Sun, 12 Dec 2010 09:26:05 +0100 (CET) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id u13M7yJqbzjl for ; Sun, 12 Dec 2010 09:26:02 +0100 (CET) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from mail-fx0-f51.google.com (mail-fx0-f51.google.com [209.85.161.51]) by theia.denx.de (Postfix) with ESMTP id 0929B28083 for ; Sun, 12 Dec 2010 09:26:00 +0100 (CET) Received: by fxm5 with SMTP id 5so4735331fxm.10 for ; Sun, 12 Dec 2010 00:26:00 -0800 (PST) Received: by 10.223.78.206 with SMTP id m14mr2989069fak.73.1292142358265; Sun, 12 Dec 2010 00:25:58 -0800 (PST) Received: from [192.168.178.23] (p5B0462A8.dip.t-dialin.net [91.4.98.168]) by mx.google.com with ESMTPS id n3sm1355065faa.29.2010.12.12.00.25.56 (version=TLSv1/SSLv3 cipher=RC4-MD5); Sun, 12 Dec 2010 00:25:57 -0800 (PST) Message-ID: <4D0486FE.9080601@googlemail.com> Date: Sun, 12 Dec 2010 09:25:34 +0100 From: Dirk Behme User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.13) Gecko/20101207 Thunderbird/3.1.7 MIME-Version: 1.0 To: Albert ARIBAUD , "u-boot@lists.denx.de" , "Premi, Sanjeev" Subject: [U-Boot] OMAP3: EVM: Linker errors across tool chain versions X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.9 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: u-boot-bounces@lists.denx.de Errors-To: u-boot-bounces@lists.denx.de What's the status of the linker errors across tool chain versions seen by building the 'omap3_evm' board? This was already discussed in the threads [1] and [2], but it seems to me that there was no acceptable fix, yet? E.g. with CodeSourcery's tool chain 2009q1-203 I get arm-none-linux-gnueabi-ld: section .bss [8003f5e0 -> 8007e33f] overlaps section .rel.dyn [8003f5e0 -> 80044e6f] arm-none-linux-gnueabi-ld: section .dynsym [80044e70 -> 80044f0f] overlaps section .bss [8003f5e0 -> 8007e33f] arm-none-linux-gnueabi-ld: u-boot: section .bss vma 0x8003f5e0 overlaps previous sections What we learned from [2] is that using [3] fixes the build (at least for me with the tool chain mentioned above). I.e. converting omap3_evm_version from u8 to int -static u8 omap3_evm_version; +static int omap3_evm_version; makes the issue disappear. But if I understood the discussion correctly, this is no acceptable fix and can be used only as debug help (?). Any further hints? Thanks Dirk [1] ARMv7: Fix linker errors across toolchain versions http://lists.denx.de/pipermail/u-boot/2010-December/082874.html [2] omap3evm: Clean-up EVM detection code http://lists.denx.de/pipermail/u-boot/2010-December/082950.html [3] board/ti/evm/evm.c | 4 ++-- board/ti/evm/evm.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) Index: u-boot.git/board/ti/evm/evm.c =================================================================== --- u-boot.git.orig/board/ti/evm/evm.c +++ u-boot.git/board/ti/evm/evm.c @@ -37,9 +37,9 @@ #include #include "evm.h" -static u8 omap3_evm_version; +static int omap3_evm_version; -u8 get_omap3_evm_rev(void) +int get_omap3_evm_rev(void) { return omap3_evm_version; } Index: u-boot.git/board/ti/evm/evm.h =================================================================== --- u-boot.git.orig/board/ti/evm/evm.h +++ u-boot.git/board/ti/evm/evm.h @@ -45,7 +45,7 @@ enum { OMAP3EVM_BOARD_GEN_2, /* EVM Rev >= Rev E */ }; -u8 get_omap3_evm_rev(void); +int get_omap3_evm_rev(void); #if defined(CONFIG_CMD_NET) static void setup_net_chip(void);