public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] Update and Cut down mach types
Date: Tue, 26 Apr 2011 22:38:34 +0200	[thread overview]
Message-ID: <4DB72D4A.5070102@aribaud.net> (raw)
In-Reply-To: <20110426194003.AD040D52706@gemini.denx.de>

Le 26/04/2011 21:40, Wolfgang Denk a ?crit :
> Dear Michael Schwingen,
>
> In message<4DB70B98.6040208@discworld.dascon.de>  you wrote:
>>
>> Nevertheless, we need a method so that we no not need to patch the
>> mach-types.h after every cacle where the Linux version is pulled in.
>
> Detlev Zundel suggested we might maintain a U-Boot local file
> "obsoleted-mach-types.h" which would contain just the MACH_ID
> definitions that 1) are being actively used in U-Boot and 2) have been
> removed from the official "mach-types.h" file.  This way we just need
> to add a "#include<obsoleted-mach-types.h>" to "mach-types.h" to
> maintain compatibility with the existing code, and we make it very
> obvious that these are somewhat special MACH_IDs.
>
>
> Comments?

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.

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?

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.

> Best regards,
>
> Wolfgang Denk

Amicalement,
-- 
Albert.

  reply	other threads:[~2011-04-26 20: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 [this message]
2011-04-26 21:32                 ` Wolfgang Denk
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=4DB72D4A.5070102@aribaud.net \
    --to=albert.u.boot@aribaud.net \
    --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