Re: [PATCH 3/3] kbuild: clean up *.dtb and *.dtb.S patterns from top-level Makefile

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tue, Oct 31, 2017 at 01:32:15AM +0900, Masahiro Yamada wrote:
> Hi Rob,
> 
> 2017-10-31 0:53 GMT+09:00 Rob Herring <robh+dt@xxxxxxxxxx>:
> > On Mon, Oct 30, 2017 at 10:33 AM, Masahiro Yamada
> > <yamada.masahiro@xxxxxxxxxxxxx> wrote:
> >> We need to add "clean-files" in Makfiles to clean up DT blobs, but we
> >> often miss to do so.
> >>
> >> Since there are no source files that end with .dtb or .dtb.S, so we
> >> can clean-up those files from the top-level Makefile.
> >>
> >> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
> >> ---
> >>
> >>  Makefile | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/Makefile b/Makefile
> >> index 63a4c0e..c577c63 100644
> >> --- a/Makefile
> >> +++ b/Makefile
> >> @@ -1544,7 +1544,7 @@ clean: $(clean-dirs)
> >>         $(call cmd,rmfiles)
> >>         @find $(if $(KBUILD_EXTMOD), $(KBUILD_EXTMOD), .) $(RCS_FIND_IGNORE) \
> >>                 \( -name '*.[oas]' -o -name '*.ko' -o -name '.*.cmd' \
> >> -               -o -name '*.ko.*' \
> >> +               -o -name '*.ko.*' -o -name '*.dtb' -o -name '*.dtb.S' \
> >
> > With this we can remove all the occurrences in the kernel:
> >
> > $ git grep 'clean.*dtb'
> >
> > Otherwise, the series looks good to me. I'm happy to apply it if the
> > kbuild folks don't.
> >
> > Rob
> 
> Yes, please.  Thanks!

Can you post a follow-up patch removing all the clean-files lines.

Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux