From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 0/3] ARM: use C pre-processor with dtc
Date: Tue, 25 Sep 2012 16:46:16 -0700 [thread overview]
Message-ID: <20120925234616.GE4840@atomide.com> (raw)
In-Reply-To: <50620B5B.5020904@wwwdotorg.org>
* Stephen Warren <swarren@wwwdotorg.org> [120925 12:53]:
> On 09/25/2012 01:35 PM, Scott Wood wrote:
> >
> > Do you have an example of where you'd actually benefit from this? I'd
> > think most things could either be done reasonably well with what's built
> > into DTC (see what we've done in arch/powerpc/boot/dts/fsl), or would
> > need math expression support in DTC (or has that been added?).
>
> Yes, support for basic integer math in cell values has indeed been
> recently added to upstream dtc. I don't believe this has been ported
> into the in-kernel dtc yet though.
>
> The primary motivation here is probably naming constants and associated
> readability. For example, instead of:
>
> nvidia,phy-reset-gpio = <&gpio 169 0>; /* gpio PV1 */
>
> You could write:
>
> nvidia,phy-reset-gpio = <&gpio TEGRA_GPIO_PV1 0>;
>
> No more opaque numbers!
This would be nice indeed.
Regards,
Tony
WARNING: multiple messages have this Message-ID (diff)
From: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
To: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Cc: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [RFC PATCH 0/3] ARM: use C pre-processor with dtc
Date: Tue, 25 Sep 2012 16:46:16 -0700 [thread overview]
Message-ID: <20120925234616.GE4840@atomide.com> (raw)
In-Reply-To: <50620B5B.5020904-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
* Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> [120925 12:53]:
> On 09/25/2012 01:35 PM, Scott Wood wrote:
> >
> > Do you have an example of where you'd actually benefit from this? I'd
> > think most things could either be done reasonably well with what's built
> > into DTC (see what we've done in arch/powerpc/boot/dts/fsl), or would
> > need math expression support in DTC (or has that been added?).
>
> Yes, support for basic integer math in cell values has indeed been
> recently added to upstream dtc. I don't believe this has been ported
> into the in-kernel dtc yet though.
>
> The primary motivation here is probably naming constants and associated
> readability. For example, instead of:
>
> nvidia,phy-reset-gpio = <&gpio 169 0>; /* gpio PV1 */
>
> You could write:
>
> nvidia,phy-reset-gpio = <&gpio TEGRA_GPIO_PV1 0>;
>
> No more opaque numbers!
This would be nice indeed.
Regards,
Tony
next prev parent reply other threads:[~2012-09-25 23:46 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-25 19:06 [RFC PATCH 0/3] ARM: use C pre-processor with dtc Stephen Warren
2012-09-25 19:06 ` Stephen Warren
2012-09-25 19:06 ` [RFC PATCH 1/3] kbuild: introduce cmd_dtc_cpp Stephen Warren
2012-09-25 19:06 ` Stephen Warren
2012-09-27 5:44 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 5:44 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 15:39 ` Stephen Warren
2012-09-27 15:39 ` Stephen Warren
2012-09-27 17:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 17:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-25 19:06 ` [RFC PATCH 2/3] ARM: use cmd_dtc_cpp for compilation of *.dts-cpp to *.dtb Stephen Warren
2012-09-25 19:06 ` Stephen Warren
2012-09-27 5:47 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 5:47 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 15:40 ` Stephen Warren
2012-09-27 15:40 ` Stephen Warren
2012-09-25 19:06 ` [RFC PATCH 3/3] ARM: tegra: compile all DT files with cpp Stephen Warren
2012-09-25 19:06 ` Stephen Warren
2012-09-25 19:35 ` [RFC PATCH 0/3] ARM: use C pre-processor with dtc Scott Wood
2012-09-25 19:35 ` Scott Wood
2012-09-25 19:51 ` Mark Brown
2012-09-25 19:51 ` Mark Brown
2012-09-25 19:59 ` Scott Wood
2012-09-25 19:59 ` Scott Wood
2012-09-25 20:05 ` Mark Brown
2012-09-25 20:05 ` Mark Brown
2012-09-25 20:05 ` Stephen Warren
2012-09-25 20:05 ` Stephen Warren
2012-09-25 19:51 ` Stephen Warren
2012-09-25 19:51 ` Stephen Warren
2012-09-25 23:46 ` Tony Lindgren [this message]
2012-09-25 23:46 ` Tony Lindgren
2012-09-27 5:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 5:40 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 15:38 ` Stephen Warren
2012-09-27 15:38 ` Stephen Warren
2012-09-27 17:27 ` Jean-Christophe PLAGNIOL-VILLARD
2012-09-27 17:27 ` Jean-Christophe PLAGNIOL-VILLARD
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=20120925234616.GE4840@atomide.com \
--to=tony@atomide.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.