From: Sascha Hauer <s.hauer@pengutronix.de>
To: BAREBOX <barebox@lists.infradead.org>
Cc: "Claude Sonnet 4.5" <noreply@anthropic.com>,
Ahmad Fatoum <a.fatoum@pengutronix.de>
Subject: [PATCH v5 17/22] ARM: cleanup barebox proper entry
Date: Fri, 16 Jan 2026 09:12:28 +0100 [thread overview]
Message-ID: <20260116-pbl-load-elf-v5-17-51d042021c5a@pengutronix.de> (raw)
In-Reply-To: <20260116-pbl-load-elf-v5-0-51d042021c5a@pengutronix.de>
As barebox proper is now an ELF file we no longer need to map the entry
function to the start of the image. Just link it to wherever the linker
wants it and drop the text_entry section. Also, remove the start()
function and set the ELF entry to barebox_non_pbl_start() directly.
While at it also remove the bare_init stuff from the barebox proper
linker script as it's only relevant to the PBL linker script which
is a separate script.
Reviewed-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
arch/arm/cpu/start.c | 11 -----------
arch/arm/lib32/barebox.lds.S | 7 +------
arch/arm/lib64/barebox.lds.S | 7 +------
3 files changed, 2 insertions(+), 23 deletions(-)
diff --git a/arch/arm/cpu/start.c b/arch/arm/cpu/start.c
index 2498bdb894..c2f14736da 100644
--- a/arch/arm/cpu/start.c
+++ b/arch/arm/cpu/start.c
@@ -189,14 +189,3 @@ __noreturn void barebox_non_pbl_start(unsigned long membase,
start_barebox();
}
-
-void start(unsigned long membase, unsigned long memsize, struct handoff_data *hd);
-/*
- * First function in the uncompressed image. We get here from
- * the pbl. The stack already has been set up by the pbl.
- */
-void __section(.text_entry) start(unsigned long membase,
- unsigned long memsize, struct handoff_data *hd)
-{
- barebox_non_pbl_start(membase, memsize, hd);
-}
diff --git a/arch/arm/lib32/barebox.lds.S b/arch/arm/lib32/barebox.lds.S
index c098993615..02db3b9790 100644
--- a/arch/arm/lib32/barebox.lds.S
+++ b/arch/arm/lib32/barebox.lds.S
@@ -6,7 +6,7 @@
OUTPUT_FORMAT(BAREBOX_OUTPUT_FORMAT)
OUTPUT_ARCH(BAREBOX_OUTPUT_ARCH)
-ENTRY(start)
+ENTRY(barebox_non_pbl_start)
PHDRS
{
@@ -27,17 +27,12 @@ SECTIONS
.text :
{
_stext = .;
- *(.text_entry*)
- __bare_init_start = .;
- *(.text_bare_init*)
- __bare_init_end = .;
. = ALIGN(0x20);
__exceptions_start = .;
KEEP(*(.text_exceptions*))
__exceptions_stop = .;
*(.text*)
} :text
- BAREBOX_BARE_INIT_SIZE
. = ALIGN(4096);
__start_rodata = .;
diff --git a/arch/arm/lib64/barebox.lds.S b/arch/arm/lib64/barebox.lds.S
index 1ce2c67df0..bd76a0ca96 100644
--- a/arch/arm/lib64/barebox.lds.S
+++ b/arch/arm/lib64/barebox.lds.S
@@ -5,7 +5,7 @@
OUTPUT_FORMAT(BAREBOX_OUTPUT_FORMAT)
OUTPUT_ARCH(BAREBOX_OUTPUT_ARCH)
-ENTRY(start)
+ENTRY(barebox_non_pbl_start)
PHDRS
{
@@ -26,13 +26,8 @@ SECTIONS
.text :
{
_stext = .;
- *(.text_entry*)
- __bare_init_start = .;
- *(.text_bare_init*)
- __bare_init_end = .;
*(.text*)
} :text
- BAREBOX_BARE_INIT_SIZE
. = ALIGN(4096);
__start_rodata = .;
--
2.47.3
next prev parent reply other threads:[~2026-01-16 8:35 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-16 8:12 [PATCH v5 00/22] PBL: Add PBL ELF loading support with dynamic relocations Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 01/22] Makefile.compiler: add objcopy-option Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 02/22] elf: only accept images matching the native ELF_CLASS Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 03/22] elf: build for PBL as well Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 04/22] elf: add elf segment iterator Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 05/22] elf: add dynamic relocation support Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 06/22] ARM: implement elf_apply_relocations() for ELF " Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 07/22] riscv: define generic relocate_image Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 08/22] riscv: implement elf_apply_relocations() for ELF relocation support Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 09/22] elf: implement elf_load_inplace() Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 10/22] elf: create elf_open_binary_into() Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 11/22] Makefile: add vmbarebox build target Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 12/22] PBL: allow to link ELF image into PBL Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 13/22] mmu: add MAP_CACHED_RO mapping type Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 14/22] ARM: drop arm_fixup_vectors() Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 15/22] ARM: linker script: create separate PT_LOAD segments for text, rodata, and data Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 16/22] ARM: link ELF image into PBL Sascha Hauer
2026-01-16 8:12 ` Sascha Hauer [this message]
2026-01-16 8:12 ` [PATCH v5 18/22] ARM: PBL: setup MMU with proper permissions from ELF segments Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 19/22] riscv: linker script: create separate PT_LOAD segments for text, rodata, and data Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 20/22] riscv: link ELF image into PBL Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 21/22] riscv: Allwinner D1: Drop M-Mode Sascha Hauer
2026-01-16 8:12 ` [PATCH v5 22/22] riscv: add ELF segment-based memory protection with MMU Sascha Hauer
2026-01-19 12:23 ` [PATCH v5 00/22] PBL: Add PBL ELF loading support with dynamic relocations Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260116-pbl-load-elf-v5-17-51d042021c5a@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=a.fatoum@pengutronix.de \
--cc=barebox@lists.infradead.org \
--cc=noreply@anthropic.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox