From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Mon, 13 Feb 2012 17:05:34 -0600 Subject: [U-Boot] [PATCH] Tricorder: Fix boot after making software ecc configurable In-Reply-To: References: <1329123412-16339-1-git-send-email-weber@corscience.de> <4F398422.20406@freescale.com> Message-ID: <4F39973E.1040108@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 02/13/2012 04:14 PM, Tom Rini wrote: > On Mon, Feb 13, 2012 at 2:44 PM, Scott Wood wrote: >> On 02/13/2012 02:56 AM, Thomas Weber wrote: >>> The tricorder stops booting with the following message: >>> >>> U-Boot 2011.12-00325-ga661206 (Feb 13 2012 - 09:29:50) >>> >>> OMAP3503-GP ES3.1, CPU-OPP2, L3-165MHz, Max CPU Clock 600 mHz >>> OMAP3 Tricorder + LPDDR/NAND >>> I2C: ready >>> DRAM: 128 MiB >>> NAND: CONFIG_MTD_ECC_SOFT not enabled >>> 0 MiB >>> MMC: OMAP SD/MMC: 0 >>> >>> Adding CONFIG_MTD_ECC_SOFT to config file fixes this problem. >>> >>> Signed-off-by: Thomas Weber >>> --- >>> include/configs/tricorder.h | 1 + >>> 1 files changed, 1 insertions(+), 0 deletions(-) >> >> Sorry about that -- I meant to exclude "nand: make 1-bit software ECC >> configurable" from the patchset I applied, due to the potential for this >> sort of breakage. >> >> We should revert that patch until the issues raised here are resolved: >> http://patchwork.ozlabs.org/patch/119149/ > > I had / have plan to take the tricorder patch tomorrow, unless you > really want to push a revert from your tree instead. Either way is > fine with me. This affects more than just tricorder -- any board that depends on soft ECC will be broken, since none of them define this symbol. I thought I had left this patch out, but apparently didn't. -Scott