public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Michal Simek <monstr@seznam.cz>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ppc440: New board: Avnet Virtex5 FXT Evaluation
Date: Mon, 25 Aug 2008 15:36:40 +0200	[thread overview]
Message-ID: <48B2B568.7000602@seznam.cz> (raw)
In-Reply-To: <aa76a2be0808250535o7ae9db19hbd38c750c8c88e54@mail.gmail.com>


Hi,

> Hello Again
> 
>  Michal, on xilinx boards almost everything can be handled by
> edk+xparameters, but in other boards this is not true. Last month I
> was working on a board that had a weird watchdog and a memory that
> needed to handled by a propietary IP, in this case, we need a board
> directory because the generic board should not support it: otherwise
> the generic board will have many defiles like #ifdef
> STRANGE_BOARD_USED_ONCE :)

I know that I wrote that not every possible choices are covered but you wrote
that you use proprietary IP.
Yes, we will need board directory for proprietary solution. Look at my first
email I wrote about. Of course if you do your project you should create some
directories only for you.
But I think we don't need special folder for reference design and board.

Your STRANGE_BOARD_USED_ONCE choice is due to you use something strange which
should not go to mainline U-BOOT. I don't like strange things. If is something
strange you have to find out why. Just writing strange is more stranger for
others which will read your code.

Please add more choices which you need in your config.

Memory,
flash,
iic,
spi,
watchdog,
dtc,
maybe gpio - but this is not important
dcr reading,
systemace,
ethernet controllers,
uarts

I see maybe 20 ifdef choices not more right now. Please correct me if I am wrong.

Michal


>   Waiting for your comments.
> 
> 
> 

  reply	other threads:[~2008-08-25 13:36 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-22 16:35 [U-Boot] [PATCH] Remove gap fill in srec object Ricardo Ribalda Delgado
2008-08-22 16:35 ` [U-Boot] [PATCH] ppc440: ml507: Add .gitignore Ricardo Ribalda Delgado
2008-08-22 16:35   ` [U-Boot] [PATCH] ppc440: New board: Avnet Virtex5 FXT Evaluation Ricardo Ribalda Delgado
2008-08-23 19:05     ` Michal Simek
2008-08-24  0:42       ` Ricardo Ribalda Delgado
2008-08-24  9:49         ` Michal Simek
2008-08-25  8:09           ` Stefan Roese
2008-08-25  9:33             ` Michal Simek
2008-08-25 12:28               ` Ricardo Ribalda Delgado
2008-08-25 12:35                 ` Ricardo Ribalda Delgado
2008-08-25 13:36                   ` Michal Simek [this message]
2008-08-25 13:37                 ` Michal Simek
2008-09-08 12:56   ` [U-Boot] [PATCH] ppc440: ml507: Add .gitignore Stefan Roese
2008-09-08 13:10     ` Ricardo Ribalda Delgado
2008-09-06 20:21 ` [U-Boot] [PATCH] Remove gap fill in srec object Wolfgang Denk
2008-09-07 21:10   ` Ricardo Ribalda Delgado

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=48B2B568.7000602@seznam.cz \
    --to=monstr@seznam.cz \
    --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