Fix bug in PPC440 NAND driver cpu/ppc4xx/ndfc.c
authorStefan Roese <sr@denx.de>
Tue, 24 Oct 2006 16:06:48 +0000 (18:06 +0200)
committerStefan Roese <sr@denx.de>
Tue, 24 Oct 2006 16:13:43 +0000 (18:13 +0200)
Patch by Stefan Roese, 24 Oct 2006

CHANGELOG
cpu/ppc4xx/ndfc.c

index ed6396df8fe8e1f8bb5a464a4f05d0f06e6c7fe5..b9dd4469a5c5f848b70758d24fcaeefb229966ed 100644 (file)
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -4,6 +4,9 @@ Changes for U-Boot 1.1.5:
 
 * Fix sequoia separate object direcory building problems.
 
+* Fix bug in PPC440 NAND driver cpu/ppc4xx/ndfc.c
+  Patch by Stefan Roese, 24 Oct 2006
+
 * Cleanup compile warnings. Prepare for release 1.1.5
 
 * Fix compile problem in include/configs/ep82xxm.h
index 2c44111da6b9e0971600de5cbfa4b85b41328dec..352173128de102e9c5f951dfc81e74ba281e85b8 100644 (file)
@@ -144,16 +144,21 @@ static int ndfc_verify_buf(struct mtd_info *mtdinfo, const uint8_t *buf, int len
 
 void board_nand_select_device(struct nand_chip *nand, int chip)
 {
+       /*
+        * Don't use "chip" to address the NAND device,
+        * generate the cs from the address where it is encoded.
+        */
+       int cs = (ulong)nand->IO_ADDR_W & 0x00000003;
        ulong base = (ulong)nand->IO_ADDR_W & 0xfffffffc;
 
        /* Set NandFlash Core Configuration Register */
        /* 1col x 2 rows */
-       out32(base + NDFC_CCR, 0x00000000 | (chip << 24));
+       out32(base + NDFC_CCR, 0x00000000 | (cs << 24));
 }
 
 void board_nand_init(struct nand_chip *nand)
 {
-       int chip = (ulong)nand->IO_ADDR_W & 0x00000003;
+       int cs = (ulong)nand->IO_ADDR_W & 0x00000003;
        ulong base = (ulong)nand->IO_ADDR_W & 0xfffffffc;
 
        nand->eccmode = NAND_ECC_SOFT;
@@ -181,8 +186,8 @@ void board_nand_init(struct nand_chip *nand)
        /*
         * Select required NAND chip in NDFC
         */
-       board_nand_select_device(nand, chip);
-       out32(base + NDFC_BCFG0 + (chip << 2), 0x80002222);
+       board_nand_select_device(nand, cs);
+       out32(base + NDFC_BCFG0 + (cs << 2), 0x80002222);
 }
 
 #endif