Hello Pali,
On 28.12.22 19:18, Pali Rohár wrote:
> U-Boot build system builds final U-Boot binary for socrates board in custom
> file u-boot-socrates.bin (instead of standard u-boot.bin). Output target
> file u-boot-socrates.bin is generated by binman as defined in board binman
> config file arch/powerpc/dts/socrates-u-boot.dtsi.
>
> But binman was disabled in commit 5af42eafd7e1 ("Makefile: Reduce usage of
> custom mpc85xx u-boot.bin target") for all mpc85xx boards which do not use
> standard powerpc binman config file arch/powerpc/dts/u-boot.dtsi and boards
> which do not require binman at all.
>
> The only such mpc85xx board is socrates. So since that commit, U-Boot does
> not final binary for socrates board anymore.
>
> Fix this issue by re-enabling binman for socrates board. And build process
> starts again producing u-boot-socrates.bin binary.
>
> Note that build process for this socrates board always produce u-boot.bin
> binary which is broken and not usable for socrates board. Long term
> solution should be to disable building broken binary u-boot.bin and then
> renaming u-boot-socrates.bin to u-boot.bin, or switching to use common
> powerpc binman config file arch/powerpc/dts/socrates-u-boot.dtsi (if it is
> possible).
>
> Fixes: 5af42eafd7e1 ("Makefile: Reduce usage of custom mpc85xx u-boot.bin
> target")
> Signed-off-by: Pali Rohár <p...@kernel.org>
> ---
> Heiko Schocher: Could you test if u-boot is still working on this board?
>
> Tom Rini: Cannot be this issue handled by CI? For example that CI check
> build process produce required output binaries?
> ---
> arch/powerpc/cpu/mpc85xx/Kconfig | 1 +
> 1 file changed, 1 insertion(+)
With this patch, u-boot-socrates.bin is build again, so yes...
Tested-by: Heiko Schocher <h...@denx.de>
... but current u-boot does not boot anymore on this board ... I have to
dig into, obvious difference I see in hexdump is:
old (2022.01) u-boot:
"""
00001930 74 65 00 6f 66 66 73 65 74 00 73 74 64 6f 75 74 |te.offset.stdout|
00001940 2d 70 61 74 68 00 ff ff ff ff ff ff ff ff ff ff |-path...........|
00001950 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |................|
*
00020000 27 05 19 56 3c 60 e4 01 60 63 3f 10 38 63 fb f0 |'..V<`..`c?.8c..|
00020010 3c 80 e4 01 60 84 40 00 38 00 00 00 38 84 ff fc |<...`.@.8...8...|
00020020 90 04 00 00 7c 04 18 40 40 82 ff f4 3c 80 e4 01 |....|..@@...<...|
"""
New
"""
00001930 74 65 00 6f 66 66 73 65 74 00 73 74 64 6f 75 74 |te.offset.stdout|
00001940 2d 70 61 74 68 00 ff ff ff ff ff ff ff ff ff ff |-path...........|
00001950 ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff |................|
*
00020000 3c 60 e4 01 60 63 3f 10 38 63 fb f0 3c 80 e4 01 |<`..`c?.8c..<...|
00020010 60 84 40 00 38 00 00 00 38 84 ff fc 90 04 00 00 |`.@.8...8.......|
00020020 7c 04 18 40 40 82 ff f4 3c 80 e4 01 60 84 3f 20 ||..@@...<...`.? |
"""
So "U-Boot magic" is misssing ... reset vector at end of image is for both
the same:
000bfff0 ff ff ff ff ff ff ff ff ff ff ff ff 4b ff f0 04 |............K...|
000c0000
I have to dig deeper into it, to find out what have changed in the meantime,
(Think I start a "git bisect") just find some more time for it...
Nevertheless, I think, this patch can go in...
bye,
Heiko
--
DENX Software Engineering GmbH, Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52 Fax: +49-8142-66989-80 Email: h...@denx.de
