Path: csiph.com!weretis.net!feeder4.news.weretis.net!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From: Masahiro Yamada <yamada.masahiro@socionext.com>
Newsgroups: linux.kernel
Subject: Re: [PATCH] ARM: remove 'i' and 'zi' targets
Date: Mon, 15 Feb 2016 12:30:02 +0100
Message-ID: <r2pvQ-e6-7@gated-at.bofh.it>
References: <qZuil-4hB-1@gated-at.bofh.it>
X-Nifty-Srcip: [209.85.161.174]
X-Gm-Message-State: AG10YOQfg2VKF/e9FBkiV/z329jrA5YS5E+5ABgL/Oxspibvj5olL8VI/osIAjYWAOA9FQUvDktKbZ16n4ujkw==
MIME-Version: 1.0
X-Received: by 10.129.83.196 with SMTP id h187mr8496378ywb.319.1455535562320; Mon, 15 Feb 2016 03:26:02 -0800 (PST)
X-Gmail-Original-Message-ID: <CAK7LNASDxBT8J3A70STLCedQTXiwfK+vxo7OVJAgUUORT=pqJQ@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Sender: robomod@news.nic.it
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Approved: robomod@news.nic.it
Lines: 64
Organization: linux.* mail to news gateway
X-Original-Cc: Michal Marek <mmarek@suse.cz>, Russell King <linux@arm.linux.org.uk>, Arnd Bergmann <arnd@arndb.de>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Masahiro Yamada <yamada.masahiro@socionext.com>, Olof Johansson <olof@lixom.net>, Russell King <rmk+kernel@arm.linux.org.uk>
X-Original-Date: Mon, 15 Feb 2016 20:26:02 +0900
X-Original-Message-ID: <CAK7LNASDxBT8J3A70STLCedQTXiwfK+vxo7OVJAgUUORT=pqJQ@mail.gmail.com>
X-Original-References: <1454839003-4342-1-git-send-email-yamada.masahiro@socionext.com>
X-Original-Sender: linux-kernel-owner@vger.kernel.org
Xref: csiph.com linux.kernel:1334407

2016-02-07 18:56 GMT+09:00 Masahiro Yamada <yamada.masahiro@socionext.com>:
> These two were introduced by commit 13d5fadf45d1 ("[ARM] Make 'i' and
> 'zi' targets work") to short-circuit the dependencies for 'install'
> and 'zinstall'.
>
> After that, commit 19514fc665ff ('arm, kbuild: make "make install"
> not depend on vmlinux') eventually made "make (z)install" equivalent
> to "make (z)i".
>
> It is true that 'i' and 'zi' might be still useful as shorthands
> but the original intention had been already lost.
>
> They do not even show up in "make ARCH=arm help", so I hope this
> deletion does not have much impact.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
>
>  arch/arm/Makefile      | 1 -
>  arch/arm/boot/Makefile | 8 --------
>  2 files changed, 9 deletions(-)
>
> diff --git a/arch/arm/Makefile b/arch/arm/Makefile
> index cd21135..1e83a45 100644
> --- a/arch/arm/Makefile
> +++ b/arch/arm/Makefile
> @@ -353,7 +353,6 @@ archclean:
>
>  # My testing targets (bypasses dependencies)
>  bp:;   $(Q)$(MAKE) $(build)=$(boot) MACHINE=$(MACHINE) $(boot)/bootpImage
> -i zi:; $(Q)$(MAKE) $(build)=$(boot) MACHINE=$(MACHINE) $@
>
>
>  define archhelp
> diff --git a/arch/arm/boot/Makefile b/arch/arm/boot/Makefile
> index ba73a20..48fab15 100644
> --- a/arch/arm/boot/Makefile
> +++ b/arch/arm/boot/Makefile
> @@ -107,12 +107,4 @@ uinstall:
>         $(CONFIG_SHELL) $(srctree)/$(src)/install.sh "$(KERNELRELEASE)" \
>         $(obj)/uImage System.map "$(INSTALL_PATH)"
>
> -zi:
> -       $(CONFIG_SHELL) $(srctree)/$(src)/install.sh "$(KERNELRELEASE)" \
> -       $(obj)/zImage System.map "$(INSTALL_PATH)"
> -
> -i:
> -       $(CONFIG_SHELL) $(srctree)/$(src)/install.sh "$(KERNELRELEASE)" \
> -       $(obj)/Image System.map "$(INSTALL_PATH)"
> -
>  subdir-            := bootp compressed dts
> --


If there is no objection to this patch,
I will put it into Russell's patch tracker.





-- 
Best Regards
Masahiro Yamada