[U-Boot] [PATCH v2 5/8] nios2: remove gp assignments in link script

Marek Vasut marex at denx.de
Fri Sep 4 16:00:31 CEST 2015


On Friday, September 04, 2015 at 02:22:20 PM, Thomas Chou wrote:
> Since we don't use gp for small data with option "-G0",
> we use gp as global data pointer. The _gp location is
> not needed.
> 
> The ALIGN(16) was for gp only. It is removed, so that we
> can get correct address of dtb in u-boot-dtb.bin image.
> 
> Signed-off-by: Thomas Chou <thomas at wytron.com.tw>
> ---
>  arch/nios2/cpu/u-boot.lds | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/nios2/cpu/u-boot.lds b/arch/nios2/cpu/u-boot.lds
> index 6e174be..951785c 100644
> --- a/arch/nios2/cpu/u-boot.lds
> +++ b/arch/nios2/cpu/u-boot.lds
> @@ -50,9 +50,10 @@ SECTIONS
>  	  *(.gnu.linkonce.d*)
>  	}
> 
> -	. = ALIGN(16);
> -	_gp = .;			/* Global pointer addr */
> -	PROVIDE (gp = .);
> +	/* gp - Since we don't use gp for small data with option "-G0",

Please make the multiline comment match the kernel coding style:

/*
 * foo
 * bar
 */

Otherwise
Acked-by: Marek Vasut <marex at denx.de>

> +	 * we will use gp as global data pointer. The _gp location is
> +	 * not needed.
> +	 */
> 
>  	.sdata :
>  	{

Best regards,
Marek Vasut


More information about the U-Boot mailing list