Re: [PATCH 0/7] microblaze: fix various problems in building boot images

From: Michal Simek
Date: Wed Dec 05 2018 - 11:41:13 EST


On 03. 12. 18 8:50, Masahiro Yamada wrote:
> This patch set fixes various issues in microblaze Makefiles.
>
> BTW, "simpleImage.<dt>" works like a phony target to generate the
> following four images, where the first three are just aliases.
>
> - arch/microblaze/boot/simpleImage.<dt>:
> identical to arch/microblaze/boot/linux.bin

It is not - fdt section should be empty.
simpleImage has this section filled.

>
> - arch/microblaze/boot/simpleImage.<dt>.unstrip:
> identical to vmlinux

The same here with filled section.

>
> - arch/microblaze/boot/simpleImage.<dt>.ub:
> identical to arch/microblaze/boot/linux.bin.ub

as above.

>
> - arch/microblaze/boot/simpleImage.<dt>.strip:
> stripped vmlinux

And this is there because unstrip version is quite huge and for early
issues you need to know only some symbols that's why debugger is not
overflow with stuff which none needs.
Maybe this is not an issue now but that strip version is used a lot.


>
> I am not sure how much useful those copies are,
> but, I tried my best to keep the same behavior.
>
> IMHO, I guess DTB=<dt> would be more sensible,
> but it is up to Michal.

What do you mean by this exactly?

Definitely thanks for looking at this.
Michal



--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs


Attachment: signature.asc
Description: OpenPGP digital signature