On Thu, Sep 16, 2021 at 10:14:03PM -0700, Max Filippov wrote:
> On Thu, Sep 16, 2021 at 10:30 AM Tom Rini <tr...@konsulko.com> wrote:
> >
> > Update the linker script macros to know that we need to include the
> > .text_version_string section now as well.
> >
> > Signed-off-by: Tom Rini <tr...@konsulko.com>
> > ---
> > arch/xtensa/include/asm/ldscript.h | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/xtensa/include/asm/ldscript.h
> > b/arch/xtensa/include/asm/ldscript.h
> > index 08f5d0135ed0..84c496e09f1b 100644
> > --- a/arch/xtensa/include/asm/ldscript.h
> > +++ b/arch/xtensa/include/asm/ldscript.h
> > @@ -72,6 +72,7 @@
> > { \
> > _text_start = ABSOLUTE(.); \
> > *(.literal .text) \
> > + *(.literal .text_version_string) \
>
> This does not belong to .text, as far as I understand it's rodata and so
> it should go with rodata, probably like this:
> ---8<---
> diff --git a/arch/xtensa/include/asm/ldscript.h
> b/arch/xtensa/include/asm/ldscript.h
> index 08f5d0135ed0..e03fcffdd6f1 100644
> --- a/arch/xtensa/include/asm/ldscript.h
> +++ b/arch/xtensa/include/asm/ldscript.h
> @@ -87,6 +87,7 @@
> _rodata_start = ABSOLUTE(.); \
> *(.rodata) \
> *(.rodata.*) \
> + *(.text_version_string) \
> *(.dtb.init.rodata) \
> *(.gnu.linkonce.r.*) \
> *(.rodata1) \So this is in context with: https://patchwork.ozlabs.org/project/uboot/list/?series=256258 and specifically: https://patchwork.ozlabs.org/project/uboot/patch/20210802131838.21097-7-p...@kernel.org/ Now that said, I think the whole point of that has been removed with: https://patchwork.ozlabs.org/project/uboot/patch/20210916195648.9424-1-tr...@konsulko.com/ and no longer needing to reference it from the linker script for PowerPC. So maybe I can just drop this whole part. -- Tom
signature.asc
Description: PGP signature
