On 03. 12. 18 8:50, Masahiro Yamada wrote: > "make ARCH=microblaze help" mentions simpleImage.<dt>.unstrip, > but it never works because Makefile assumes "system.unstrip" is > the name of DT. > > $ make ARCH=microblaze CROSS_COMPILE=microblaze-linux- simpleImage.system.unstrip > [ snip ] > make[1]: *** No rule to make target 'arch/microblaze/boot/dts/system.unstrip.dtb', needed by 'arch/microblaze/boot/dts/system.dtb'. Stop. > make: *** [Makefile;1060: arch/microblaze/boot/dts] Error 2 > make: *** Waiting for unfinished jobs.... > > Rip off the never-working target. > > In my understanding, simpleImage.<dt> works like a phony target that > generates multiple images. Reflect the behavior to the help message. > > Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> > --- > > arch/microblaze/Makefile | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/arch/microblaze/Makefile b/arch/microblaze/Makefile > index 0823d29..97e1384 100644 > --- a/arch/microblaze/Makefile > +++ b/arch/microblaze/Makefile > @@ -89,9 +89,7 @@ define archhelp > echo '* linux.bin - Create raw binary' > echo ' linux.bin.gz - Create compressed raw binary' > echo ' linux.bin.ub - Create U-Boot wrapped raw binary' > - echo ' simpleImage.<dt> - ELF image with $(arch)/boot/dts/<dt>.dts linked in' > - echo ' - stripped elf with fdt blob' > - echo ' simpleImage.<dt>.unstrip - full ELF image with fdt blob' > + echo ' simpleImage.<dt> - Create images with $(arch)/boot/dts/<dt>.dts linked in' > echo ' *_defconfig - Select default config from arch/microblaze/configs' > echo '' > echo ' Targets with <dt> embed a device tree blob inside the image' > I understand what you are trying to say but I would still like to keep information about unstrip file. It is correct that it is not build target. It is just generated file and message above was used for description what it is. Definitely agree that this should be the part of targets but it should be in description. The same is for missing description for simpleImage.<dt>.strip and simpleImage.<dt>.ub files. Thanks, 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
