* [PATCH] kbuild: support make dir/file.i for *.S
@ 2015-12-10 16:33 Wang YanQing
2015-12-11 16:29 ` Michal Marek
2015-12-12 19:13 ` Brian Gerst
0 siblings, 2 replies; 7+ messages in thread
From: Wang YanQing @ 2015-12-10 16:33 UTC (permalink / raw)
To: mmarek; +Cc: linux-kbuild, linux-kernel
This feature could reduce a little time to expand all
the macros in *.S for reading.
Signed-off-by: Wang YanQing <udknight@gmail.com>
---
Makefile | 2 ++
scripts/Makefile.build | 6 ++++++
2 files changed, 8 insertions(+)
diff --git a/Makefile b/Makefile
index 7558c9c..d7e2060 100644
--- a/Makefile
+++ b/Makefile
@@ -1552,6 +1552,8 @@ endif
$(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
%.s: %.S prepare scripts FORCE
$(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
+%.i: %.S prepare scripts FORCE
+ $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
%.o: %.S prepare scripts FORCE
$(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
%.symtypes: %.c prepare scripts FORCE
diff --git a/scripts/Makefile.build b/scripts/Makefile.build
index 2c47f9c..2a51d49 100644
--- a/scripts/Makefile.build
+++ b/scripts/Makefile.build
@@ -287,6 +287,12 @@ cmd_as_s_S = $(CPP) $(a_flags) -o $@ $<
$(obj)/%.s: $(src)/%.S FORCE
$(call if_changed_dep,as_s_S)
+quiet_cmd_cc_i_S = CPP $(quiet_modtag) $@
+cmd_cc_i_S = $(CPP) $(a_flags) -o $@ $<
+
+$(obj)/%.i: $(src)/%.S FORCE
+ $(call if_changed_dep,cc_i_S)
+
quiet_cmd_as_o_S = AS $(quiet_modtag) $@
cmd_as_o_S = $(CC) $(a_flags) -c -o $@ $<
--
1.8.5.6.2.g3d8a54e.dirty
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] kbuild: support make dir/file.i for *.S
2015-12-10 16:33 [PATCH] kbuild: support make dir/file.i for *.S Wang YanQing
@ 2015-12-11 16:29 ` Michal Marek
2015-12-12 19:13 ` Brian Gerst
1 sibling, 0 replies; 7+ messages in thread
From: Michal Marek @ 2015-12-11 16:29 UTC (permalink / raw)
To: Wang YanQing; +Cc: linux-kbuild, linux-kernel
Dne 10.12.2015 v 17:33 Wang YanQing napsal(a):
> This feature could reduce a little time to expand all
> the macros in *.S for reading.
>
> Signed-off-by: Wang YanQing <udknight@gmail.com>
Makes sense. Applied to kbuild.git#kbuild.
Michal
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kbuild: support make dir/file.i for *.S
2015-12-10 16:33 [PATCH] kbuild: support make dir/file.i for *.S Wang YanQing
2015-12-11 16:29 ` Michal Marek
@ 2015-12-12 19:13 ` Brian Gerst
2015-12-13 8:17 ` Masahiro Yamada
2015-12-14 16:14 ` Wang YanQing
1 sibling, 2 replies; 7+ messages in thread
From: Brian Gerst @ 2015-12-12 19:13 UTC (permalink / raw)
To: Wang YanQing, mmarek, linux-kbuild, Linux Kernel Mailing List
On Thu, Dec 10, 2015 at 11:33 AM, Wang YanQing <udknight@gmail.com> wrote:
> This feature could reduce a little time to expand all
> the macros in *.S for reading.
>
> Signed-off-by: Wang YanQing <udknight@gmail.com>
> ---
> Makefile | 2 ++
> scripts/Makefile.build | 6 ++++++
> 2 files changed, 8 insertions(+)
>
> diff --git a/Makefile b/Makefile
> index 7558c9c..d7e2060 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1552,6 +1552,8 @@ endif
> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> %.s: %.S prepare scripts FORCE
> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> +%.i: %.S prepare scripts FORCE
> + $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> %.o: %.S prepare scripts FORCE
> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> %.symtypes: %.c prepare scripts FORCE
> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
> index 2c47f9c..2a51d49 100644
> --- a/scripts/Makefile.build
> +++ b/scripts/Makefile.build
> @@ -287,6 +287,12 @@ cmd_as_s_S = $(CPP) $(a_flags) -o $@ $<
> $(obj)/%.s: $(src)/%.S FORCE
> $(call if_changed_dep,as_s_S)
>
> +quiet_cmd_cc_i_S = CPP $(quiet_modtag) $@
> +cmd_cc_i_S = $(CPP) $(a_flags) -o $@ $<
> +
> +$(obj)/%.i: $(src)/%.S FORCE
> + $(call if_changed_dep,cc_i_S)
> +
> quiet_cmd_as_o_S = AS $(quiet_modtag) $@
> cmd_as_o_S = $(CC) $(a_flags) -c -o $@ $<
*.s (lower case) is the suffix used for preprocessed assembly files,
and there is already a rule for that. *.i is for preprocessed C
files.
--
Brian Gerst
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kbuild: support make dir/file.i for *.S
2015-12-12 19:13 ` Brian Gerst
@ 2015-12-13 8:17 ` Masahiro Yamada
2015-12-14 16:14 ` Wang YanQing
1 sibling, 0 replies; 7+ messages in thread
From: Masahiro Yamada @ 2015-12-13 8:17 UTC (permalink / raw)
To: Brian Gerst
Cc: Wang YanQing, Michal Marek, Linux Kbuild mailing list,
Linux Kernel Mailing List
2015-12-13 4:13 GMT+09:00 Brian Gerst <brgerst@gmail.com>:
> On Thu, Dec 10, 2015 at 11:33 AM, Wang YanQing <udknight@gmail.com> wrote:
>> This feature could reduce a little time to expand all
>> the macros in *.S for reading.
>>
>> Signed-off-by: Wang YanQing <udknight@gmail.com>
>> ---
>> Makefile | 2 ++
>> scripts/Makefile.build | 6 ++++++
>> 2 files changed, 8 insertions(+)
>>
>> diff --git a/Makefile b/Makefile
>> index 7558c9c..d7e2060 100644
>> --- a/Makefile
>> +++ b/Makefile
>> @@ -1552,6 +1552,8 @@ endif
>> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>> %.s: %.S prepare scripts FORCE
>> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>> +%.i: %.S prepare scripts FORCE
>> + $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>> %.o: %.S prepare scripts FORCE
>> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>> %.symtypes: %.c prepare scripts FORCE
>> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
>> index 2c47f9c..2a51d49 100644
>> --- a/scripts/Makefile.build
>> +++ b/scripts/Makefile.build
>> @@ -287,6 +287,12 @@ cmd_as_s_S = $(CPP) $(a_flags) -o $@ $<
>> $(obj)/%.s: $(src)/%.S FORCE
>> $(call if_changed_dep,as_s_S)
>>
>> +quiet_cmd_cc_i_S = CPP $(quiet_modtag) $@
>> +cmd_cc_i_S = $(CPP) $(a_flags) -o $@ $<
>> +
>> +$(obj)/%.i: $(src)/%.S FORCE
>> + $(call if_changed_dep,cc_i_S)
>> +
>> quiet_cmd_as_o_S = AS $(quiet_modtag) $@
>> cmd_as_o_S = $(CC) $(a_flags) -c -o $@ $<
>
> *.s (lower case) is the suffix used for preprocessed assembly files,
> and there is already a rule for that. *.i is for preprocessed C
> files.
>
Right. We already support *.s and it is completely the same as
what this patch adds (except that *.i is now an alias of *.s,
which is also wrong as Brian pointed out.)
quiet_cmd_as_s_S = CPP $(quiet_modtag) $@
cmd_as_s_S = $(CPP) $(a_flags) -o $@ $<
$(obj)/%.s: $(src)/%.S FORCE
$(call if_changed_dep,as_s_S)
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kbuild: support make dir/file.i for *.S
2015-12-12 19:13 ` Brian Gerst
2015-12-13 8:17 ` Masahiro Yamada
@ 2015-12-14 16:14 ` Wang YanQing
2015-12-14 17:16 ` Michal Marek
1 sibling, 1 reply; 7+ messages in thread
From: Wang YanQing @ 2015-12-14 16:14 UTC (permalink / raw)
To: Brian Gerst, mmarek
Cc: linux-kbuild, Linux Kernel Mailing List, yamada.masahiro
On Sat, Dec 12, 2015 at 02:13:03PM -0500, Brian Gerst wrote:
> On Thu, Dec 10, 2015 at 11:33 AM, Wang YanQing <udknight@gmail.com> wrote:
> > This feature could reduce a little time to expand all
> > the macros in *.S for reading.
> >
> > Signed-off-by: Wang YanQing <udknight@gmail.com>
> > ---
> > Makefile | 2 ++
> > scripts/Makefile.build | 6 ++++++
> > 2 files changed, 8 insertions(+)
> >
> > diff --git a/Makefile b/Makefile
> > index 7558c9c..d7e2060 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -1552,6 +1552,8 @@ endif
> > $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> > %.s: %.S prepare scripts FORCE
> > $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> > +%.i: %.S prepare scripts FORCE
> > + $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> > %.o: %.S prepare scripts FORCE
> > $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
> > %.symtypes: %.c prepare scripts FORCE
> > diff --git a/scripts/Makefile.build b/scripts/Makefile.build
> > index 2c47f9c..2a51d49 100644
> > --- a/scripts/Makefile.build
> > +++ b/scripts/Makefile.build
> > @@ -287,6 +287,12 @@ cmd_as_s_S = $(CPP) $(a_flags) -o $@ $<
> > $(obj)/%.s: $(src)/%.S FORCE
> > $(call if_changed_dep,as_s_S)
> >
> > +quiet_cmd_cc_i_S = CPP $(quiet_modtag) $@
> > +cmd_cc_i_S = $(CPP) $(a_flags) -o $@ $<
> > +
> > +$(obj)/%.i: $(src)/%.S FORCE
> > + $(call if_changed_dep,cc_i_S)
> > +
> > quiet_cmd_as_o_S = AS $(quiet_modtag) $@
> > cmd_as_o_S = $(CC) $(a_flags) -c -o $@ $<
>
> *.s (lower case) is the suffix used for preprocessed assembly files,
> and there is already a rule for that. *.i is for preprocessed C
> files.
Agreed, you are right!
Thanks.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kbuild: support make dir/file.i for *.S
2015-12-14 16:14 ` Wang YanQing
@ 2015-12-14 17:16 ` Michal Marek
2015-12-15 1:41 ` Masahiro Yamada
0 siblings, 1 reply; 7+ messages in thread
From: Michal Marek @ 2015-12-14 17:16 UTC (permalink / raw)
To: Wang YanQing, Brian Gerst, linux-kbuild,
Linux Kernel Mailing List, yamada.masahiro
Dne 14.12.2015 v 17:14 Wang YanQing napsal(a):
> On Sat, Dec 12, 2015 at 02:13:03PM -0500, Brian Gerst wrote:
>> On Thu, Dec 10, 2015 at 11:33 AM, Wang YanQing <udknight@gmail.com> wrote:
>>> This feature could reduce a little time to expand all
>>> the macros in *.S for reading.
>>>
>>> Signed-off-by: Wang YanQing <udknight@gmail.com>
>>> ---
>>> Makefile | 2 ++
>>> scripts/Makefile.build | 6 ++++++
>>> 2 files changed, 8 insertions(+)
>>>
>>> diff --git a/Makefile b/Makefile
>>> index 7558c9c..d7e2060 100644
>>> --- a/Makefile
>>> +++ b/Makefile
>>> @@ -1552,6 +1552,8 @@ endif
>>> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>>> %.s: %.S prepare scripts FORCE
>>> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>>> +%.i: %.S prepare scripts FORCE
>>> + $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>>> %.o: %.S prepare scripts FORCE
>>> $(Q)$(MAKE) $(build)=$(build-dir) $(target-dir)$(notdir $@)
>>> %.symtypes: %.c prepare scripts FORCE
>>> diff --git a/scripts/Makefile.build b/scripts/Makefile.build
>>> index 2c47f9c..2a51d49 100644
>>> --- a/scripts/Makefile.build
>>> +++ b/scripts/Makefile.build
>>> @@ -287,6 +287,12 @@ cmd_as_s_S = $(CPP) $(a_flags) -o $@ $<
>>> $(obj)/%.s: $(src)/%.S FORCE
>>> $(call if_changed_dep,as_s_S)
>>>
>>> +quiet_cmd_cc_i_S = CPP $(quiet_modtag) $@
>>> +cmd_cc_i_S = $(CPP) $(a_flags) -o $@ $<
>>> +
>>> +$(obj)/%.i: $(src)/%.S FORCE
>>> + $(call if_changed_dep,cc_i_S)
>>> +
>>> quiet_cmd_as_o_S = AS $(quiet_modtag) $@
>>> cmd_as_o_S = $(CC) $(a_flags) -c -o $@ $<
>>
>> *.s (lower case) is the suffix used for preprocessed assembly files,
>> and there is already a rule for that. *.i is for preprocessed C
>> files.
>
> Agreed, you are right!
Ah, I missed. I'll revert the patch in the kbuild tree.
Michal
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] kbuild: support make dir/file.i for *.S
2015-12-14 17:16 ` Michal Marek
@ 2015-12-15 1:41 ` Masahiro Yamada
0 siblings, 0 replies; 7+ messages in thread
From: Masahiro Yamada @ 2015-12-15 1:41 UTC (permalink / raw)
To: Michal Marek
Cc: Wang YanQing, Brian Gerst, Linux Kbuild mailing list,
Linux Kernel Mailing List
Hi Michal,
2015-12-15 2:16 GMT+09:00 Michal Marek <mmarek@suse.com>:
>
> Ah, I missed. I'll revert the patch in the kbuild tree.
Please drop the noise commit rather than reverting it.
It was just applied a few days ago, and there is only one commit on top of it.
--
Best Regards
Masahiro Yamada
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-12-15 1:41 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-10 16:33 [PATCH] kbuild: support make dir/file.i for *.S Wang YanQing
2015-12-11 16:29 ` Michal Marek
2015-12-12 19:13 ` Brian Gerst
2015-12-13 8:17 ` Masahiro Yamada
2015-12-14 16:14 ` Wang YanQing
2015-12-14 17:16 ` Michal Marek
2015-12-15 1:41 ` Masahiro Yamada
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).