From patchwork Tue Jul 21 08:15:19 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peng Fan X-Patchwork-Id: 498050 X-Patchwork-Delegate: scottwood@freescale.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id A8CD01402AC for ; Tue, 21 Jul 2015 18:16:37 +1000 (AEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 705B34B615; Tue, 21 Jul 2015 10:16:35 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id fRK1ww1wBLtD; Tue, 21 Jul 2015 10:16:35 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id ED36A4B692; Tue, 21 Jul 2015 10:16:23 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id D20F64B62B for ; Tue, 21 Jul 2015 10:15:50 +0200 (CEST) Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id hBSaJABWckVU for ; Tue, 21 Jul 2015 10:15:49 +0200 (CEST) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from na01-bn1-obe.outbound.protection.outlook.com (mail-bn1on0140.outbound.protection.outlook.com [157.56.110.140]) by theia.denx.de (Postfix) with ESMTPS id D71B74B615 for ; Tue, 21 Jul 2015 10:15:42 +0200 (CEST) Received: from CH1PR03CA005.namprd03.prod.outlook.com (10.255.156.150) by CY1PR03MB1486.namprd03.prod.outlook.com (10.163.17.16) with Microsoft SMTP Server (TLS) id 15.1.213.14; Tue, 21 Jul 2015 08:15:38 +0000 Received: from BN1BFFO11FD017.protection.gbl (10.255.156.132) by CH1PR03CA005.outlook.office365.com (10.255.156.150) with Microsoft SMTP Server (TLS) id 15.1.201.16 via Frontend Transport; Tue, 21 Jul 2015 08:15:38 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=freescale.com; freescale.mail.onmicrosoft.com; dkim=none (message not signed) header.d=none; Received-SPF: Fail (protection.outlook.com: domain of freescale.com does not designate 192.88.168.50 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.168.50; helo=tx30smr01.am.freescale.net; Received: from tx30smr01.am.freescale.net (192.88.168.50) by BN1BFFO11FD017.mail.protection.outlook.com (10.58.144.80) with Microsoft SMTP Server (TLS) id 15.1.213.8 via Frontend Transport; Tue, 21 Jul 2015 08:15:38 +0000 Received: from linux-jyl1.ap.freescale.net (b51431-11.ap.freescale.net [10.193.102.86]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id t6L8FOn3024313; Tue, 21 Jul 2015 01:15:36 -0700 From: Peng Fan To: , Date: Tue, 21 Jul 2015 16:15:19 +0800 Message-ID: <1437466521-27856-1-git-send-email-Peng.Fan@freescale.com> X-Mailer: git-send-email 1.8.4 X-EOPAttributedMessage: 0 X-Microsoft-Exchange-Diagnostics: 1; BN1BFFO11FD017; 1:x6MSFbszezJqo2rCTkfZ6hQgsv9gFq6ss8MzLS5CBFI/T76H+VZBAHOOeDY9Y0IpxE9Wu8VswmQafjOAKRZNrTvk0deI0eeCSlPhaZKan7DIbYnZpDUYvbULKsNNTTRwbJwd28bcaRCpIDVDQ0bX+gqOmTtQTKKCFQpOWcH9kakDnudGQqnkgIN3mrRtQV1r3hi4TKg6QixUwRnKX7cNOqp/tjbisa64BTw43PgMB5P7Bh7M270oPMz6ICf7c6ZWRzXbSibi/K8XNerh7hPD8M+TGnX37KxE5n383mPo8VjkO8kT0YMoTLH2VVM4V/LqbLdW1OO+FSsmF34/Viugi9DFIUM2IoGAUbHUZ3KobhkweLtXSSjyhpocOk+UFiLat+sVmqzh2/fBs7m7neBrgfF0TYgFVjUwzQzuGQkwQszgqIBRwsjOrv0XkuDKsoBP X-Forefront-Antispam-Report: CIP:192.88.168.50; CTRY:US; IPV:NLI; EFV:NLI; SFV:NSPM; SFS:(10019020)(6009001)(2980300002)(339900001)(189002)(199003)(50226001)(5003940100001)(50466002)(6806004)(5001960100002)(107886002)(48376002)(106466001)(77096005)(77156002)(62966003)(92566002)(5001770100001)(15975445007)(189998001)(47776003)(50986999)(105606002)(229853001)(104016003)(46102003)(19580395003)(19580405001)(575784001)(86362001)(85426001)(87936001)(36756003)(4001430100001)(2101003); DIR:OUT; SFP:1102; SCL:1; SRVR:CY1PR03MB1486; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; MLV:sfv; MX:1; A:1; LANG:en; MIME-Version: 1.0 X-Microsoft-Exchange-Diagnostics: 1; CY1PR03MB1486; 2:U84dz4teMyMci5ln6vvaA9gNCiaVtWDAiA/LB/WYs94AsUrw4fUKjEwTJkGuEO01; 3:nr3Npe/rJe0jC+GDcROEj9ovdFv/yyJ37o7Gw9UmAOQhZMmAcPyf9aWpMSvoT79ntmndXRYhq7GaRWQJZUXUC6CUN4T6dGwQKX6o3XIni04qdbaU3UEDA7kK75S4b8FWLudlVqlG+E93whr3Cl3py6ixTee7SxVyeLhFvT9P7oEfX6Hyg7E4Po41ylIU2+8yejeFC/g70unuwfXvkhyHAzPAyoqsLW6JMErig6X4Ky4=; 25:aks6ab9qdY7O4AqO8DHGrEq3DdsOV3nKFfMkTmSCyKLQ5dKCy/YZvWVJ+H+1aHyuCLo8kqLTW3vspN/F9WO6rmO6vBFCQYPwguyrHM3IkCHZIexZBpAIwaa6dNchHu/MhHyPgV2i5SYr6mgb+VQrRW7xh74z3E6QL8+OB8QvVl1SHLZhA3+p+hW9OIKdd9F/7PAwreJKwcbyudslrEawEm+Y14AgUFs0hBt2GHKLNVpovoQAGoEhoswDq13Z/I7RXDk1+Ww0wM/6dhMz/+bZJg==; 20:gunBXTexBpWP8FHgsQ8jQGw2e2g2D17pzLz041BziiBi9ADS1U+7q3SLyqsASBLxIxzGDWJbjQRBgDcpVZUzqg5UbBG01ban3E4frcI7kIWq/+T7INvrIvXR2tSiPBvuCurUopo03E5GPo7LL7nzXpT5Z9MkYipTR8+fcTV78xyDQmdH9iewesxQxZ9D84Wzt6CHoMoYvy5kTVGfJAQg1WdDIeO2/VTNi2qQnYfSOaEfVKcHfoDGO76NLPeiS+cq50ws5OoQPROQVnmHqnQtpZYeg3mu6o3Mew5w3xklzW5YtPEKm8PhUI64tgMKpqFgazKMy5cc0frcuvNslHEIHe0d1IJXqHqdPa9CV/tD//U= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:CY1PR03MB1486; CY1PR03MB1486: X-MS-Exchange-Organization-RulesExecuted X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(601004)(5005006)(3002001); SRVR:CY1PR03MB1486; BCL:0; PCL:0; RULEID:; SRVR:CY1PR03MB1486; X-Microsoft-Exchange-Diagnostics: 1; CY1PR03MB1486; 4:0keZeeNYxy/WxJ5IXok1Pm10cIaUlOlnR8Nele8S5dOIjMTJq9RUTxLI5QM1FIzrE7PL/aFOoXK8uefwNQxLjNPLVPy6IwDG+dMT2p4cIYiTU7JWiVMCR0jfawT1hnF6LDqrd7EYdITiXE2L7kVTXGSs0SPFrgJ1NfZPHEhv1pMNp5gctBTdsED/AXPYd7elsJWYW4a1hSby/2VL7M/h4jRzhnVC1UpeALj6uPX+ui3ICkG5D1uIJ8wQ8uqrV+JTmzoNOySYDcSUHqgtZrc3FfFU1o8C60tod3jOx9p6LDI= X-Forefront-PRVS: 0644578634 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; CY1PR03MB1486; 23:TMzcta22uNxSqWMEDoycI2t8P9o64mSCQeSVzUjpq?= =?us-ascii?Q?sr4AREcUJCboUzAlfh0ONfuKEtNNkS71OBLSpa7jxUrLjszaO79VB08zmwUH?= =?us-ascii?Q?DDB3zbrK+Dyb8LsJEkcO33M6Rq8XqV+FrZ1N2pXBdbgLSPQZxqLpJtIiJ8Lj?= =?us-ascii?Q?ROY8kEWA+nWZ1Fk4nCIV/5rYco3YaU8wzoOYfxiCvclB7TsPmRSJCXKA2ztn?= =?us-ascii?Q?E38/YHNBkwZcWzjfYpbVWhJMSc1kZLi3T1l3G5pBKnI0SpWPa4PW9t7uvJKd?= =?us-ascii?Q?rlQI42ozVHlOhvEQYWVmn11PQsloVpQ79d8h5/DYWWJniduKTbyfWOu9NygC?= =?us-ascii?Q?pN5ygoz3DElN5CiyXC2rJrKNmupvxxto5urxsGfmA1kN3M0dtEnd/vvY5G32?= =?us-ascii?Q?h+B4P82EN8koInKR8lpLjSpQO2zBib31Rmebpf3TEhRo57Cw6SR1S4VlDx2S?= =?us-ascii?Q?81zST96cXKz6ZLZDtYc7AeONLqJ0Y35mZMoI5HfOacdzeU91uiUO9wDqkHFQ?= =?us-ascii?Q?QGs5iYhU7lBrxAWx+sRhwMxb94l1fsp2NVNfzdklKfVIPuWiK1rxOghtF+bQ?= =?us-ascii?Q?kScwtjVR3CnOUV4aUnazk5s2uiAEsKcIAA95dnLYs7SP2MvIoMQaLmL4aKOs?= =?us-ascii?Q?zqSXCG7SmPYzY0yF2dCLvaa9kR2pBHGXtDyLKlgrCKdeJWZxGT4Wrj348lQD?= =?us-ascii?Q?N9UisHM3XocfBhJGC/EXHvk+bxyK9p5eV0nZb9JKU641Ix60kh5pMOC+jLqm?= =?us-ascii?Q?ILki5oXBzpvi8N5fje5AvQhCYxyy9Lu6DyC8fgeCb/0bblEDrCNjyFZqFpuy?= =?us-ascii?Q?8at9NL+qrnqIK8ba3PGpFFZ/NmBthzzmzcGW1xRn3lsNezImsIuyXE7tnvX0?= =?us-ascii?Q?GmmUuaYxMLAIRnOc7edCHbbk28N5FkfFrU7L8z4u5FqfE68xuJ41hVkN+HZ1?= =?us-ascii?Q?nJjVYdtMzN6ke0gRCYotTuZwajMncftsXBSTHyqcQ=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1; CY1PR03MB1486; 5:HfgLcJYmcyVFs9WMIriRCTP36I2e3rkfJMYhyZ1MK4OOt5SKodmZiQibi/NhWkU6bm9Hk4emg+IXXoEcC8aiLm2TMMk8ho7ugbe6C8cIEu71U35qXtyT9LYXs0f/sGodI01StmCd7rQiw0/2KYB7Eg==; 24:j9ClCV2o3q037A62QXT37h+lZOvcuaEZBcvTAmXSnmvwkVjixahAOy1LgkcqgcM64BXmKNnnn0Gyl2UvKWDBIxozrzYB1fZeoqyWw2J+6HM=; 20:9g+cfLL8z4CEGgh9bYtejHvaO/+mfW7w4x0HS9vMk9Rhycr+FXXcAzrMu0gI8Hh6XAYbP1T9upxewGRcge9lrw== X-OriginatorOrg: freescale.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Jul 2015 08:15:38.2957 (UTC) X-MS-Exchange-CrossTenant-Id: 710a03f5-10f6-4d38-9ff4-a80b81da590d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=710a03f5-10f6-4d38-9ff4-a80b81da590d; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY1PR03MB1486 Cc: marex@denx.de, u-boot@lists.denx.de Subject: [U-Boot] [PATCH resend V2 1/3] mtd: nand: mxs support oobsize bigger than 512 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.15 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" If ecc chunk data size is 512 and oobsize is bigger than 512, there is a chance that block_mark_bit_offset conflicts with bch ecc area. The following graph is modified from kernel gpmi-nand.c driver with each data block 512 bytes. We can see that Block Mark conflicts with ecc area from bch view. We can enlarge the ecc chunk size to avoid this problem to those oobsize which is larger than 512. | P | |<----------------------------------------------------------------->| | | | (Block Mark) | | P' | | | | |<--------------------------------------------------->| D | | O'| | |<--------->| |<->| V V V V V +---+--------------+-+--------------+-+--------------+-+----------+-+---+ | M | data |E| data |E| data |E| data |E| | +---+--------------+-+--------------+-+--------------+-+----------+-+---+ ^ ^ | O | |<---------------->| P : the page size for BCH module. E : The ECC strength. G : the length of Galois Field. N : The chunk count of per page. M : the metasize of per page. C : the ecc chunk size, aka the "data" above. P': the nand chip's page size. O : the nand chip's oob size. O': the free oob. Signed-off-by: Peng Fan Reviewed-by: Marek Vasut Tested-By: Tim Harvey --- Changes v2: none Changes v1: The previous patch is https://patchwork.ozlabs.org/patch/422757/ This version contains a minor change to the name from gf_len to galois_field. Also add Marek's Reviewed-by. arch/arm/include/asm/imx-common/regs-bch.h | 2 ++ drivers/mtd/nand/mxs_nand.c | 32 +++++++++++++++++++++--------- 2 files changed, 25 insertions(+), 9 deletions(-) diff --git a/arch/arm/include/asm/imx-common/regs-bch.h b/arch/arm/include/asm/imx-common/regs-bch.h index a33d341..5c47783 100644 --- a/arch/arm/include/asm/imx-common/regs-bch.h +++ b/arch/arm/include/asm/imx-common/regs-bch.h @@ -148,6 +148,7 @@ struct mxs_bch_regs { #define BCH_FLASHLAYOUT0_ECC0_ECC30 (0xf << 12) #define BCH_FLASHLAYOUT0_ECC0_ECC32 (0x10 << 12) #define BCH_FLASHLAYOUT0_GF13_0_GF14_1 (1 << 10) +#define BCH_FLASHLAYOUT0_GF13_0_GF14_1_OFFSET 10 #define BCH_FLASHLAYOUT0_DATA0_SIZE_MASK 0xfff #define BCH_FLASHLAYOUT0_DATA0_SIZE_OFFSET 0 @@ -178,6 +179,7 @@ struct mxs_bch_regs { #define BCH_FLASHLAYOUT1_ECCN_ECC30 (0xf << 12) #define BCH_FLASHLAYOUT1_ECCN_ECC32 (0x10 << 12) #define BCH_FLASHLAYOUT1_GF13_0_GF14_1 (1 << 10) +#define BCH_FLASHLAYOUT1_GF13_0_GF14_1_OFFSET 10 #define BCH_FLASHLAYOUT1_DATAN_SIZE_MASK 0xfff #define BCH_FLASHLAYOUT1_DATAN_SIZE_OFFSET 0 diff --git a/drivers/mtd/nand/mxs_nand.c b/drivers/mtd/nand/mxs_nand.c index 9c144a2..33ce817 100644 --- a/drivers/mtd/nand/mxs_nand.c +++ b/drivers/mtd/nand/mxs_nand.c @@ -68,6 +68,8 @@ struct mxs_nand_info { }; struct nand_ecclayout fake_ecc_layout; +static int chunk_data_size = MXS_NAND_CHUNK_DATA_CHUNK_SIZE; +static int galois_field = 13; /* * Cache management functions @@ -130,12 +132,12 @@ static void mxs_nand_return_dma_descs(struct mxs_nand_info *info) static uint32_t mxs_nand_ecc_chunk_cnt(uint32_t page_data_size) { - return page_data_size / MXS_NAND_CHUNK_DATA_CHUNK_SIZE; + return page_data_size / chunk_data_size; } static uint32_t mxs_nand_ecc_size_in_bits(uint32_t ecc_strength) { - return ecc_strength * MXS_NAND_BITS_PER_ECC_LEVEL; + return ecc_strength * galois_field; } static uint32_t mxs_nand_aux_status_offset(void) @@ -157,8 +159,8 @@ static inline uint32_t mxs_nand_get_ecc_strength(uint32_t page_data_size, * (page oob size - meta data size) * (bits per byte) */ ecc_strength = ((page_oob_size - MXS_NAND_METADATA_SIZE) * 8) - / (MXS_NAND_BITS_PER_ECC_LEVEL * - mxs_nand_ecc_chunk_cnt(page_data_size)); + / (galois_field * + mxs_nand_ecc_chunk_cnt(page_data_size)); return round_down(ecc_strength, 2); } @@ -173,7 +175,7 @@ static inline uint32_t mxs_nand_get_mark_offset(uint32_t page_data_size, uint32_t block_mark_chunk_bit_offset; uint32_t block_mark_bit_offset; - chunk_data_size_in_bits = MXS_NAND_CHUNK_DATA_CHUNK_SIZE * 8; + chunk_data_size_in_bits = chunk_data_size * 8; chunk_ecc_size_in_bits = mxs_nand_ecc_size_in_bits(ecc_strength); chunk_total_size_in_bits = @@ -972,6 +974,16 @@ static int mxs_nand_scan_bbt(struct mtd_info *mtd) struct mxs_bch_regs *bch_regs = (struct mxs_bch_regs *)MXS_BCH_BASE; uint32_t tmp; + if (mtd->oobsize > MXS_NAND_CHUNK_DATA_CHUNK_SIZE) { + galois_field = 14; + chunk_data_size = MXS_NAND_CHUNK_DATA_CHUNK_SIZE * 2; + } + + if (mtd->oobsize > chunk_data_size) { + printf("Not support the NAND chips whose oob size is larger then %d bytes!\n", chunk_data_size); + return -EINVAL; + } + /* Configure BCH and set NFC geometry */ mxs_reset_block(&bch_regs->hw_bch_ctrl_reg); @@ -981,16 +993,18 @@ static int mxs_nand_scan_bbt(struct mtd_info *mtd) tmp |= MXS_NAND_METADATA_SIZE << BCH_FLASHLAYOUT0_META_SIZE_OFFSET; tmp |= (mxs_nand_get_ecc_strength(mtd->writesize, mtd->oobsize) >> 1) << BCH_FLASHLAYOUT0_ECC0_OFFSET; - tmp |= MXS_NAND_CHUNK_DATA_CHUNK_SIZE - >> MXS_NAND_CHUNK_DATA_CHUNK_SIZE_SHIFT; + tmp |= chunk_data_size >> MXS_NAND_CHUNK_DATA_CHUNK_SIZE_SHIFT; + tmp |= (14 == galois_field ? 1 : 0) << + BCH_FLASHLAYOUT0_GF13_0_GF14_1_OFFSET; writel(tmp, &bch_regs->hw_bch_flash0layout0); tmp = (mtd->writesize + mtd->oobsize) << BCH_FLASHLAYOUT1_PAGE_SIZE_OFFSET; tmp |= (mxs_nand_get_ecc_strength(mtd->writesize, mtd->oobsize) >> 1) << BCH_FLASHLAYOUT1_ECCN_OFFSET; - tmp |= MXS_NAND_CHUNK_DATA_CHUNK_SIZE - >> MXS_NAND_CHUNK_DATA_CHUNK_SIZE_SHIFT; + tmp |= chunk_data_size >> MXS_NAND_CHUNK_DATA_CHUNK_SIZE_SHIFT; + tmp |= (14 == galois_field ? 1 : 0) << + BCH_FLASHLAYOUT1_GF13_0_GF14_1_OFFSET; writel(tmp, &bch_regs->hw_bch_flash0layout1); /* Set *all* chip selects to use layout 0 */