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

Tom Rini
2021-09-16T19:17:25Z

[-- Attachment #1: Type: text/plain, Size: 927 bytes --]

On Sun, Aug 08, 2021 at 01:20:38PM +0200, Pali Rohár wrote:

> It is unknown why version string is placed at specific position on these
> powerpc mpc platforms. But there is no need to overload version_string
> symbol. Just use common definition of version_string and modify linker
> script to put it at "correct place".
>
> Signed-off-by: Pali Rohár <pali@kernel.org>
> ---
> Changes in v2:
> * Put explicit ".section" keyword before declaring ".text_pre" section as
> some gcc versions cannot recognize specifying custom section without it.
> * Tested compilation for:
> $ make CROSS_COMPILE=powerpc-linux-gnu- MCR3000_defconfig u-boot.bin
> and checked that u-boot.bin header is same with and without this patch

Please note that qemu-ppce500 still fails to build, now with a different
error:
https://source.denx.de/u-boot/u-boot/-/jobs/323027
which I am investigating how to resolve.

--
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]