Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-17 Thread Max Filippov
On Fri, Sep 17, 2021 at 5:04 AM Tom Rini wrote: > 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/ Yes, I've found that series and the purpose

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-17 Thread Tom Rini
On Thu, Sep 16, 2021 at 10:14:03PM -0700, Max Filippov wrote: > On Thu, Sep 16, 2021 at 10:30 AM Tom Rini 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 > > --- > > arch/xtensa/inclu

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-17 Thread Tom Rini
On Thu, Sep 16, 2021 at 10:21:29PM -0700, Max Filippov wrote: > On Thu, Sep 16, 2021 at 12:42 PM Tom Rini wrote: > > We also don't have qemu support for it > > I'm curious what happened to it and what I should do to update it? > > xtensa is still supported in the QEMU and AFAIK nothing has chan

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Max Filippov
On Thu, Sep 16, 2021 at 12:42 PM Tom Rini wrote: > We also don't have qemu support for it I'm curious what happened to it and what I should do to update it? xtensa is still supported in the QEMU and AFAIK nothing has changed about it: neither building nor invocation. -- Thanks. -- Max

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Max Filippov
On Thu, Sep 16, 2021 at 10:30 AM Tom Rini 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 > --- > arch/xtensa/include/asm/ldscript.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arc

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Francesco Dolcini
On Thu, Sep 16, 2021 at 04:13:31PM -0400, Tom Rini wrote: > On Thu, Sep 16, 2021 at 09:50:35PM +0200, Francesco Dolcini wrote: > > On Thu, Sep 16, 2021 at 03:42:20PM -0400, Tom Rini wrote: > > > On Thu, Sep 16, 2021 at 09:38:19PM +0200, Francesco Dolcini wrote: > > > > On Thu, Sep 16, 2021 at 01:30

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Tom Rini
On Thu, Sep 16, 2021 at 09:50:35PM +0200, Francesco Dolcini wrote: > On Thu, Sep 16, 2021 at 03:42:20PM -0400, Tom Rini wrote: > > On Thu, Sep 16, 2021 at 09:38:19PM +0200, Francesco Dolcini wrote: > > > On Thu, Sep 16, 2021 at 01:30:21PM -0400, Tom Rini wrote: > > > > *(.literal .t

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Francesco Dolcini
On Thu, Sep 16, 2021 at 03:42:20PM -0400, Tom Rini wrote: > On Thu, Sep 16, 2021 at 09:38:19PM +0200, Francesco Dolcini wrote: > > On Thu, Sep 16, 2021 at 01:30:21PM -0400, Tom Rini wrote: > > > *(.literal .text) \ > > > + *(.literal .text_ver

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Tom Rini
On Thu, Sep 16, 2021 at 09:38:19PM +0200, Francesco Dolcini wrote: > On Thu, Sep 16, 2021 at 01:30:21PM -0400, Tom Rini wrote: > > *(.literal .text) \ > > + *(.literal .text_version_string)\ > > Isn't ".litteral"

Re: [PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Francesco Dolcini
On Thu, Sep 16, 2021 at 01:30:21PM -0400, Tom Rini wrote: > *(.literal .text) \ > + *(.literal .text_version_string)\ Isn't ".litteral" a duplication? Even if I'm pretty sure it will not cause any difference i

[PATCH] xtensa: Put U-Boot version string at correct place by linker script

2021-09-16 Thread Tom Rini
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 --- 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.