On Mon, Oct 26, 2026 at 10:55:03PM +0000, Vencislav Atanasov wrote:

> Copied from armv7/start.S. Tested on an ARM1176JZF-S SoC.
> 
> Signed-off-by: Vencislav Atanasov <[email protected]>
> ---
>  arch/arm/cpu/arm1176/start.S | 42 ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 42 insertions(+)
> 
> diff --git a/arch/arm/cpu/arm1176/start.S b/arch/arm/cpu/arm1176/start.S
> index d3ab5928eee..7bdcb5fb97b 100644
> --- a/arch/arm/cpu/arm1176/start.S
> +++ b/arch/arm/cpu/arm1176/start.S
> @@ -37,6 +37,41 @@ reset:
>       b       save_boot_params
>  .globl       save_boot_params_ret
>  save_boot_params_ret:
> +#ifdef CONFIG_POSITION_INDEPENDENT
> +     /*
> +      * Fix .rela.dyn relocations. This allows U-Boot to loaded to and
> +      * executed at a different address than it was linked at.
> +      */
> +pie_fixup:
> +     adr r0, reset   /* r0 <- Runtime value of reset label */

I saw the first patch is correct, but this one didn't totally correct
the spacing. There's a tab, not space after the instruction and before
arguments. Consistency within the file is important, sorry.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to