From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Rules for board/* directory, was: [PATCH v3] Adding support for DevKit8000
Date: Fri, 21 Aug 2009 17:22:59 +0200 [thread overview]
Message-ID: <m24os18a3w.fsf@ohwell.denx.de> (raw)
In-Reply-To: <4A8EB875.30309@googlemail.com> (Dirk Behme's message of "Fri, 21 Aug 2009 17:08:37 +0200")
Hi Dirk,
>> That being said, I think it
>> would make sense to put the devkit8000 in either board/devkit8000/ or
>> board/embedinfo/devkit8000 now as that is the "correct" place for it.
>
> Well, I just can't see what the advantage of this "correct" place
> might be. So from the rule point of view, it might make sense, but
> maybe we should adapt the rule, then?
>
> Looking at the TI stuff, it seems to me that a lot of (small?
> different?) companies are using the same SoCs and doing boards with
> these. Most of the U-Boot code is similar, then. But these companies
> are doing only one or two boards. So it makes more sense to group
> these boards based on the SoC (vendor), instead of the board vendor or
> even worse the board name.
Well actually (I think) we agreed on doing the board/vendor scheme. For
example look at board/amcc - there are all the AMCC evalboards basically
each one with a different SoC. Turning this around into board/<soc>
would throw pieces all over the places, which is definitely not what we
want.
Let's look at it from this perspective - on a board level there is
really more adhesion between two different cpu boards from one vendor
than between two same cpu boards from different vendors. Just take the
AMCC boards - they all have the same feel to them, so this is the
natural way to group the boards.
Even more, sharing of stuff should be done outside of board/ - if it
applies to all omap3, common stuff should be in cpu/arm_cortexa8/omap3
and *not at all* below board/.
Finding boards with the same architecture was always very easy by
grepping the include/config/* files. We do not need a representation of
this fact below board/.
Although I think that these arguments carry some value, I know that
one can come up with - basically arbitrarily many other arguments. But
still, we had this discussion already and I do not see that anything
fundamental has changed since the last time around, so please let's not
got into bike-shed painting right now ;)
Cheers
Detlev
--
["From 2.4 to 2.6 to 2.7" discussing release numbering of the Linux kernel]
Let the bike-shed-painting begin.
-- Linus Torvalds
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de
next prev parent reply other threads:[~2009-08-21 15:22 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-20 16:47 [U-Boot] [PATCH v3] Adding support for DevKit8000 Frederik Kriewitz
2009-08-20 17:02 ` Peter Tyser
2009-08-20 17:28 ` Dirk Behme
2009-08-20 18:37 ` Frederik Kriewitz
2009-08-20 20:20 ` Jean-Christophe PLAGNIOL-VILLARD
2009-08-21 13:00 ` Dirk Behme
2009-08-21 14:34 ` Peter Tyser
2009-08-21 15:08 ` [U-Boot] Rules for board/* directory, was: " Dirk Behme
2009-08-21 15:22 ` Detlev Zundel [this message]
2009-08-21 15:41 ` Dirk Behme
2009-08-21 16:04 ` Detlev Zundel
2009-08-21 18:07 ` Wolfgang Denk
2009-08-21 17:59 ` Wolfgang Denk
2009-08-21 23:27 ` Frederik Kriewitz
2009-08-22 8:14 ` Wolfgang Denk
2009-08-21 15:28 ` Peter Tyser
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=m24os18a3w.fsf@ohwell.denx.de \
--to=dzu@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