Hi Fabio, Am Do., 19. Aug. 2021 um 21:28 Uhr schrieb Fabio Estevam <feste...@denx.de>: > > Clean the binaries generated by binman on imx8mm-evk: > spl.* mkimage*.mkimage imx-boot.* > > Reported-by: Frieder Schrempf <frieder.schre...@kontron.de> > Signed-off-by: Fabio Estevam <feste...@denx.de> > --- > Changes since v2: > - None. Newly introducedin this series. > > Makefile | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index 3c8437d21a..7096fdf895 100644 > --- a/Makefile > +++ b/Makefile > @@ -2095,7 +2095,8 @@ CLEAN_FILES += include/bmp_logo.h > include/bmp_logo_data.h tools/version.h \ > boot* u-boot* MLO* SPL System.map fit-dtb.blob* \ > u-boot-ivt.img.log u-boot-dtb.imx.log SPL.log u-boot.imx.log \ > lpc32xx-* bl31.c bl31.elf bl31_*.bin image.map tispl.bin* \ > - idbloader.img flash.bin flash.log defconfig keep-syms-lto.c > + idbloader.img flash.bin flash.log defconfig keep-syms-lto.c \ > + spl.* mkimage*.mkimage imx-boot.*
it might be useful to use one variable for all BINMAN clean files. Otherwise it is difficult to understand by whom the files were created. Something like that: --- a/Makefile +++ b/Makefile @@ -2091,12 +2091,14 @@ CLEAN_DIRS += $(MODVERDIR) \ $(foreach d, spl tpl, $(patsubst %,$d/%, \ $(filter-out include, $(shell ls -1 $d 2>/dev/null)))) +BINMAN_CLEAN_FILES = spl.* mkimage*.mkimage + CLEAN_FILES += include/bmp_logo.h include/bmp_logo_data.h tools/version.h \ boot* u-boot* MLO* SPL System.map fit-dtb.blob* \ u-boot-ivt.img.log u-boot-dtb.imx.log SPL.log u-boot.imx.log \ lpc32xx-* bl31.c bl31.elf bl31_*.bin image.map tispl.bin* \ idbloader.img flash.bin flash.log defconfig keep-syms-lto.c \ - spl.* mkimage*.mkimage imx-boot.* + imx-boot.* $(BINMAN_CLEAN_FILES) + I had the idea, if binman could create a list with the created files. This could then be used to set the files to be deleted. I think of other users where the output files have a different name. Then the files could be deleted "automatically" with clean. -- Heiko