From patchwork Wed Dec 19 10:27:18 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Josh Wu X-Patchwork-Id: 207309 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from merlin.infradead.org (merlin.infradead.org [IPv6:2001:4978:20e::2]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 67F622C008E for ; Wed, 19 Dec 2012 21:34:18 +1100 (EST) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1TlGxy-0008CL-Ou; Wed, 19 Dec 2012 10:33:35 +0000 Received: from newsmtp5.atmel.com ([204.2.163.5] helo=sjogate2.atmel.com) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1TlGx2-0007xz-ER; Wed, 19 Dec 2012 10:32:38 +0000 Received: from penbh01.corp.atmel.com ([10.168.5.31]) by sjogate2.atmel.com (8.13.6/8.13.6) with ESMTP id qBJAQuYM018602; Wed, 19 Dec 2012 02:27:07 -0800 (PST) Received: from penmb01.corp.atmel.com ([10.168.5.33]) by penbh01.corp.atmel.com with Microsoft SMTPSVC(6.0.3790.3959); Wed, 19 Dec 2012 18:32:12 +0800 Received: from shaarm01.corp.atmel.com ([10.217.6.34]) by penmb01.corp.atmel.com with Microsoft SMTPSVC(6.0.3790.3959); Wed, 19 Dec 2012 18:32:11 +0800 From: Josh Wu To: linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/3] MTD: at91: atmel_nand: for PMECC, add code to choose the ecc bits and sector size according to the ONFI parameter ECC requirement. Date: Wed, 19 Dec 2012 18:27:18 +0800 Message-Id: <1355912840-29501-2-git-send-email-josh.wu@atmel.com> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1355912840-29501-1-git-send-email-josh.wu@atmel.com> References: <1355912840-29501-1-git-send-email-josh.wu@atmel.com> X-OriginalArrivalTime: 19 Dec 2012 10:32:12.0013 (UTC) FILETIME=[1B0B19D0:01CDDDD4] X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20121219_053236_832856_AD89C37F X-CRM114-Status: GOOD ( 18.89 ) X-Spam-Score: -2.6 (--) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-2.6 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 SPF_PASS SPF: sender matches SPF record -0.7 RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: nicolas.ferre@atmel.com, plagnioj@jcrosoft.com, Josh Wu X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: linux-mtd-bounces@lists.infradead.org Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org This patch will check NAND flash's ecc minimum requirement in ONFI parameter. If it is equal or smaller than pmecc-cap in dtsi, then use ecc_bits in ONFI. otherwise, return an error since pmecc-cap in dtsi don't meet the ecc minimum reqirement. This patch also check sector size (codeword) requirement in ONFI. If it is equal or bigger than sector_size in dtsi, then use the one of ONFI. otherwise return error. Currently we don't support to read the ECC parameter in ONFI extended parameter page. So in that case we just use the value specified in dts. For non-ONFI nand flash, we assume the minimum ecc requirement is 2bits in 512 bytes. Signed-off-by: Josh Wu --- drivers/mtd/nand/atmel_nand.c | 89 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 89 insertions(+) diff --git a/drivers/mtd/nand/atmel_nand.c b/drivers/mtd/nand/atmel_nand.c index 9144557..7855ccd 100644 --- a/drivers/mtd/nand/atmel_nand.c +++ b/drivers/mtd/nand/atmel_nand.c @@ -901,6 +901,84 @@ static void atmel_pmecc_core_init(struct mtd_info *mtd) pmecc_writel(host->ecc, CTRL, PMECC_CTRL_ENABLE); } +/* + * Get ECC requirement in ONFI parameters, returns -1 if ONFI + * parameters is not supported. + * return 0 if success to get the ECC requirement. + */ +static int get_onfi_ecc_param(struct nand_chip *chip, + int *ecc_bits, int *sector_size) +{ + *ecc_bits = *sector_size = 0; + + if (chip->onfi_params.ecc_bits == 0xff) + /* TODO: the sector_size and ecc_bits need to be find in + * extended ecc parameter, currently we don't support it. + */ + return -1; + + *ecc_bits = chip->onfi_params.ecc_bits; + + /* The default sector size (ecc codeword size) is 512 */ + *sector_size = 512; + + return 0; +} + +/* + * Choose ecc cap and sector size in atmel_nand_host according to ONFI + * parameters ecc requirement. + * return 0 if success. otherwise return error code. + */ +static int pmecc_choose_ecc_bits(struct atmel_nand_host *host, + struct nand_chip *chip) +{ + int ecc_bits, sector_size; + + if (!get_onfi_ecc_param(chip, &ecc_bits, §or_size)) { + dev_info(host->dev, "ONFI params, minimum required ECC: %dbits in %d bytes\n", + ecc_bits, sector_size); + + if (ecc_bits > host->pmecc_corr_cap) { + dev_err(host->dev, "Error: Need to set a bigger pmecc-cap in dts. Current is: %d\n", + host->pmecc_corr_cap); + return -EINVAL; + } + if (sector_size < host->pmecc_sector_size) { + dev_err(host->dev, "Error: Need to set a smaller pmecc-sector-size in dts. Current is: %d\n", + host->pmecc_sector_size); + return -EINVAL; + } + + /* use the most fitable ecc bits (the near bigger one ) */ + if (ecc_bits <= 2) + host->pmecc_corr_cap = 2; + else if (ecc_bits <= 4) + host->pmecc_corr_cap = 4; + else if (ecc_bits < 8) + host->pmecc_corr_cap = 8; + else if (ecc_bits < 12) + host->pmecc_corr_cap = 12; + else if (ecc_bits < 24) + host->pmecc_corr_cap = 24; + else + return -EINVAL; + + /* use the most fitable sector size (the near smaller one ) */ + if (sector_size >= 1024) + host->pmecc_sector_size = 1024; + else if (sector_size >= 512) + host->pmecc_sector_size = 512; + else + return -EINVAL; + } else { + dev_info(host->dev, + "NAND chip ECC reqirement is in Extended ONFI parameter, we don't support yet. So use default setting in dts\n"); + } + + return 0; +} + static int __init atmel_pmecc_nand_init_params(struct platform_device *pdev, struct atmel_nand_host *host) { @@ -909,8 +987,19 @@ static int __init atmel_pmecc_nand_init_params(struct platform_device *pdev, struct resource *regs, *regs_pmerr, *regs_rom; int cap, sector_size, err_no; + if (nand_chip->onfi_version) { + /* Choose ecc cap and sector size base on ONFI parameters */ + err_no = pmecc_choose_ecc_bits(host, nand_chip); + if (err_no) + return err_no; + } else { + dev_info(host->dev, "NAND chip is not ONFI compliant, assume ecc_bits is 2"); + host->pmecc_corr_cap = 2; + } + cap = host->pmecc_corr_cap; sector_size = host->pmecc_sector_size; + dev_info(host->dev, "Initialize PMECC params, cap: %d, sector: %d\n", cap, sector_size);