All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfram Sang <w.sang@pengutronix.de>
To: Huang Shijie <b32955@freescale.com>
Cc: thierry.nolf.barco@gmail.com, LW@KARO-electronics.de,
	linux-mtd@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org, dedekind1@gmail.com
Subject: Re: [PATCH v7 0/3]  add the GPMI controller driver for IMX23/IMX28
Date: Thu, 30 Jun 2011 12:40:09 +0200	[thread overview]
Message-ID: <20110630104009.GC1997@pengutronix.de> (raw)
In-Reply-To: <1309335878-16678-1-git-send-email-b32955@freescale.com>

[-- Attachment #1: Type: text/plain, Size: 1821 bytes --]

On Wed, Jun 29, 2011 at 04:24:35PM +0800, Huang Shijie wrote:
> The general-purpose media interface(GPMI) controller is a flexible interface
> to up to several NAND flashs.
> 
> The Bose Ray-Choudhury Hocquenghem(BCH) module is a hardware ECC accelerator.
> 
> With the help of BCH, the GPMI controller can choose to do the hardware ECC or
> not.
> 
> This driver is a _pure_ MTD NAND controller driver now.

It seems writing does not work for me. I use a MX28EVK, NAND is reported as:

[    1.670000] NAND device: Manufacturer ID: 0x2c, Chip ID: 0xdc (Micron MT29F4G08ABADAWP)

With ubiformat or the mtd_stresstest-module, I get stuff like this:

root@device:~ ubiformat /dev/mtd0
ubiformat: mtd0 (nand), size 536870912 bytes (512.0 MiB), 4096 eraseblocks of 131072 bytes (128.0 KiB), min. I/O size 2048 bytes
libscan: scanning eraseblock 4095 -- 100 % complete  
ubiformat: 4094 eraseblocks are supposedly empty
ubiformat: 2 bad eraseblocks found, numbers: 1243, 3935
ubiformat: formatting eraseblock 2 --  0 % complete  
[  172.430000] [ start_dma_without_bch_irq : 393 ] DMA timeout!!!
[  173.430000] [ start_dma_with_bch_irq : 424 ] bch timeout!!!
[  173.430000] [ mil_ecc_write_page : 1647 ] Error in ECC-based write: -110
[  173.440000] [ send_command : 305 ] step 1 error
[  173.440000] [ mil_cmd_ctrl : 1369 ] Chip: 0, Error -1
[  173.450000] [ send_command : 305 ] step 1 error
[  173.450000] [ mil_cmd_ctrl : 1369 ] Chip: 0, Error -1
[  173.460000] [ read_data : 386 ] step 1 error
...

Reading works though, mtd_readtest passes. Wild guess: something wrong
with the safe timings, perhaps?

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: w.sang@pengutronix.de (Wolfram Sang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 0/3]  add the GPMI controller driver for IMX23/IMX28
Date: Thu, 30 Jun 2011 12:40:09 +0200	[thread overview]
Message-ID: <20110630104009.GC1997@pengutronix.de> (raw)
In-Reply-To: <1309335878-16678-1-git-send-email-b32955@freescale.com>

On Wed, Jun 29, 2011 at 04:24:35PM +0800, Huang Shijie wrote:
> The general-purpose media interface(GPMI) controller is a flexible interface
> to up to several NAND flashs.
> 
> The Bose Ray-Choudhury Hocquenghem(BCH) module is a hardware ECC accelerator.
> 
> With the help of BCH, the GPMI controller can choose to do the hardware ECC or
> not.
> 
> This driver is a _pure_ MTD NAND controller driver now.

It seems writing does not work for me. I use a MX28EVK, NAND is reported as:

[    1.670000] NAND device: Manufacturer ID: 0x2c, Chip ID: 0xdc (Micron MT29F4G08ABADAWP)

With ubiformat or the mtd_stresstest-module, I get stuff like this:

root at device:~ ubiformat /dev/mtd0
ubiformat: mtd0 (nand), size 536870912 bytes (512.0 MiB), 4096 eraseblocks of 131072 bytes (128.0 KiB), min. I/O size 2048 bytes
libscan: scanning eraseblock 4095 -- 100 % complete  
ubiformat: 4094 eraseblocks are supposedly empty
ubiformat: 2 bad eraseblocks found, numbers: 1243, 3935
ubiformat: formatting eraseblock 2 --  0 % complete  
[  172.430000] [ start_dma_without_bch_irq : 393 ] DMA timeout!!!
[  173.430000] [ start_dma_with_bch_irq : 424 ] bch timeout!!!
[  173.430000] [ mil_ecc_write_page : 1647 ] Error in ECC-based write: -110
[  173.440000] [ send_command : 305 ] step 1 error
[  173.440000] [ mil_cmd_ctrl : 1369 ] Chip: 0, Error -1
[  173.450000] [ send_command : 305 ] step 1 error
[  173.450000] [ mil_cmd_ctrl : 1369 ] Chip: 0, Error -1
[  173.460000] [ read_data : 386 ] step 1 error
...

Reading works though, mtd_readtest passes. Wild guess: something wrong
with the safe timings, perhaps?

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20110630/391f685e/attachment.sig>

  parent reply	other threads:[~2011-06-30 10:40 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-29  8:24 [PATCH v7 0/3] add the GPMI controller driver for IMX23/IMX28 Huang Shijie
2011-06-29  8:24 ` Huang Shijie
2011-06-29  8:24 ` Huang Shijie
2011-06-29  8:24   ` Huang Shijie
2011-06-29  8:24 ` [PATCH v7 1/3] MTD : add the common code for GPMI-NFC controller driver Huang Shijie
2011-06-29  8:24   ` Huang Shijie
2011-06-29 10:06   ` Wolfram Sang
2011-06-29 10:06     ` Wolfram Sang
2011-06-29 10:33     ` Wolfram Sang
2011-06-29 10:33       ` Wolfram Sang
2011-06-29 10:46       ` Huang Shijie
2011-06-29 10:46         ` Huang Shijie
2011-06-29 12:29         ` Wolfram Sang
2011-06-29 12:29           ` Wolfram Sang
2011-06-29 12:42           ` Russell King - ARM Linux
2011-06-29 12:42             ` Russell King - ARM Linux
2011-06-29 12:49             ` Wolfram Sang
2011-06-29 12:49               ` Wolfram Sang
2011-06-29 14:00           ` Shawn Guo
2011-06-29 14:00             ` Shawn Guo
2011-06-29 14:15             ` Wolfram Sang
2011-06-29 14:15               ` Wolfram Sang
2011-06-29 14:37               ` Shawn Guo
2011-06-29 14:37                 ` Shawn Guo
2011-06-30  4:46                 ` Wolfram Sang
2011-06-30  4:46                   ` Wolfram Sang
2011-06-30  5:28                   ` Shawn Guo
2011-06-30  5:28                     ` Shawn Guo
2011-06-30  8:06                     ` [PATCH] dmaengine: mxs-dma: skip request_irq for NO_IRQ Shawn Guo
2011-06-30  8:06                       ` Shawn Guo
2011-07-07  0:15                       ` Shawn Guo
2011-07-07  0:15                         ` Shawn Guo
2011-07-07  2:06                         ` Vinod Koul
2011-07-07  2:06                           ` Vinod Koul
2011-07-07  3:24                           ` Shawn Guo
2011-07-07  3:24                             ` Shawn Guo
2011-07-07  3:23                             ` Huang Shijie
2011-07-07  3:23                               ` Huang Shijie
2011-07-13 23:31                             ` Koul, Vinod
2011-07-13 23:31                               ` Koul, Vinod
2011-06-29 12:38   ` [PATCH v7 1/3] MTD : add the common code for GPMI-NFC controller driver Russell King - ARM Linux
2011-06-29 12:38     ` Russell King - ARM Linux
2011-06-30  1:50     ` Huang Shijie
2011-06-30  1:50       ` Huang Shijie
2011-06-30 10:27   ` Wolfram Sang
2011-06-30 10:27     ` Wolfram Sang
2011-06-30 17:04     ` Wolfram Sang
2011-06-30 17:04       ` Wolfram Sang
2011-07-01  5:41       ` Artem Bityutskiy
2011-07-01  5:41         ` Artem Bityutskiy
2011-07-01  6:47         ` Huang Shijie
2011-07-01  6:47           ` Huang Shijie
2011-06-29  8:24 ` [PATCH v7 2/3] MTD : add support for imx23 and imx28 Huang Shijie
2011-06-29  8:24   ` Huang Shijie
2011-06-29  8:24 ` [PATCH v7 3/3] MTD : add GPMI-NFC driver in the config and Makefile Huang Shijie
2011-06-29  8:24   ` Huang Shijie
2011-06-30 10:40 ` Wolfram Sang [this message]
2011-06-30 10:40   ` [PATCH v7 0/3] add the GPMI controller driver for IMX23/IMX28 Wolfram Sang

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=20110630104009.GC1997@pengutronix.de \
    --to=w.sang@pengutronix.de \
    --cc=LW@KARO-electronics.de \
    --cc=b32955@freescale.com \
    --cc=dedekind1@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=thierry.nolf.barco@gmail.com \
    /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.