From mboxrd@z Thu Jan  1 00:00:00 1970
From: plagnioj@jcrosoft.com (Jean-Christophe PLAGNIOL-VILLARD)
Date: Thu, 27 Sep 2012 19:27:25 +0200
Subject: [RFC PATCH 0/3] ARM: use C pre-processor with dtc
In-Reply-To: <506472E4.5060107@wwwdotorg.org>
References: <1348599998-2729-1-git-send-email-swarren@wwwdotorg.org>
 <20120927054038.GL17667@game.jcrosoft.org>
 <506472E4.5060107@wwwdotorg.org>
Message-ID: <20120927172725.GT17667@game.jcrosoft.org>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On 09:38 Thu 27 Sep     , Stephen Warren wrote:
> On 09/26/2012 11:40 PM, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 13:06 Tue 25 Sep     , Stephen Warren wrote:
> >> From: Stephen Warren <swarren@nvidia.com>
> >>
> >> This series adds some build rules to run cpp on *.dts-cpp prior to
> >> invoking dtc, and converts Tegra to the new rule as an example. What do
> >> people think?
> >>
> >> I assume that you've applied the dtc patches I sent yesterday. They
> >> aren't in this series. See:
> >>
> >> https://lists.ozlabs.org/pipermail/devicetree-discuss/2012-September/020182.html
> >> https://lists.ozlabs.org/pipermail/devicetree-discuss/2012-September/020183.html
> >> https://lists.ozlabs.org/pipermail/devicetree-discuss/2012-September/020181.html
> >>
> >> Note: those patches are against upstream dtc. If you wish to test this
> >> series, apply the dtc patches to upstream dtc, build it, and copy the
> >> resultant dtc binary over the top of scripts/dtc/dtc.
> >>
> >> Stephen Warren (3):
> >>   kbuild: introduce cmd_dtc_cpp
> >>   ARM: use cmd_dtc_cpp for compilation of *.dts-cpp to *.dtb
> >>   ARM: tegra: compile all DT files with cpp 
> >
> > I do not like the extention
> 
> It is a bit unwieldy. Perhaps *.dtsp, *.dtsip would be better.
> 
> > can we run gcc everytime?
> 
> Not without editing all the device tree files; any property (or node)
> name the starts with a # must be escaped; changed from "#foo" to "\#foo"
> to avoid cpp attempting to interpret it as a pre-processor directive.
> 
> > or just do as done for lds
> > 
> > *.dts.S
> 
> .S means assembly. (that's why I went with .dts-cpp rather than .dts.cpp).
on lds we do this way lds.S => lds

Best Regards,
J.