diff mbox

arm64: Add dtb files to archclean rule

Message ID FF2CE3A3-56DA-4637-833F-264DD986F28C@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Jungseok Lee Jan. 17, 2015, 1:59 p.m. UTC
As dts files have been reorganised under vendor subdirs, dtb files
cannot be removed with "make distclean" now. Thus, this patch adds
dtb files to archclean rule.

Cc: Robert Richter <rrichter@cavium.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Signed-off-by: Jungseok Lee <jungseoklee85@gmail.com>
---
 arch/arm64/Makefile | 1 +
 1 file changed, 1 insertion(+)

Comments

Will Deacon Jan. 19, 2015, 10:39 a.m. UTC | #1
On Sat, Jan 17, 2015 at 01:59:31PM +0000, Jungseok Lee wrote:
> As dts files have been reorganised under vendor subdirs, dtb files
> cannot be removed with "make distclean" now. Thus, this patch adds
> dtb files to archclean rule.
> 
> Cc: Robert Richter <rrichter@cavium.com>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
> Cc: Will Deacon <will.deacon@arm.com>
> Signed-off-by: Jungseok Lee <jungseoklee85@gmail.com>
> ---
>  arch/arm64/Makefile | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/Makefile b/arch/arm64/Makefile
> index 1c43cec..0666888 100644
> --- a/arch/arm64/Makefile
> +++ b/arch/arm64/Makefile
> @@ -85,6 +85,7 @@ vdso_install:
>  # We use MRPROPER_FILES and CLEAN_FILES now
>  archclean:
>  	$(Q)$(MAKE) $(clean)=$(boot)
> +	$(Q)$(MAKE) $(clean)=$(boot)/dts

Looks sensible. I think we can also remove the always/clean-files entries
from arch/arm64/boot/dts/Makefile.

Will
Jungseok Lee Jan. 19, 2015, 2:48 p.m. UTC | #2
On Jan 19, 2015, at 7:39 PM, Will Deacon wrote:
> On Sat, Jan 17, 2015 at 01:59:31PM +0000, Jungseok Lee wrote:
>> As dts files have been reorganised under vendor subdirs, dtb files
>> cannot be removed with "make distclean" now. Thus, this patch adds
>> dtb files to archclean rule.
>> 
>> Cc: Robert Richter <rrichter@cavium.com>
>> Cc: Catalin Marinas <catalin.marinas@arm.com>
>> Cc: Will Deacon <will.deacon@arm.com>
>> Signed-off-by: Jungseok Lee <jungseoklee85@gmail.com>
>> ---
>> arch/arm64/Makefile | 1 +
>> 1 file changed, 1 insertion(+)
>> 
>> diff --git a/arch/arm64/Makefile b/arch/arm64/Makefile
>> index 1c43cec..0666888 100644
>> --- a/arch/arm64/Makefile
>> +++ b/arch/arm64/Makefile
>> @@ -85,6 +85,7 @@ vdso_install:
>> # We use MRPROPER_FILES and CLEAN_FILES now
>> archclean:
>> 	$(Q)$(MAKE) $(clean)=$(boot)
>> +	$(Q)$(MAKE) $(clean)=$(boot)/dts
> 
> Looks sensible. I think we can also remove the always/clean-files entries
> from arch/arm64/boot/dts/Makefile.

I agree with the point.
I will do respin with that change.

Best Regards
Jungseok Lee
diff mbox

Patch

diff --git a/arch/arm64/Makefile b/arch/arm64/Makefile
index 1c43cec..0666888 100644
--- a/arch/arm64/Makefile
+++ b/arch/arm64/Makefile
@@ -85,6 +85,7 @@  vdso_install:
 # We use MRPROPER_FILES and CLEAN_FILES now
 archclean:
 	$(Q)$(MAKE) $(clean)=$(boot)
+	$(Q)$(MAKE) $(clean)=$(boot)/dts
 
 define archhelp
   echo  '* Image.gz      - Compressed kernel image (arch/$(ARCH)/boot/Image.gz)'