Re: [PATCH] kbuild: support make dir/file.i for *.S

Path	csiph.com!au2pb.net!feeder.erje.net!2.us.feeder.erje.net!newsfeed.fsmpi.rwth-aachen.de!newsfeed.straub-nv.de!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod
From	Michal Marek <mmarek@suse.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] kbuild: support make dir/file.i for *.S
Date	Mon, 14 Dec 2015 18:20:02 +0100
Message-ID	<qFEX0-DM-25@gated-at.bofh.it> (permalink)
References	<qEcq6-cf-5@gated-at.bofh.it> <qEXS2-6dM-21@gated-at.bofh.it> <qFEX0-DM-17@gated-at.bofh.it>
X-Original-To	Wang YanQing <udknight@gmail.com>, Brian Gerst <brgerst@gmail.com>, linux-kbuild@vger.kernel.org, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, yamada.masahiro@socionext.com
User-Agent	Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0
MIME-Version	1.0
Content-Type	text/plain; charset=windows-1252
Content-Transfer-Encoding	7bit
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	57
Organization	linux.* mail to news gateway
X-Original-Date	Mon, 14 Dec 2015 18:16:50 +0100
X-Original-Message-ID	<566EF982.7040404@suse.com>
X-Original-References	<20151210163329.GA18947@udknight> <CAMzpN2jG+mKQdAxKnvTebQtF1J79pC+E4QxELLO_C2K6ZR8PzA@mail.gmail.com> <20151214161401.GA8036@udknight>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1291363

Show key headers only | View raw

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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

Re: [PATCH] kbuild: support make dir/file.i for *.S Brian Gerst <brgerst@gmail.com> - 2015-12-12 20:20 +0100
  Re: [PATCH] kbuild: support make dir/file.i for *.S Masahiro Yamada <yamada.masahiro@socionext.com> - 2015-12-13 09:20 +0100
  Re: [PATCH] kbuild: support make dir/file.i for *.S Wang YanQing <udknight@gmail.com> - 2015-12-14 18:20 +0100
    Re: [PATCH] kbuild: support make dir/file.i for *.S Michal Marek <mmarek@suse.com> - 2015-12-14 18:20 +0100
      Re: [PATCH] kbuild: support make dir/file.i for *.S Masahiro Yamada <yamada.masahiro@socionext.com> - 2015-12-15 02:50 +0100

csiph-web