public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] ARM mach-types.h (was: Re: [PATCH 1/5] add DOCKSTAR machine type)
Date: Wed, 08 Sep 2010 14:55:48 +0200	[thread overview]
Message-ID: <20100908125548.0D3291506AA@gemini.denx.de> (raw)
In-Reply-To: <4C875885.9020709@emk-elektronik.de>

Dear Reinhard Meyer,

In message <4C875885.9020709@emk-elektronik.de> you wrote:
>
> Not sure about that. I was thinking of a simple script that reads Linux'
> mach-types.h (or arch/arm/tools/mach-types directly) and generates the
> u-boot's mach-types.h file on the fly. That way only the #define lines could
> be generated, producing a less than 100k files instead of > 1MB.

I don't think mach-types.h is worth spending so much effort on it, and
coming up with another set of tools, incompatible to everything else
in the world.

> The same script would read the existing mach-types.h file, scanning for
> something like
> 
> /* MANUAL ENTRIES START */
> #define MACH_TYPE_XYZ 4711
> #define MACH_TYPE_ABC 4712
> /* MANUAL ENTRIES END */

I will not accept this. It willonly result in people adding entries
that have not been registered so they cause conflicts sooner or later.
We have been there before, and it did not work. I will not go back
there.

> Otherwise:
> 
> It essentially means one can send patches for a new ARM based board
> only if the MACH_TYPE has trickled through all the instances.

Right. But this should be no problem, as you can register your MACH_ID
as early as you want.

> This is, somehow, unpleasing:). We should not "punish" ourselves in u-boot
> just because someone has invented a method to prolong time-to-market ;)

Just react in a pro-active way and register your machines as soon as
you start working on the hardware design, and everything is well.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
History is only a confused heap of facts.
                                       -- Philip Earl of Chesterfield

  reply	other threads:[~2010-09-08 12:55 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-08  7:11 [U-Boot] ARM mach-types.h (was: Re: [PATCH 1/5] add DOCKSTAR machine type) Reinhard Meyer
2010-09-08  8:13 ` Wolfgang Denk
2010-09-08  8:40   ` Reinhard Meyer
2010-09-08  9:08     ` Wolfgang Denk
2010-09-08  9:33       ` Reinhard Meyer
2010-09-08 12:55         ` Wolfgang Denk [this message]
2010-09-08 18:58       ` Paulraj, Sandeep
2010-09-08 19:22         ` Reinhard Meyer
2010-09-08 19:53         ` Wolfgang Denk
2010-09-08 20:05           ` Paulraj, Sandeep
2010-09-13 13:49             ` Thomas Weber

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=20100908125548.0D3291506AA@gemini.denx.de \
    --to=wd@denx.de \
    --cc=u-boot@lists.denx.de \
    /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