From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Fri, 06 Dec 2013 08:00:41 +0100 Subject: [U-Boot] [PATCH v9 0/4] mtd: nand: omap: optimize OMAP NAND driver In-Reply-To: <20980858CB6D3A4BAE95CA194937D5E73EA52408@DBDE04.ent.ti.com> References: <1384839138-10885-1-git-send-email-pekon@ti.com> <20980858CB6D3A4BAE95CA194937D5E73EA52408@DBDE04.ent.ti.com> Message-ID: <52A17619.7070801@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 06.12.2013 07:22, Gupta, Pekon wrote: >> From: Gupta, Pekon >> Subject: [PATCH v9 0/4] mtd: nand: omap: optimize OMAP NAND driver >> >> OMAP NAND drivers support various ecc-schemes like (HAM1, BCH4, BCH8..) >> And, each ecc-scheme usually has 2 implementations >> - HAM1_HW or BCHx_HW (using in-build h/w engine for all computation) >> - HAM1_SW or BCHx_HW_DETECTION_SW) (using software library for some computation) >> >> This patch series is Part-2 (split) from earlier series, as mentioned below: >> http://lists.denx.de/pipermail/u-boot/2013-November/167270.html >> >> This series aims at following: >> - cleans-up redundant code, duplicate in different ecc-schemes >> - optimizes ecc calculation and correction paths >> - improves scalability and readability, for adding newer ecc-schemes in future. >> - updates following callbacks for different ecc-schmes: >> [PATCH 1/4] nand_chip->ecc.hwctl(): configures h/w controller before for Read/Write acccess >> [PATCH 2/4] nand_chip->ecc.calculate(): calculates and fetches ECC syndrome from h/w controller. >> [PATCH 3/4] nand_chip->ecc.correct(): detects and corrects ECC errors on read access >> [PATCH 4/4] update doc and README >> >> >> *changes in v9* >> - split from main series, rebased and forked independently >> > > Request you to please review this series. And accept it if it looks okay. > > And a bug-fix which causes regression for x8 NAND devices > using HAM1 ecc-scheme. This was reported & (un-officially) > tested by Stefan Roese . I tested this patch series plus the fix from Pekon on an x8 and x16 OMAP3530 board. Both without any problems. So for this series: Tested-by: Stefan Roese Thanks, Stefan