public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3] Adding support for DevKit8000
Date: Thu, 20 Aug 2009 22:20:32 +0200	[thread overview]
Message-ID: <20090820202032.GA8470@game.jcrosoft.org> (raw)
In-Reply-To: <f67028d40908201137p509226f1vb3bd200e6f2203e9@mail.gmail.com>

On 20:37 Thu 20 Aug     , Frederik Kriewitz wrote:
> On Thu, Aug 20, 2009 at 7:28 PM, Dirk Behme<dirk.behme@googlemail.com> wrote:
> > Peter Tyser wrote:
> >>
> >> Hi Frederik,
> >> I had some minor aesthetic nitpicks. ?I'd change the title to "Add
> >> support for the DevKit8000 board".
> >>
> 
> I'll fix them
> 
> >>>
> >>> ?Unknown / orphaned boards:
> >>> diff --git a/MAKEALL b/MAKEALL
> >>> index edebaea..34235b7 100755
> >>> --- a/MAKEALL
> >>> +++ b/MAKEALL
> >>> @@ -581,6 +581,7 @@ LIST_ARM_CORTEX_A8=" ? ? ? ? ? ? ? ?\
> >>> ? ? ? ?omap3_pandora ? ? ? ? ? \
> >>> ? ? ? ?omap3_zoom1 ? ? ? ? ? ? \
> >>> ? ? ? ?omap3_zoom2 ? ? ? ? ? ? \
> >>> + ? ? ? devkit8000 ? ? ?\
> >>> ?"
> >>
> >> You should maintain the alphabetical order of LIST_ARM_CORTEX_A8.
> >
> > What's about using 'omap3_devkit8000' ?
> 
> On Thu, Aug 20, 2009 at 7:02 PM, Peter Tyser<ptyser@xes-inc.com> wrote:
> > Hi Frederik,
> > I had some minor aesthetic nitpicks.  I'd change the title to "Add
> > support for the DevKit8000 board".
> 
> Ok, I'll fix them.
> 
> Jean-Christophe asked me to move it out of the omap3 "vendor" directory:
I agree with Dirk for the config name it will be good to have omap3_devkit8000
but if you prefer devkit8000 it's fine also

and yes I've ask you about the boardir to move it out of the omap3 dirent

Best Regards,
J.

  reply	other threads:[~2009-08-20 20:20 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 [this message]
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
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=20090820202032.GA8470@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.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