All of lore.kernel.org
 help / color / mirror / Atom feed
From: Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
To: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
Cc: devicetree-discuss
	<devicetree-discuss-mnsaURCQ41sdnm+yROfE0A@public.gmane.org>
Subject: Re: [PATCH 0/8] Implement a new DTS Source Language
Date: Thu, 25 Sep 2008 10:25:07 -0500	[thread overview]
Message-ID: <48DBAD53.7090201@freescale.com> (raw)
In-Reply-To: <20080925042613.GJ15169-787xzQ0H9iRg7VrjXcPTGA@public.gmane.org>

David Gibson wrote:
>> /define/ make_cpu(\cpu)
> 
> Eck.  The \identified parameters certainly seem to violate the
> look-like-C principle.

It's to disambiguate from existing doesn't-look-like-C constructs 
(property and node definitions), though it's not really needed in this 
context.

>> {
>> 		device_type = "cpu";
>> 		reg = < (\cpu) >;
> 
> So.. in the node name the \cpu parameter is treated as a string, here
> it's treated as a number.  What makes the difference?  Does the %
> operator implicitly convert?

Yes, it implicitly converts.

> Can you define multiple nodes with one
> function.  Can you define properties only with a function?

Yes to both.

>> 			for \i in 0 .. 3 {
> 
> This would seem to be another violation of the look-like-C principle.

I don't think we particularly need to inherit C's warts.

> This for syntax is pretty much unprecedented.

It's similar to Python, albeit with less flexibility (I suggested to Jon 
that we introduce a "set" construct to remedy this).

-Scott

  parent reply	other threads:[~2008-09-25 15:25 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-23 19:04 [PATCH 0/8] Implement a new DTS Source Language Jon Loeliger
     [not found] ` <1222196652-13811-1-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04   ` [PATCH 1/8] Remove support for the legacy DTS source file format Jon Loeliger
     [not found]     ` <1222196652-13811-2-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04       ` [PATCH 2/8] Add conditionalized debug() print macro Jon Loeliger
     [not found]         ` <1222196652-13811-3-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04           ` [PATCH 3/8] Enhance source position implementation Jon Loeliger
     [not found]             ` <1222196652-13811-4-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04               ` [PATCH 4/8] Add header files for new Internal Representation form Jon Loeliger
     [not found]                 ` <1222196652-13811-5-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04                   ` [PATCH 5/8] Add most of the new IR implementation files Jon Loeliger
     [not found]                     ` <1222196652-13811-6-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04                       ` [PATCH 6/8] Add the main IR evaluation implementation Jon Loeliger
     [not found]                         ` <1222196652-13811-7-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04                           ` [PATCH 7/8] Introduce new DTS language Jon Loeliger
     [not found]                             ` <1222196652-13811-8-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-23 19:04                               ` [PATCH 8/8] Add documentation for the " Jon Loeliger
     [not found]                                 ` <1222196652-13811-9-git-send-email-jdl-CYoMK+44s/E@public.gmane.org>
2008-09-25 13:00                                   ` Josh Boyer
2008-09-24 22:25                       ` [PATCH 5/8] Add most of the new IR implementation files Scott Wood
2008-09-24 19:17                   ` [PATCH 4/8] Add header files for new Internal Representation form Scott Wood
2008-09-24 17:07               ` [PATCH 3/8] Enhance source position implementation Scott Wood
     [not found]                 ` <48DA73DA.5000603-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2008-09-24 17:17                   ` Jon Loeliger
2008-09-24 17:21                     ` Scott Wood
2008-09-24 17:23                   ` Warner Losh
2008-09-24 17:23               ` Scott Wood
2008-09-25 12:42           ` [PATCH 2/8] Add conditionalized debug() print macro Josh Boyer
2008-09-24  2:34   ` [PATCH 0/8] Implement a new DTS Source Language Kumar Gala
     [not found]     ` <097BFF8D-317F-4E85-AC2A-4C0A8D6C608B-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2008-09-24 16:51       ` Jon Loeliger
2008-09-24 18:48         ` Jon Loeliger
2008-09-25  4:31           ` David Gibson
2008-09-25  4:26         ` David Gibson
     [not found]           ` <20080925042613.GJ15169-787xzQ0H9iRg7VrjXcPTGA@public.gmane.org>
2008-09-25 15:25             ` Scott Wood [this message]
2008-09-25  3:50       ` David Gibson

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=48DBAD53.7090201@freescale.com \
    --to=scottwood-kzfg59tc24xl57midrcfdg@public.gmane.org \
    --cc=david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org \
    --cc=devicetree-discuss-mnsaURCQ41sdnm+yROfE0A@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 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.