[PATCH v2] configs: Remove redundant CONFIG_TEXT_BASE assignments for Renesas defconfigs
Lad, Prabhakar
prabhakar.csengg at gmail.com
Mon Jul 21 18:29:44 CEST 2025
Hi Marek,
On Sun, Jul 20, 2025 at 9:50 PM Marek Vasut <marek.vasut at mailbox.org> wrote:
>
> On 7/17/25 12:52 PM, Prabhakar wrote:
> > From: Lad Prabhakar <prabhakar.mahadev-lad.rj at bp.renesas.com>
> >
> > The Renesas board defconfigs explicitly set CONFIG_TEXT_BASE=0x50000000,
> > however U-Boot's POSITION_INDEPENDENT=y build default already places text
> > at 0x0. These hardcoded overrides are therefore unnecessary and will be
> > pruned automatically in upcoming resyncs.
> >
> > Remove the CONFIG_TEXT_BASE lines from the following defconfigs:
> > - hihope_rzg2_defconfig
> > - r8a77970_eagle_defconfig
> > - r8a77970_v3msk_defconfig
> > - r8a77990_ebisu_defconfig
> > - r8a77995_draak_defconfig
> > - renesas_rzg2l_smarc_defconfig
> > - rz2_beacon_defconfig
> > - silinux_ek874_defconfig
> >
> > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj at bp.renesas.com>
> > ---
> > v1->v2:
> > - Instead of setting CONFIG_TEXT_BASE=0x0, removed the line entirely
> > as it is redundant and will be pruned in future resyncs.
> > - Updated commit message to clarify the reason for removal.
> > - Included renesas_rzg2l_smarc_defconfig
> > ---
> > configs/hihope_rzg2_defconfig | 1 -
> > configs/r8a77970_eagle_defconfig | 1 -
> > configs/r8a77970_v3msk_defconfig | 1 -
> > configs/r8a77990_ebisu_defconfig | 1 -
> > configs/r8a77995_draak_defconfig | 1 -
> > configs/renesas_rzg2l_smarc_defconfig | 1 -
> > configs/rzg2_beacon_defconfig | 1 -
> > configs/silinux_ek874_defconfig | 1 -
> > 8 files changed, 8 deletions(-)
> What about r8a779a0_falcon_defconfig , does that need update too ?
Agreed, this needs an update too (the script which I ran only checked
for CONFIG_RCAR_GEN4). I'll send a v3 with `r8a779a0_falcon_defconfig`
Cheers,
Prabhakar
More information about the U-Boot
mailing list