On Tue, 24 Dec 2019 at 09:51, Eugeniu Rosca <roscaeuge...@gmail.com> wrote: > > Rename the existing 'dtimg' command to 'adtimg', in order to: > - Suggest the Android origins and scope > - Be consistent with the upcoming 'abootimg' command (naming > suggested by Simon [*]) > > The change in _not_ backward compatible, but its benefits outweigh its > downsides, given that we don't expect active users of 'dtimg' today. > > Perform the rename in several steps: > 1. Rename *.c file and Kconfig symbol. This should allow > 'git log --follow' to properly track the history of 'adtimg.c' > 2. 's/dtimg/adtimg/g' in the internal namespace of 'adtimg.c' > > ELF comparison [**] before and after shows no functional change. > > [*] https://patchwork.ozlabs.org/patch/1182212/#2291600 > [**] diff -u <(objdump -d cmd/dtimg.o) <(objdump -d cmd/adtimg.o) > > Cc: Tom Rini <tr...@konsulko.com> > Cc: Sam Protsenko <semen.protse...@linaro.org> > Cc: Simon Glass <s...@chromium.org> > Signed-off-by: Eugeniu Rosca <ero...@de.adit-jv.com> > --- > cmd/Kconfig | 4 ++-- > cmd/Makefile | 2 +- > cmd/{dtimg.c => adtimg.c} | 0 > common/Makefile | 2 +- > configs/am57xx_evm_defconfig | 2 +- > configs/am57xx_hs_evm_defconfig | 2 +- > configs/am57xx_hs_evm_usb_defconfig | 2 +- > 7 files changed, 7 insertions(+), 7 deletions(-) > rename cmd/{dtimg.c => adtimg.c} (100%) >
Reviewed-by: Simon Glass<s...@chromium.org>