From: Johan Jonker <jbx6244@gmail.com>
To: Quentin Schulz <foss+uboot@0leil.net>
Cc: bharat.gooty@broadcom.com, rayagonda.kokatanur@broadcom.com,
sjg@chromium.org, philipp.tomsich@vrull.eu,
kever.yang@rock-chips.com, jagan@amarulasolutions.com,
alpernebiyasak@gmail.com, xypron.glpk@gmx.de,
heiko.thiery@gmail.com, u-boot@lists.denx.de,
Quentin Schulz <quentin.schulz@theobroma-systems.com>
Subject: Re: [PATCH v3 3/8] rockchip: remove binman temporary files when cleaning
Date: Mon, 1 Aug 2022 20:59:42 +0200 [thread overview]
Message-ID: <2ede24c8-ab44-e1b2-1f31-dd100ea84f28@gmail.com> (raw)
In-Reply-To: <20220801160610.2330151-4-foss+uboot@0leil.net>
Hi Quentin,
Beside removing the "real" annoyance is that these files popup when adding with "git add ." and are blocking branch changing with "git branch ...".
Maybe add to ".gitignore" as well or delete immediately. What's the use to keep them around?
Johan
On 8/1/22 18:06, Quentin Schulz wrote:
> From: Quentin Schulz <quentin.schulz@theobroma-systems.com>
>
> Binman mkimage entry generates temporary files so let's remove them
> when calling `make clean`.
>
> Fixes: 9b312e26fc77 ("rockchip: Enable building a SPI ROM image on jerry")
> Cc: Quentin Schulz <foss+uboot@0leil.net>
> Reported-by: Johan Jonker <jbx6244@gmail.com>
> Signed-off-by: Quentin Schulz <quentin.schulz@theobroma-systems.com>
> ---
>
> added in v3
>
> Makefile | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index ff25f92974..bbb2a4c088 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2220,7 +2220,8 @@ CLEAN_FILES += include/bmp_logo.h include/bmp_logo_data.h \
> lpc32xx-* bl31.c bl31.elf bl31_*.bin image.map tispl.bin* \
> idbloader.img flash.bin flash.log defconfig keep-syms-lto.c \
> mkimage-out.spl.mkimage mkimage.spl.mkimage imx-boot.map \
> - itb.fit.fit itb.fit.itb itb.map spl.map
> + itb.fit.fit itb.fit.itb itb.map spl.map mkimage-out.rom.mkimage \
> + mkimage.rom.mkimage rom.map simple-bin.map
>
> # Directories & files removed with 'make mrproper'
> MRPROPER_DIRS += include/config include/generated spl tpl \
next prev parent reply other threads:[~2022-08-01 18:59 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-01 16:06 [PATCH v3 0/8] migrate u-boot-rockchip.bin to binman and generate an image for SPI Quentin Schulz
2022-08-01 16:06 ` [PATCH v3 1/8] binman: add support for skipping file concatenation for mkimage Quentin Schulz
2022-08-01 16:06 ` [PATCH v3 2/8] binman: allow user-defined filenames for mkimage entry Quentin Schulz
2022-08-01 19:13 ` Simon Glass
2022-08-01 16:06 ` [PATCH v3 3/8] rockchip: remove binman temporary files when cleaning Quentin Schulz
2022-08-01 18:59 ` Johan Jonker [this message]
2022-08-01 19:13 ` Simon Glass
2022-08-01 16:06 ` [PATCH v3 4/8] rockchip: generate idbloader.img content for u-boot-rockchip.bin with binman for ARM Quentin Schulz
2022-08-01 19:13 ` Simon Glass
2022-08-01 16:06 ` [PATCH v3 5/8] rockchip: generate u-boot-rockchip.bin with binman for ARM64 boards Quentin Schulz
2022-08-01 19:13 ` Simon Glass
2022-08-01 16:06 ` [PATCH v3 6/8] rockchip: simplify binman image dependencies addition to INPUTS Quentin Schulz
2022-08-01 16:24 ` Quentin Schulz
2022-08-01 16:06 ` [PATCH v3 7/8] rockchip: allow to build SPI images even without HAS_ROM option Quentin Schulz
2022-08-01 16:06 ` [PATCH v3 8/8] rockchip: add u-boot-rockchip-spi.bin image for booting from SPI-NOR flash Quentin Schulz
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=2ede24c8-ab44-e1b2-1f31-dd100ea84f28@gmail.com \
--to=jbx6244@gmail.com \
--cc=alpernebiyasak@gmail.com \
--cc=bharat.gooty@broadcom.com \
--cc=foss+uboot@0leil.net \
--cc=heiko.thiery@gmail.com \
--cc=jagan@amarulasolutions.com \
--cc=kever.yang@rock-chips.com \
--cc=philipp.tomsich@vrull.eu \
--cc=quentin.schulz@theobroma-systems.com \
--cc=rayagonda.kokatanur@broadcom.com \
--cc=sjg@chromium.org \
--cc=u-boot@lists.denx.de \
--cc=xypron.glpk@gmx.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.