public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Dirk Behme <dirk.behme@googlemail.com>
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:41:19 +0200	[thread overview]
Message-ID: <4A8EC01F.7040307@googlemail.com> (raw)
In-Reply-To: <m24os18a3w.fsf@ohwell.denx.de>

Hi Detlev,

Detlev Zundel wrote:
> 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.

Yes, I agree that it makes no sense to *completely* change the rule.

Maybe we should just be a little bit more flexible about this rule and 
have look, where something else makes more sense.

> 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.

I could add the opposite example:

A <vendor == TI> OMAP3 based board (e.g. Beagle) has no adhesion with 
a <vendor == TI> DaVinci board.

> 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/.

Sounds like you propose to put omap3 *board* common stuff into *cpu* 
directory?

> 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/.

But it wouldn't hurt?

> Although I think that these arguments carry some value, I know that
> one can come up with - basically arbitrarily many other arguments.

Yes ;)

> 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 ;)

Could we agree to be more flexible with this rule?

Or, the other way around:

Independent of the rule, do you see any advantage of switching existing

board/omap3/
board/davinci/

into something like

board/DigiKey/beagle (or board/TI/beagle?)
board/gumstix/overo
board/mistral/evm (or board/TI/evm? )
board/xx/pandora
board/zz/zoom1
board/yy/zoom2

etc.?

Except to follow the rule?

Thanks

Dirk

  reply	other threads:[~2009-08-21 15:41 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
2009-08-21 15:41               ` Dirk Behme [this message]
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=4A8EC01F.7040307@googlemail.com \
    --to=dirk.behme@googlemail.com \
    --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