From: Heiko Schocher <hs@denx.de>
To: "Pali Rohár" <pali@kernel.org>, "Tom Rini" <trini@konsulko.com>
Cc: u-boot@lists.denx.de
Subject: Re: [PATCH u-boot] powerpc/mpc85xx: socrates: Re-enable building u-boot-socrates.bin
Date: Sat, 31 Dec 2022 10:36:07 +0100 [thread overview]
Message-ID: <80cb040a-ccc1-e022-2a37-34e682b46918@denx.de> (raw)
In-Reply-To: <20221228181839.22003-1-pali@kernel.org>
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 <pali@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 <hs@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: hs@denx.de
next prev parent reply other threads:[~2022-12-31 9:36 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-28 18:18 [PATCH u-boot] powerpc/mpc85xx: socrates: Re-enable building u-boot-socrates.bin Pali Rohár
2022-12-29 0:11 ` Tom Rini
2022-12-29 1:34 ` Pali Rohár
2022-12-29 7:01 ` Heiko Schocher
2022-12-30 12:44 ` Pali Rohár
2022-12-30 15:20 ` Tom Rini
2022-12-31 9:36 ` Heiko Schocher [this message]
2022-12-31 12:58 ` Pali Rohár
2022-12-31 13:36 ` Heiko Schocher
2022-12-31 15:31 ` Heiko Schocher
2022-12-31 15:37 ` Pali Rohár
2023-01-01 6:57 ` Heiko Schocher
2023-01-02 8:29 ` Heiko Schocher
2023-01-11 0:09 ` Pali Rohár
2023-01-06 16:52 ` Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=80cb040a-ccc1-e022-2a37-34e682b46918@denx.de \
--to=hs@denx.de \
--cc=pali@kernel.org \
--cc=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.