From mboxrd@z Thu Jan  1 00:00:00 1970
From: Allen Martin <amartin@nvidia.com>
Date: Fri, 19 Oct 2012 22:46:45 -0700
Subject: [U-Boot] [PATCH V2 1/5] ARM: fix u-boot.lds for
 -ffunction-sections/-fdata-sections
In-Reply-To: <1350602759-22605-1-git-send-email-swarren@wwwdotorg.org>
References: <1350602759-22605-1-git-send-email-swarren@wwwdotorg.org>
Message-ID: <20121020054644.GB25104@badger>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Thu, Oct 18, 2012 at 04:25:55PM -0700, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
> 
> When -ffunction-sections or -fdata-section are used, symbols are placed
> into sections such as .data.eserial1_device and .bss.serial_current.
> Update the linker script to explicitly include these. Without this
> change (at least with my gcc-4.5.3 built using crosstool-ng), I see that
> the sections do end up being included, but __bss_end__ gets set to the
> same value as __bss_start.
> 

Acked-by: Allen Martin <amartin@nvidia.com>

> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> v2: Removed changes from some entries where it wasn't needed.
> ---
>  arch/arm/cpu/u-boot.lds |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/cpu/u-boot.lds b/arch/arm/cpu/u-boot.lds
> index e49ca0c..9153c3d 100644
> --- a/arch/arm/cpu/u-boot.lds
> +++ b/arch/arm/cpu/u-boot.lds
> @@ -34,8 +34,8 @@ SECTIONS
>  	.text :
>  	{
>  		__image_copy_start = .;
> -		CPUDIR/start.o (.text)
> -		*(.text)
> +		CPUDIR/start.o (.text*)
> +		*(.text*)
>  	}
>  
>  	. = ALIGN(4);
> @@ -43,7 +43,7 @@ SECTIONS
>  
>  	. = ALIGN(4);
>  	.data : {
> -		*(.data)
> +		*(.data*)
>  	}
>  
>  	. = ALIGN(4);
> @@ -81,7 +81,7 @@ SECTIONS
>  
>  	.bss __rel_dyn_start (OVERLAY) : {
>  		__bss_start = .;
> -		*(.bss)
> +		*(.bss*)
>  		 . = ALIGN(4);
>  		__bss_end__ = .;
>  	}
> -- 
> 1.7.0.4
> 

-- 
nvpublic