From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Thu, 22 Aug 2013 17:25:23 -0500 Subject: [U-Boot] [PATCH v2 1/4] mtd: nand: omap: enable BCH ECC scheme using ELM for generic platform In-Reply-To: <1376461015-29007-2-git-send-email-pekon@ti.com> References: <1376461015-29007-1-git-send-email-pekon@ti.com> <1376461015-29007-2-git-send-email-pekon@ti.com> Message-ID: <1377210323.20722.26.camel@snotra.buserror.net> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, 2013-08-14 at 11:46 +0530, Pekon Gupta wrote: > BCH8_ECC scheme implemented in omap_gpmc.c driver has following two favours > +-----------------------------------+-----------------+-----------------+ > |ECC Scheme | ECC Calculation | Error Detection | > +-----------------------------------+-----------------+-----------------+ > |OMAP_ECC_BCH8_CODE_HW |GPMC |ELM H/W engine | > |OMAP_ECC_BCH8_CODE_HW_DETECTION_SW |GPMC |S/W BCH library | > +-----------------------------------+-----------------+-----------------+ > > Current implementation enables of BCH8_CODE_HW only for AM33xx SoC family. > (using CONFIG_AM33XX). However, other SoC families (like TI81xx) also have > ELM hardware module, and can support ECC error detection using ELM. > > This patch > - replaces CONFIG_AM33xx define with > CONFIG_NAND_OMAP_ECC_BCH8_CODE_HW > so that all device families having required h/w capability can use ELM for > error detection in ECC_BCHx schemes. > > - replaces CONFIG_NAND_OMAP_BCH8 with > CONFIG_NAND_OMAP_ECC_BCH8_CODE_HW_DETECTION_SW && CONFIG_BCH > and separates out code for above mentioned BCH8_ECC implementations so that > driver can be build independently using anyone of them. > CONFIG_BCH is used to enable software BCH library in lib/bch.c > > Signed-off-by: Pekon Gupta > --- > doc/README.nand | 20 +++++++ > drivers/mtd/nand/omap_gpmc.c | 128 ++++++++++++++++++++++++------------------- > include/configs/am335x_evm.h | 1 + > include/configs/ti814x_evm.h | 2 +- > include/configs/tricorder.h | 2 +- > 5 files changed, 96 insertions(+), 57 deletions(-) The ti814x_evm.h changes do not apply. What tree is this against? Can someone familiar with the OMAP driver review this patchset? -Scott