On Sat, Jul 31, 2021 at 5:30 AM Nick Desaulniers <ndesaulni...@google.com> wrote: > > On Thu, Jul 29, 2021 at 7:22 AM Masahiro Yamada <masahi...@kernel.org> wrote: > > > > Currently, the install target in arch/powerpc/Makefile descends into > > arch/powerpc/boot/Makefile to invoke the shell script, but there is no > > good reason to do so. > > Sure, but there are more arch/ subdirs that DO invoke install.sh from > arch/<arch>/boot/Makefile than, not: > > arch/<arch>/boot/Makefile: > - parisc > - nios2 > - arm > - nds32 > - sparc > - riscv > - 390 > - ppc (this patch) > - x86 > - arm64
I sent patches for these architectures. Check LKML. > arch/<arch>/Makefile: > - ia64 > - m68k > > Patch is fine, but right now the tree is a bit inconsistent. > > > > > arch/powerpc/Makefile can run the shell script directly. > > > > Signed-off-by: Masahiro Yamada <masahi...@kernel.org> > > --- > > > > arch/powerpc/Makefile | 3 ++- > > arch/powerpc/boot/Makefile | 6 ------ > > 2 files changed, 2 insertions(+), 7 deletions(-) > > > > diff --git a/arch/powerpc/Makefile b/arch/powerpc/Makefile > > index 6505d66f1193..9aaf1abbc641 100644 > > --- a/arch/powerpc/Makefile > > +++ b/arch/powerpc/Makefile > > @@ -407,7 +407,8 @@ endef > > > > PHONY += install > > install: > > - $(Q)$(MAKE) $(build)=$(boot) install > > + sh -x $(srctree)/$(boot)/install.sh "$(KERNELRELEASE)" vmlinux \ > > + System.map "$(INSTALL_PATH)" > > > > archclean: > > $(Q)$(MAKE) $(clean)=$(boot) > > diff --git a/arch/powerpc/boot/Makefile b/arch/powerpc/boot/Makefile > > index 0d165bd98b61..10c0fb306f15 100644 > > --- a/arch/powerpc/boot/Makefile > > +++ b/arch/powerpc/boot/Makefile > > @@ -444,12 +444,6 @@ $(obj)/zImage: $(addprefix $(obj)/, > > $(image-y)) > > $(obj)/zImage.initrd: $(addprefix $(obj)/, $(initrd-y)) > > $(Q)rm -f $@; ln $< $@ > > > > -# Only install the vmlinux > > -install: > > - sh -x $(srctree)/$(src)/install.sh "$(KERNELRELEASE)" vmlinux > > System.map "$(INSTALL_PATH)" > > - > > -PHONY += install > > - > > # anything not in $(targets) > > clean-files += $(image-) $(initrd-) cuImage.* dtbImage.* treeImage.* \ > > zImage zImage.initrd zImage.chrp zImage.coff zImage.holly \ > > -- > > 2.27.0 > > > > > -- > Thanks, > ~Nick Desaulniers -- Best Regards Masahiro Yamada