All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] nand flash controller on 440ep/epx
Date: Fri, 6 Oct 2006 21:40:21 +0200	[thread overview]
Message-ID: <200610062140.21393.sr@denx.de> (raw)
In-Reply-To: <1628E43D99629C46988BE46087A3FBB9729A26@ep-01.EmbeddedPlanet.local>

Hi Jeff,

On Friday 06 October 2006 19:56, Jeff Mann wrote:
> It appears that the nand flash controller support recently added to
> u-boot for the ppc400ep/epx only supports the use of a single nand chip.

Yes. That's the current status.

> The nand flash controller supports up to four devices using chip selects
> 0-3. I am working on making two devices work right now for my use, but I
> would like to update the flash-controller support. I am open for
> comments on this.
>
> What I think I need to do is add an #config CFG_440_USENANDCONTROLLER
> option and an entry into the nand_chip struct for holding the chip
> select numbers for each chip. Then in addition to configuring an array
> of nand bases {CFG_NAND_BASE} (which would all be the same address
> because it is the address of the nand controller), the chip selects are
> put in a configured array and put into the nand_chip nand struct. The
> read and write functions would then need to be updated to enable the
> right chip select in the nand controller config register too.
>
> Comments? Suggestions? What am I missing?

It seems to me that we are missing a board/cpu specific function to select a 
different NAND chip, when the device is changed via the "nand device x" 
command. This 440EP(x) specific function would then setup the NDFC 
configuration register (NDFC0_CR) to enable the desired NAND chip select. And 
to make it easy I suggest that NAND device 0 represents chips select 0 and so 
on... So no need for additional config options or config arrays.

What do you think?

Best regards,
Stefan

  reply	other threads:[~2006-10-06 19:40 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-06 17:56 [U-Boot-Users] nand flash controller on 440ep/epx Jeff Mann
2006-10-06 19:40 ` Stefan Roese [this message]
  -- strict thread matches above, loose matches on Subject: below --
2006-10-06 20:51 Jeff Mann
2006-10-07 14:15 ` Stefan Roese
2006-10-09 14:15 Jeff Mann
2006-10-09 15:14 ` Stefan Roese

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=200610062140.21393.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.