From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Date: Thu, 6 Dec 2018 15:42:17 +0100 Subject: [U-Boot] [PATCH v1] mtd: nand: raw: allow to disable unneeded ECC layouts In-Reply-To: <20181206135709.8518-1-stefan@agner.ch> References: <20181206135709.8518-1-stefan@agner.ch> Message-ID: <20181206154217.5c0f543b@jawa> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, 6 Dec 2018 14:57:09 +0100 Stefan Agner wrote: > From: Stefan Agner > > Each ECC layout consumes about 2984 bytes in the .data section. Allow > to disable the default ECC layouts if a driver is known to provide its > own ECC layout. > > Signed-off-by: Stefan Agner > --- > > drivers/mtd/nand/raw/Kconfig | 7 +++++++ > drivers/mtd/nand/raw/nand_base.c | 4 ++++ > 2 files changed, 11 insertions(+) > > diff --git a/drivers/mtd/nand/raw/Kconfig > b/drivers/mtd/nand/raw/Kconfig index 008f7b4b4b..b06c45788a 100644 > --- a/drivers/mtd/nand/raw/Kconfig > +++ b/drivers/mtd/nand/raw/Kconfig > @@ -9,6 +9,12 @@ config SYS_NAND_SELF_INIT > This option, if enabled, provides more flexible and > linux-like NAND initialization process. > > +config SYS_NAND_DRIVER_ECC_LAYOUT > + bool > + help > + Omit standard ECC layouts to safe space. Select this if > your driver > + is known to provide its own ECC layout. > + > config NAND_ATMEL > bool "Support Atmel NAND controller" > imply SYS_NAND_USE_FLASH_BBT > @@ -81,6 +87,7 @@ config NAND_OMAP_ELM > config NAND_VF610_NFC > bool "Support for Freescale NFC for VF610" > select SYS_NAND_SELF_INIT > + select SYS_NAND_DRIVER_ECC_LAYOUT > imply CMD_NAND > help > Enables support for NAND Flash Controller on some Freescale > diff --git a/drivers/mtd/nand/raw/nand_base.c > b/drivers/mtd/nand/raw/nand_base.c index 92daebe120..6d2ff58d86 100644 > --- a/drivers/mtd/nand/raw/nand_base.c > +++ b/drivers/mtd/nand/raw/nand_base.c > @@ -47,6 +47,7 @@ > #include > > /* Define default oob placement schemes for large and small page > devices */ +#ifdef CONFIG_SYS_NAND_DRIVER_ECC_LAYOUT > static struct nand_ecclayout nand_oob_8 = { > .eccbytes = 3, > .eccpos = {0, 1, 2}, > @@ -89,6 +90,7 @@ static struct nand_ecclayout nand_oob_128 = { > {.offset = 2, > .length = 78} } > }; > +#endif > > static int nand_get_device(struct mtd_info *mtd, int new_state); > > @@ -4339,6 +4341,7 @@ int nand_scan_tail(struct mtd_info *mtd) > */ > if (!ecc->layout && (ecc->mode != NAND_ECC_SOFT_BCH)) { > switch (mtd->oobsize) { > +#ifdef CONFIG_SYS_NAND_DRIVER_ECC_LAYOUT > case 8: > ecc->layout = &nand_oob_8; > break; > @@ -4351,6 +4354,7 @@ int nand_scan_tail(struct mtd_info *mtd) > case 128: > ecc->layout = &nand_oob_128; > break; > +#endif > default: > pr_warn("No oob scheme defined for oobsize > %d\n", mtd->oobsize); Reviewed-by: Lukasz Majewski Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: