All of lore.kernel.org
 help / color / mirror / Atom feed
From: Reinhard Meyer <u-boot@emk-elektronik.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Update and Cut down mach types
Date: Tue, 26 Apr 2011 23:38:36 +0200	[thread overview]
Message-ID: <4DB73B5C.5090601@emk-elektronik.de> (raw)
In-Reply-To: <20110426213200.2F204D52705@gemini.denx.de>

On 26.04.2011 23:32, Wolfgang Denk wrote:
> 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.

Hi Wolfgang, Albert,

why don't we just create the #define MACH_xxx lines directly from the
"http://www.arm.linux.org.uk/developer/machines/download.php". We don't
need all the *_is_* macros in u-boot anyway. Then we would have just a few 1000
lines of #define MACH_*

Reinhard

  reply	other threads:[~2011-04-26 21:38 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
2011-04-26 21:38                   ` Reinhard Meyer [this message]
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=4DB73B5C.5090601@emk-elektronik.de \
    --to=u-boot@emk-elektronik.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 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.