From: Masahiro Yamada <yamada.masahiro@socionext.com>
To: "Luis R. Rodriguez" <mcgrof@kernel.org>
Cc: Michal Marek <mmarek@suse.com>,
Linux Kbuild mailing list <linux-kbuild@vger.kernel.org>,
ming.lei@canonical.com,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Tom Gundersen <teg@jklm.no>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
David Woodhouse <dwmw2@infradead.org>
Subject: Re: [PATCH v2] firmware/Makefile: force recompilation if makefile changes
Date: Fri, 17 Mar 2017 02:43:16 +0900 [thread overview]
Message-ID: <CAK7LNAQmVSXVQt4-GHiOTUh4Dc0s0xB7Ob4kyA7G_qiM5CEDZA@mail.gmail.com> (raw)
In-Reply-To: <20170315005348.GK28800@wotan.suse.de>
Hi Luis,
2017-03-15 9:53 GMT+09:00 Luis R. Rodriguez <mcgrof@kernel.org>:
> 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 <mcgrof@kernel.org>:
>> > 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 <mcgrof@kernel.org>
>> > ---
>> >
>> > 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...
--
Best Regards
Masahiro Yamada
next prev parent reply other threads:[~2017-03-16 17:43 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-18 17:31 [PATCH] firmware/Makefile: force recompilation if makefile changes Luis R. Rodriguez
2017-01-19 10:15 ` Greg KH
2017-01-23 14:59 ` Luis R. Rodriguez
2017-01-23 15:02 ` Greg KH
2017-01-23 15:04 ` Luis R. Rodriguez
2017-01-23 15:07 ` [PATCH v2] " Luis R. Rodriguez
2017-02-06 22:03 ` Luis R. Rodriguez
2017-02-14 19:34 ` Luis R. Rodriguez
2017-02-21 19:38 ` Luis R. Rodriguez
2017-03-11 5:37 ` Masahiro Yamada
2017-03-15 0:53 ` Luis R. Rodriguez
2017-03-16 17:43 ` Masahiro Yamada [this message]
2017-03-16 17:55 ` Luis R. Rodriguez
2017-03-18 13:32 ` Masahiro Yamada
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAK7LNAQmVSXVQt4-GHiOTUh4Dc0s0xB7Ob4kyA7G_qiM5CEDZA@mail.gmail.com \
--to=yamada.masahiro@socionext.com \
--cc=dwmw2@infradead.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mcgrof@kernel.org \
--cc=ming.lei@canonical.com \
--cc=mmarek@suse.com \
--cc=teg@jklm.no \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).