From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Thu, 16 Aug 2012 09:59:31 +0200 Subject: [U-Boot] [PATCH 08/17] ARM: SPL: Remove NAND_MODE_HW_ECC from spl_nand.c In-Reply-To: <1345066254-6911-9-git-send-email-trini@ti.com> References: <1345066254-6911-1-git-send-email-trini@ti.com> <1345066254-6911-9-git-send-email-trini@ti.com> Message-ID: <502CA863.1020206@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 15/08/2012 23:30, Tom Rini wrote: > This detection code doesn't (and can't) do anything currently, so > remove. > > Signed-off-by: Tom Rini > --- > arch/arm/cpu/armv7/omap3/board.c | 3 --- > arch/arm/include/asm/omap_common.h | 1 - > arch/arm/lib/spl_nand.c | 13 +++---------- > 3 files changed, 3 insertions(+), 14 deletions(-) > > diff --git a/arch/arm/cpu/armv7/omap3/board.c b/arch/arm/cpu/armv7/omap3/board.c > index 1640696..a487374 100644 > --- a/arch/arm/cpu/armv7/omap3/board.c > +++ b/arch/arm/cpu/armv7/omap3/board.c > @@ -77,9 +77,6 @@ u32 spl_boot_mode(void) > case BOOT_DEVICE_MMC1: > return MMCSD_MODE_FAT; > break; > - case BOOT_DEVICE_NAND: > - return NAND_MODE_HW_ECC; > - break; > default: > puts("spl: ERROR: unknown device - can't select boot mode\n"); > hang(); > diff --git a/arch/arm/include/asm/omap_common.h b/arch/arm/include/asm/omap_common.h > index 9ac0cc2..1e8f59b 100644 > --- a/arch/arm/include/asm/omap_common.h > +++ b/arch/arm/include/asm/omap_common.h > @@ -76,7 +76,6 @@ void preloader_console_init(void); > #define MMCSD_MODE_UNDEFINED 0 > #define MMCSD_MODE_RAW 1 > #define MMCSD_MODE_FAT 2 > -#define NAND_MODE_HW_ECC 3 > > struct spl_image_info { > const char *name; > diff --git a/arch/arm/lib/spl_nand.c b/arch/arm/lib/spl_nand.c > index ea7cc8a..39221d5 100644 > --- a/arch/arm/lib/spl_nand.c > +++ b/arch/arm/lib/spl_nand.c > @@ -35,16 +35,9 @@ void spl_nand_load_image(void) > int *src __attribute__((unused)); > int *dst __attribute__((unused)); > > - switch (spl_boot_mode()) { > - case NAND_MODE_HW_ECC: > - debug("spl: nand - using hw ecc\n"); > - gpmc_init(); > - nand_init(); > - break; > - default: > - puts("spl: ERROR: This bootmode is not implemented - hanging"); > - hang(); > - } > + debug("spl: nand - using hw ecc\n"); > + gpmc_init(); > + nand_init(); > This also does not work with other SOCs, where there is not a gpmc init. My idea was to add here a soc specific function, where gpmc_init and nand_init can be called. Regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================