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] Update and Cut down mach types
Date: Tue, 26 Apr 2011 23:32:00 +0200	[thread overview]
Message-ID: <20110426213200.2F204D52705@gemini.denx.de> (raw)
In-Reply-To: <4DB72D4A.5070102@aribaud.net>

Dear Albert ARIBAUD,

In message <4DB72D4A.5070102@aribaud.net> you wrote:
>
> Well, as you stated yourself recently, why would/should we maintain 
> mach-types that are apparently not going to be used? Do machine types 
> have other uses than for Linux? No code in U-Boot should worry about the 
> mach-id if not for Linux.

Well, in principle you are of course right.

But I am well aware that there is a ton of Linux BSPs out there which
have never been pushed upstream into mainline by their respective
creators for some reason or another.  Also I see a chance that other
uses of the mach-ids might exist - the Linux ARM folks have, fro a
very long time, always explained what a clever idea this is to
describe hardware features.

I hesitate to cut off all these exitisting or even potential users
lightly, when there is a solution that works reasonably well for them
and, at the same time, brings only minimal maintenance burdon for us.

> Also, if we still decide to maintain our own list of mach-types, we will 
> need some rule to decide when to remove mach-types from this special 
> list eventually. Otherwise, it'll become asymptotically identical to the 
> full lits that is also availabe, and then, what would be the point of 
> maintaining our own?

That rule can be simple: we will only allow to add the now existing
(in U-Boot mainline code) mach-ids, so this list should not grow
further after the initial creation.  OK, ther eis a slight chance that
any newly added boards (to U-Boot) will get removed from the Linux
master file later, but I consider this a small risk - especially as I
expect to see more and ore device-tree based ARM ports quickly, so the
whole mach-id thing becomes less and less of a pain.

> So IMO, if we have mach-types in U-Boot for supporting Linux, then we 
> should keep using a (reasonably) up-to-date Linux machine ID list just 
> like we do now -- mach-types that disappear from the list mean Linux 
> support has become useless for that machine in U-Boot. And if we have 
> our own mach-type policy, different from "has linux support", then we 
> need to specify what this policy is and how it is implemented.

I think we should be gentle to users of existing code and avoid
breaking it. From now on, we could establish a policy that a mach-id
can only be referenced when and as long mainline Linux support for
this board exists.

I'm open for suggestions.

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
There is a time in the tides of men, Which, taken at its flood, leads
on to success. On the other hand, don't count on it.   - T. K. Lawson

  reply	other threads:[~2011-04-26 21:32 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-19 12:42 [U-Boot] Update and Cut down mach types Paulraj, Sandeep
2011-04-19 13:39 ` Matthias Weißer
2011-04-19 13:45   ` Paulraj, Sandeep
2011-04-20  8:44     ` Albert ARIBAUD
2011-04-19 14:21   ` Wolfgang Denk
2011-04-19 18:42     ` Matthias Weisser
2011-04-19 18:44     ` Michael Schwingen
2011-04-20  8:15       ` Detlev Zundel
2011-04-20  8:58 ` Igor Grinberg
2011-04-20 17:15   ` Michael Schwingen
2011-04-20 17:49     ` Albert ARIBAUD
2011-04-20 19:26       ` Michael Schwingen
2011-04-21 11:39         ` Albert ARIBAUD
2011-04-26 18:14           ` Michael Schwingen
2011-04-26 19:40             ` Wolfgang Denk
2011-04-26 20:38               ` Albert ARIBAUD
2011-04-26 21:32                 ` Wolfgang Denk [this message]
2011-04-26 21:38                   ` Reinhard Meyer
2011-04-27 10:19                     ` Michael Schwingen
2011-04-28  6:20                       ` Igor Grinberg
2011-04-29  8:58                         ` Detlev Zundel
2011-05-01 10:10                           ` [U-Boot] [PATCH 1/3] arm: omap: innovator: fix compilation error Igor Grinberg
2011-05-17 12:40                             ` Igor Grinberg
2011-05-21 21:40                               ` Paulraj, Sandeep
2011-05-01 10:10                           ` [U-Boot] [PATCH 2/3] arm: omap: innovator: Prepare for mach-types.h changes Igor Grinberg
2011-05-01 20:28                             ` Alessandro Rubini
2011-05-02  7:18                               ` Igor Grinberg
2011-05-03 10:08                                 ` [U-Boot] [PATCH v2 " Igor Grinberg
2011-05-03 12:29                                   ` Wolfgang Denk
2011-05-03 13:00                                     ` Igor Grinberg
2011-05-04  7:13                                       ` [U-Boot] [PATCH v3 " Igor Grinberg
2011-05-01 10:10                           ` [U-Boot] [PATCH 3/3] arm: at91: ether: " Igor Grinberg
2011-05-01 19:38                             ` Reinhard Meyer
2011-05-02  7:29                               ` Igor Grinberg
2011-05-02 10:09                                 ` Detlev Zundel
2011-05-02 12:49                                   ` [U-Boot] [PATCH v2 " Igor Grinberg
2011-05-16 13:31                                     ` Igor Grinberg
2011-04-27 11:44                     ` [U-Boot] Update and Cut down mach types Detlev Zundel

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=20110426213200.2F204D52705@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