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
next prev parent 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox