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 <tho...@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 <ma...@denx.de>

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

Best regards,
Marek Vasut
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to