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
next prev parent 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox