Any special reason this in MPC5121 Revision 2 only?
I did not see anything that contradicted the rev 3 manual in the code.

But there is one harmless bug (since the NFC_CMD_FAIL symbol is never used), i.e.

git diff

diff --git a/drivers/mtd/nand/mpc5121_nfc.c b/drivers/mtd/nand/mpc5121_nfc.c
index 191bf99..38a99ef 100644
--- a/drivers/mtd/nand/mpc5121_nfc.c
+++ b/drivers/mtd/nand/mpc5121_nfc.c

@@ -100,7 +100,7 @@
  #define NFC_OUTPUT             (1 << 3)
  #define NFC_ID                 (1 << 4)
  #define NFC_STATUS             (1 << 5)
-#define NFC_CMD_FAIL           (1 << 15)
+#define NFC_CMD_FAIL           (1 << 14)
  #define NFC_INT                        (1 << 15)

  /* Bit Definitions: NFC_WRPROT */

The NAND chip is is detected (given that I disable the Revison check),
but the /dev7mtd0 is not accessable as block device. Is that normal?
The UBIFS also gives up I am assuming because the /dev/mtd0 is not block device.

rg
kd

On 02/16/2010 08:11 AM, Artem Bityutskiy wrote:

On Mon, 2010-02-15 at 18:35 +0100, Anatolij Gustschin wrote:

quoted

Adds NAND Flash Controller driver for MPC5121 Revision 2.
All device features, except hardware ECC and power management,
are supported.

Signed-off-by: Piotr Ziecik<redacted>
Signed-off-by: Wolfgang Denk<redacted>
Signed-off-by: Anatolij Gustschin<agust@denx.de>
Acked-by: Grant Likely<redacted>
Cc:[off-list ref]
Cc: Grant Likely<redacted>
Cc: John Rigby<redacted>
---
Changes since v3:
  - include 'asm/mpc5121.h' header instead of 'asm/mpc5xxx.h'.
    This change is needed because arch patch adding reset
    module definition was reworked and doesn't add mpc5121
    specific definitions to common header for mpc52xx/mpc5121.

Changes since v2:
  - move the arch bits into separate patch
    (it is the next patch in this series now)
  - use __devinit/__devexit/__devexit_p and __devinitdata

Changes since v1:
  - add logfile with changes since previous version

Changes since the patch version submitted in May 2009:

  - move mpc5121_nfc.h to the driver .c as there is only one user
  - remove DRV_VERSION macro
  - replace printk() by dev_*()
  - drop unnecessary .suspend and .resume initializations
  - remove duplicate .name/.owner settings
  - fix mpc5121_nfc_init() to "return of_register_platform_driver(&mpc5121_nfc_driver);"
  - move module_init() to just below the init function
  - remove MODULE_VERSION
  - use "mtd: Add MPC5121 NAND Flash Controller driver" as the subject,
    previously it was "mpc5121: Added NAND Flash Controller driver.

  drivers/mtd/nand/Kconfig       |    7 +
  drivers/mtd/nand/Makefile      |    1 +
  drivers/mtd/nand/mpc5121_nfc.c |  916 ++++++++++++++++++++++++++++++++++++++++
  3 files changed, 924 insertions(+), 0 deletions(-)
  create mode 100644 drivers/mtd/nand/mpc5121_nfc.c

Pushed to my l2-mtd-2.6.git / dunno.