diff options
| author | Jessica Clarke <[email protected]> | 2025-07-10 01:29:35 +0100 |
|---|---|---|
| committer | Anup Patel <[email protected]> | 2025-07-21 16:39:49 +0530 |
| commit | e10a45752fe77a1e6fc75e254947dadc3db633ec (patch) | |
| tree | ccddce611f0e874ff8b0649437988b84d5c26f34 | |
| parent | 4825a3f87f8d3ee45b7f9b76ed62de7bd1f2dfa8 (diff) | |
firmware: Rename __rel_dyn_start/end to __rela_dyn_start/end
We are using and expecting the RELA format, not the REL format, and this
is the conventional linker-generated name for the start/end symbols, so
use it rather than confusing things by making it look like we're
accessing .rel.dyn, which would be in the REL format with no explicit
addend.
Signed-off-by: Jessica Clarke <[email protected]>
Reviewed-by: Samuel Holland <[email protected]>
Tested-by: Samuel Holland <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Anup Patel <[email protected]>
| -rw-r--r-- | firmware/fw_base.S | 4 | ||||
| -rw-r--r-- | firmware/fw_base.ldS | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/firmware/fw_base.S b/firmware/fw_base.S index a008fa74..9e2c017c 100644 --- a/firmware/fw_base.S +++ b/firmware/fw_base.S @@ -76,8 +76,8 @@ _sc_fail: li t0, FW_TEXT_START /* link start */ lla t1, _fw_start /* load start */ sub t2, t1, t0 /* load offset */ - lla t0, __rel_dyn_start - lla t1, __rel_dyn_end + lla t0, __rela_dyn_start + lla t1, __rela_dyn_end beq t0, t1, _relocate_done 2: REG_L t5, REGBYTES(t0) /* t5 <-- relocation info:type */ diff --git a/firmware/fw_base.ldS b/firmware/fw_base.ldS index a33746a0..12c7a844 100644 --- a/firmware/fw_base.ldS +++ b/firmware/fw_base.ldS @@ -47,9 +47,9 @@ . = ALIGN(0x1000); /* Ensure next section is page aligned */ .rela.dyn : { - PROVIDE(__rel_dyn_start = .); + PROVIDE(__rela_dyn_start = .); *(.rela*) - PROVIDE(__rel_dyn_end = .); + PROVIDE(__rela_dyn_end = .); } PROVIDE(_rodata_end = .); |