[RFC PATCH 0/3] ARM: use C pre-processor with dtc

Jean-Christophe PLAGNIOL-VILLARD plagnioj at jcrosoft.com
Fri Sep 28 03:27:25 EST 2012


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 at 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.


More information about the devicetree-discuss mailing list