From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [PATCH 4/5] AT91CAP9 support : board/ files
Date: Sat, 2 Feb 2008 06:59:35 +0100 [thread overview]
Message-ID: <200802020659.35968.sr@denx.de> (raw)
In-Reply-To: <1201899144.7645.18.camel@galileo>
On Friday 01 February 2008, Stelian Pop wrote:
> Le jeudi 31 janvier 2008 ? 23:59 +0100, Ulf Samuelsson a ?crit :
> > > Signed-off-by: Stelian Pop <stelian@popies.net>
> > > ---
> > > board/at91cap9adk/Makefile | 50 +++++++
> > > board/at91cap9adk/at91cap9adk.c | 283
> > > +++++++++++++++++++++++++++++++++++++++ board/at91cap9adk/config.mk
> > > | 1 +
> > > board/at91cap9adk/led.c | 80 +++++++++++
> > > board/at91cap9adk/nand.c | 71 ++++++++++
> > > board/at91cap9adk/u-boot.lds | 57 ++++++++
> >
> > I think it would be better if these files were located in the
> > "board/atmel/at91cap9adk", instead of "board/at91cap9adk"
>
> I did think of that. I have no strong feeling about that. Except that if
> we put at91cap9adk under board/atmel, at91rm9200dk should go there too.
Right. I always prefer to "collect" board ports from one manufacturer/brand
into a specific directory. This way the board directory doesn't
get "polluted" that much. So it would be good if you could provide another
patch to move all existing Atmel ports into the board/atmel directory which
already exists.
Thanks.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
next prev parent reply other threads:[~2008-02-02 5:59 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-31 21:15 [U-Boot-Users] [PATCH ARM 0/5] AT91CAP9 support Stelian Pop
2008-01-31 21:15 ` [U-Boot-Users] [PATCH 1/5] AT91CAP9 support : build integration Stelian Pop
2008-01-31 21:15 ` [U-Boot-Users] [PATCH 2/5] AT91CAP9 support : include/ files Stelian Pop
2008-01-31 21:15 ` [U-Boot-Users] [PATCH 3/5] AT91CAP9 support : cpu/ files Stelian Pop
2008-01-31 23:24 ` Ulf Samuelsson
2008-02-01 20:48 ` Stelian Pop
2008-01-31 21:15 ` [U-Boot-Users] [PATCH 4/5] AT91CAP9 support : board/ files Stelian Pop
2008-01-31 22:59 ` Ulf Samuelsson
2008-02-01 20:52 ` Stelian Pop
2008-02-02 5:59 ` Stefan Roese [this message]
2008-02-02 21:25 ` Stelian Pop
2008-02-03 6:44 ` Ulf Samuelsson
2008-02-03 15:49 ` Wolfgang Denk
2008-02-03 12:54 ` Haavard Skinnemoen
2008-02-03 20:58 ` Jean-Christophe PLAGNIOL-VILLARD
2008-02-04 9:42 ` [U-Boot-Users] [PATCH ARM] AT91CAP9 support : move board files to Atmel vendor directory Stelian Pop
2008-02-04 9:54 ` Stefan Roese
2008-02-04 10:26 ` [U-Boot-Users] [PATCH ARM] AT91CAP9 support : move board files to Atmel vendordirectory Peter Pearse
2008-02-02 11:40 ` [U-Boot-Users] [PATCH] Move AT91RM9200DK board support under board/atmel Haavard Skinnemoen
2008-02-02 12:37 ` Haavard Skinnemoen
2008-02-14 14:20 ` Peter Pearse
2008-02-02 12:47 ` Stefan Roese
2008-01-31 21:15 ` [U-Boot-Users] [PATCH 5/5] AT91CAP9 support : MACB changes Stelian Pop
-- strict thread matches above, loose matches on Subject: below --
2008-01-29 19:49 [U-Boot-Users] [PATCH ARM 0/5] AT91CAP9 support Stelian Pop
2008-01-29 19:49 ` [U-Boot-Users] [PATCH 4/5] AT91CAP9 support : board/ files Stelian Pop
2008-01-31 7:28 ` Jean-Christophe PLAGNIOL-VILLARD
2008-01-31 10:13 ` Ladislav Michl
2008-01-31 21:20 ` Stelian Pop
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=200802020659.35968.sr@denx.de \
--to=sr@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