public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] mtd: nand: mxs: Add support for multiple NAND chips
Date: Sat, 6 Dec 2014 14:07:00 +0100	[thread overview]
Message-ID: <201412061407.00278.marex@denx.de> (raw)
In-Reply-To: <1417526774-32508-1-git-send-email-sr@denx.de>

On Tuesday, December 02, 2014 at 02:26:14 PM, Stefan Roese wrote:
> This patch adds support for multiple NAND chips connected to the
> i.MX6. Linux already supports this configuration. So lets port
> the missing features to the U-Boot driver to support more than
> one NAND chip here as well.
> 
> The necessary changes in detail are:
> 
> - Only use DMA channel 0 for all NAND chips:
>   Linux: a7c12d01 (mtd: gpmi: use DMA channel 0 for all the
>                    nand chips)
>          d159d8b7 (mtd: gpmi: decouple the chip select from
>                    the DMA channel)
> - On i.MX6 only use ready/busy pin for CS0:
>   Linux: 7caa4fd2 (mtd: gpmi: imx6: fix the wrong method for
>                    checking ready/busy)
> 
> To enable this feature the board needs to configure
> CONFIG_SYS_NAND_MAX_CHIPS to 2 (or more).
> 
> With these changes I'm able to detect and acces 2 NAND chips:
> 
> => nand device
> 
> Device 0: 2x nand0, sector size 128 KiB
>   Page size      2048 b
>   OOB size         64 b
>   Erase size   131072 b

Shouldn't you see "Device 0" and "Device 1" ?

> Please note that this is also needed to support a NAND chip with
> multiple chips embedded in one die, e.g. Micron MT29F32G08QAA.
> 
> Tested on a i.MX6DL based board with 2 Micron MT29F4G08AB
> chips.
> 
> Signed-off-by: Stefan Roese <sr@denx.de>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Fabio Estevam <fabio.estevam@freescale.com>
> Cc: Scott Wood <scottwood@freescale.com>

Looks pretty trivial, thanks!

Reviewed-by: Marek Vasut <marex@denx.de>

Best regards,
Marek Vasut

  reply	other threads:[~2014-12-06 13:07 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-02 13:26 [U-Boot] [PATCH] mtd: nand: mxs: Add support for multiple NAND chips Stefan Roese
2014-12-06 13:07 ` Marek Vasut [this message]
2014-12-06 23:45   ` Scott Wood
2014-12-07  1:32     ` Marek Vasut
2014-12-07  1:50       ` Scott Wood
2014-12-07  5:26         ` Marek Vasut
2014-12-08 22:53           ` Scott Wood
2014-12-09  7:20             ` Stefan Roese
2014-12-09  7:34               ` Marek Vasut

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=201412061407.00278.marex@denx.de \
    --to=marex@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