devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
To: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: "devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org"
	<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
	Arnaud Lacombe <lacombar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Subject: Re: [RFC PATCH] dtc: Add support for named constants
Date: Wed, 31 Aug 2011 13:33:53 +1000	[thread overview]
Message-ID: <20110831033353.GB26462@yookeroo.fritz.box> (raw)
In-Reply-To: <74CDBE0F657A3D45AFBB94109FB122FF04B3279D55-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>

On Tue, Aug 30, 2011 at 09:37:40AM -0700, Stephen Warren wrote:
> David Gibson wrote at Monday, August 29, 2011 10:23 PM:
> > On Tue, Aug 23, 2011 at 04:43:20PM -0600, Stephen Warren wrote:
> > > You may define constants as follows:
> > >
> > > /define/ $TWO 2;
> > > /define/ $FOUR 4;
> > > /define/ $OTHER $FOUR;
> > >
> > > And properties may use these values as follows:
> > >
> > > foo = <1 $TWO 3 $FOUR 5>;
> ...
> > > Note 2: I'd prefer the syntax of /define/ to be:
> > >
> > > /define/ TWO 2;
> > 
> > Oh goodness, yes.  The dollar signs are revolting.
> > 
> > > but I assume that'd cause the lexing for DT_DEFINEREF to conflict with
> > > that for DT_LABEL?
> > 
> > Nope, the final colon should distinguish DT_LABEL.  In dts-v1 C-like
> > identifiers should be lexically distinct in most contexts.  This is
> > not by accident.  They could be confused with node or property names,
> > but that shouldn't cause trouble.
> 
> OK, I'll take a stab at that and see.
> 
> As background, I found the following thread from 2008:
> 
> http://lists.ozlabs.org/pipermail/devicetree-discuss/2008-September/000149.html
> 
> ... where Jon Loeliger was working on much more extensive new syntax,
> and it sounded like you had some patches at least for math expressions
> in cells etc. The conversation seemed to die off after a few weeks. Did
> anything come of that; is anyone else working on e.g. at least the math
> expressions and defines part, even if not the scripting stuff?
> 
> I see that a little later, Jon pushed:
> 
> http://git.jdl.com/gitweb/?p=dtc.git;a=shortlog;h=refs/heads/testing
> 
> I'm mainly asking because if /any/ of that is something that's still
> useful to pursue, I might want to align this patch with some of that so
> as not to preclude any of that being ported to ToT.

Sigh, yeah.  So I think both Jon and I would really like to get rather
more comprehensive support for this sort of thing into dtc.  However,
we have somewhat different visions of how exactly it would work.  A
combination of that and lac of time to pursue it in depth lead to it
getting shelved at the time.

In the meantime the include and overlay mechanics have been introduced
which allow a limited amount of what people are askinf for in this
area.  Resurrecting that work of Jon's would, amongst other things,
require some reconsideration of how it's affected by those features.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

      parent reply	other threads:[~2011-08-31  3:33 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-23 22:43 [RFC PATCH] dtc: Add support for named constants Stephen Warren
     [not found] ` <1314139400-31984-1-git-send-email-swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2011-08-30  4:23   ` David Gibson
     [not found]     ` <20110830042316.GH4254-787xzQ0H9iQXU02nzanrWNbf9cGiqdzd@public.gmane.org>
2011-08-30 16:37       ` Stephen Warren
     [not found]         ` <74CDBE0F657A3D45AFBB94109FB122FF04B3279D55-C7FfzLzN0UxDw2glCA4ptUEOCMrvLtNR@public.gmane.org>
2011-08-30 21:04           ` Paul Walmsley
     [not found]             ` <alpine.DEB.2.00.1108301459340.26173-rwI8Ez+7Ko+d5PgPZx9QOdBPR1lH4CV8@public.gmane.org>
2011-08-31  3:37               ` David Gibson
2011-08-31  3:33           ` David Gibson [this message]

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=20110831033353.GB26462@yookeroo.fritz.box \
    --to=david-xt8fgy+axnrb3ne2bgzf6laj5h9x9tb+@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=lacombar-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.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 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).