From: John Ogness <john.ogness@linutronix.de>
To: linux-mtd@lists.infradead.org
Cc: s.hauer@pengutronix.de
Subject: [PATCH] mxc_nand: i.MX35 hardware ECC fixups
Date: Tue, 03 Nov 2009 18:06:13 +0100 [thread overview]
Message-ID: <804opb7d2y.fsf@merkur.tec.linutronix.de> (raw)
This patch is against the recently posted patches from Sascha Hauer
(21 Oct 2009 mxc_nand). This patch does the following:
- fixes ECC hardware error detection for the i.MX35
- sets up an appropriate ECC mode (4-bit) for the i.MX35
- configures the spare area size for the i.MX35
I tested this patch with ubifs on an i.MX35 PDK board.
Signed-off-by: John Ogness <john.ogness@linutronix.de>
---
mxc_nand.c | 35 ++++++++++++++++++++++++++++++-----
1 file changed, 30 insertions(+), 5 deletions(-)
diff --git a/drivers/mtd/nand/mxc_nand.c b/drivers/mtd/nand/mxc_nand.c
index d5445cd..de056aa 100644
--- a/drivers/mtd/nand/mxc_nand.c
+++ b/drivers/mtd/nand/mxc_nand.c
@@ -86,6 +86,7 @@
* Status operation */
#define NFC_INT 0x8000
+#define NFC_ECC_4BIT (1 << 0)
#define NFC_SP_EN (1 << 2)
#define NFC_ECC_EN (1 << 3)
#define NFC_INT_MSK (1 << 4)
@@ -351,11 +352,25 @@ static int mxc_nand_correct_data(struct mtd_info *mtd, u_char *dat,
*/
uint16_t ecc_status = readw(host->regs + NFC_ECC_STATUS_RESULT);
- if (((ecc_status & 0x3) == 2) || ((ecc_status >> 2) == 2)) {
- DEBUG(MTD_DEBUG_LEVEL0,
- "MXC_NAND: HWECC uncorrectable 2-bit ECC error\n");
- return -1;
- }
+ if (nfc_is_v21()) {
+ if ((ecc_status & 0xf) > 1 ||
+ ((ecc_status >> 4) & 0xf) > 1 ||
+ ((ecc_status >> 8) & 0xf) > 1 ||
+ ((ecc_status >> 12) & 0xf) > 1) {
+ DEBUG(MTD_DEBUG_LEVEL0,
+ "MXC_NAND: "
+ "HWECC uncorrectable 2-bit ECC error\n");
+ return -1;
+ }
+ } else if (nfc_is_v1()) {
+ if (((ecc_status & 0x3) == 2) || ((ecc_status >> 2) == 2)) {
+ DEBUG(MTD_DEBUG_LEVEL0,
+ "MXC_NAND: "
+ "HWECC uncorrectable 2-bit ECC error\n");
+ return -1;
+ }
+ } else
+ BUG();
return 0;
}
@@ -756,6 +771,8 @@ static int __init mxcnd_probe(struct platform_device *pdev)
tmp = readw(host->regs + NFC_CONFIG1);
tmp |= NFC_INT_MSK;
tmp &= ~NFC_SP_EN;
+ if (nfc_is_v21())
+ tmp |= NFC_ECC_4BIT;
writew(tmp, host->regs + NFC_CONFIG1);
init_waitqueue_head(&host->irq_waitq);
@@ -773,6 +790,14 @@ static int __init mxcnd_probe(struct platform_device *pdev)
/* Unlock the internal RAM Buffer */
writew(0x2, host->regs + NFC_CONFIG);
+ if (nfc_is_v21()) {
+ /* configure spare size (in 16-bit blocks) */
+ tmp = readw(host->regs + NFC_RSLTSPARE_AREA);
+ tmp &= ~0xff;
+ tmp |= host->spare_len / 2;
+ writew(tmp, host->regs + NFC_RSLTSPARE_AREA);
+ }
+
/* Blocks to be unlocked */
if (nfc_is_v21()) {
writew(0x0, host->regs + NFC_V21_UNLOCKSTART_BLKADDR);
next reply other threads:[~2009-11-03 17:06 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-03 17:06 John Ogness [this message]
[not found] ` <528f13590911071823sa26e1ane027d51d16357c79@mail.gmail.com>
[not found] ` <80y6mg5e34.fsf@merkur.tec.linutronix.de>
2009-12-12 23:58 ` [PATCH] mxc_nand: i.MX35 hardware ECC fixups alfred steele
2009-12-13 14:00 ` Eric Bénard
2009-12-13 14:36 ` John Ogness
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=804opb7d2y.fsf@merkur.tec.linutronix.de \
--to=john.ogness@linutronix.de \
--cc=linux-mtd@lists.infradead.org \
--cc=s.hauer@pengutronix.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