[PATCH 2/2] sandbox: u-boot.lds: Remove bogus __bss_start symbol

[Simon Glass]

On Mon, 17 Aug 2020 at 12:30, Ovidiu Panait <ovidiu.panait at windriver.com> wrote:
>
> The sections described in the sandbox linker script are inserted before
> data section via "INSERT BEFORE .data;". Running readelf -S on sandbox
> u-boot binary shows that the bss section is located after the data
> section:
>
>   Section Headers:
>   [Nr] Name              Type             Address           Offset
>        Size              EntSize          Flags  Link  Info  Align
>   ...
>   [25] .u_boot_list      PROGBITS         000000000041d1c8  0021d1c8
>        000000000000dd90  0000000000000000  WA       0     0     8
>   [26] _u_boot_sandbox_g PROGBITS         000000000042af58  0022af58
>        00000000000000a0  0000000000000000  WA       0     0     8
>   [27] .data             PROGBITS         000000000042b000  0022b000
>        000000000000f708  0000000000000000  WA       0     0     32
>   [28] .bss              NOBITS           000000000043a720  0023a708
>        0000000000018930  0000000000000000  WA       0     0     32
>
> This means that the __bss_start assignment in the linker script is bogus,
> as the actual bss section start is located elsewhere. Remove this
> assignment, as the __bss_start symbol is not used on sandbox anyway.
>
> Signed-off-by: Ovidiu Panait <ovidiu.panait at windriver.com>
> ---
>
>  arch/sandbox/cpu/u-boot-spl.lds | 2 --
>  arch/sandbox/cpu/u-boot.lds     | 2 --
>  2 files changed, 4 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>