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
