From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751571AbdCRNoz (ORCPT ); Sat, 18 Mar 2017 09:44:55 -0400 Received: from condef-07.nifty.com ([202.248.20.72]:46738 "EHLO condef-07.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751337AbdCRNox (ORCPT ); Sat, 18 Mar 2017 09:44:53 -0400 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-06.nifty.com v2IDWvbV014586 X-Nifty-SrcIP: [209.85.161.172] MIME-Version: 1.0 In-Reply-To: References: <20170118173156.16060-1-mcgrof@kernel.org> <20170123150727.4883-1-mcgrof@kernel.org> <20170315005348.GK28800@wotan.suse.de> From: Masahiro Yamada Date: Sat, 18 Mar 2017 22:32:56 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v2] firmware/Makefile: force recompilation if makefile changes To: "Luis R. Rodriguez" Cc: Andrew Morton , Michal Marek , Linux Kbuild mailing list , Ming Lei , Greg Kroah-Hartman , Tom Gundersen , Linux Kernel Mailing List , David Woodhouse Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Luis, 2017-03-17 2:55 GMT+09:00 Luis R. Rodriguez : > On Thu, Mar 16, 2017 at 10:43 AM, Masahiro Yamada > wrote: >> Hi Luis, >> >> 2017-03-15 9:53 GMT+09:00 Luis R. Rodriguez : >>> On Sat, Mar 11, 2017 at 02:37:02PM +0900, Masahiro Yamada wrote: >>>> Hi Luis, >>>> >>>> >>>> 2017-01-24 0:07 GMT+09:00 Luis R. Rodriguez : >>>> > If you modify the target asm we currently do not force the >>>> > recompilation of the firmware files. The target asm is in >>>> > the firmware/Makefile, peg this file as a dependency to >>>> > require re-compilation of firmware targets when the asm >>>> > changes. >>>> > >>>> > Signed-off-by: Luis R. Rodriguez >>>> > --- >>>> > >>>> > Michal, >>>> > >>>> > I had this patch as part of my linker table series [0] but have split it >>>> > off as its a small atomic separate change and can go in separately. Greg >>>> > prefers this be reviewed by the kbuild tree so sending it your way. >>>> > This v2 has no modifications, just resending it to the kbuild tree. >>>> > >>>> > [0] https://lkml.kernel.org/r/20170115211057.17167-1-mcgrof@kernel.org >>>> > >>>> > firmware/Makefile | 3 ++- >>>> > 1 file changed, 2 insertions(+), 1 deletion(-) >>>> > >>>> > diff --git a/firmware/Makefile b/firmware/Makefile >>>> > index e297e1b52636..fa3e81c2a97b 100644 >>>> > --- a/firmware/Makefile >>>> > +++ b/firmware/Makefile >>>> > @@ -176,7 +176,8 @@ quiet_cmd_fwbin = MK_FW $@ >>>> > wordsize_deps := $(wildcard include/config/64bit.h include/config/32bit.h \ >>>> > include/config/ppc32.h include/config/ppc64.h \ >>>> > include/config/superh32.h include/config/superh64.h \ >>>> > - include/config/x86_32.h include/config/x86_64.h) >>>> > + include/config/x86_32.h include/config/x86_64.h \ >>>> > + firmware/Makefile) >>>> > >>>> > $(patsubst %,$(obj)/%.gen.S, $(fw-shipped-y)): %: $(wordsize_deps) >>>> > $(call cmd,fwbin,$(patsubst %.gen.S,%,$@)) >>>> >>>> >>>> Why don't you use $(call filechk,...) or $(call if_changed,...) >>>> instead of wordsize_deps ? >>> >>> Care to send a patch? >> >> >> Sorry for my late reply. >> >> Yes, if you want, I will give it a try. >> >> But, please wait until this weekend... > > Andrew picked up my patch already as its been sitting in limbo without > any feedback for a long time OK. (and, sorry for the delay.) It is not a good idea to make your work stuck even longer. > --- so you can work off of what is on > linux-next or we can request Andrew to drop this patch once and if you > get a replacement as its not yet on Linus' tree. > Please go ahead with your patch. When I find time to take a close look, I may come back to firmware/Makefile. But, I guess it will not happen soon. -- Best Regards Masahiro Yamada