From mboxrd@z Thu Jan  1 00:00:00 1970
From: Auer, Lukas <lukas.auer@aisec.fraunhofer.de>
Date: Sun, 10 Feb 2019 18:31:02 +0000
Subject: [U-Boot] [PATCH v6 02/16] Makefile: Fix mrproper make target
In-Reply-To: <20190209063052.29092-3-anup.patel@wdc.com>
References: <20190209063052.29092-1-anup.patel@wdc.com>
 <20190209063052.29092-3-anup.patel@wdc.com>
Message-ID: <85104a49adc4b1e3adb7fed91840ff855dcbe8ef.camel@aisec.fraunhofer.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sat, 2019-02-09 at 06:31 +0000, Anup Patel wrote:
> Currently, the mrproper make target tries to force remove all
> "arch/*/include/asm/arch" paths assuming they are symlinks but
> this prevents us from adding place-holder headers under the
> arch/riscv/include/asm/arch directory.
> 
> To solve this, we fix mrproper make target to only remove
> "arch/*/include/asm/arch" paths which are symlinks and do
> nothing for paths which are directory.
> 
> Signed-off-by: Anup Patel <anup.patel@wdc.com>
> ---
>  Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Makefile b/Makefile
> index cf7b2b10bc..2d05b8da9a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1782,7 +1782,7 @@ $(mrproper-dirs):
>  mrproper: clean $(mrproper-dirs)
>  	$(call cmd,rmdirs)
>  	$(call cmd,rmfiles)
> -	@rm -f arch/*/include/asm/arch
> +	@for d in `dirname arch/*/include/asm/arch`; do if [ -L
> $$d/arch ]; then echo "  CLEAN   "$$d/arch; rm -f $$d/arch; fi; done
>  
>  # distclean
>  #

This looks good to me, but I am not familiar enough with the other
architectures to know if this could affect them somehow. I have added
Tom on CC. Maybe he can weigh in on this.

Thanks,
Lukas