From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by bombadil.infradead.org with esmtps (Exim 4.72 #1 (Red Hat Linux)) id 1OlcM7-0001gR-Lq for barebox@lists.infradead.org; Wed, 18 Aug 2010 06:42:36 +0000 Date: Wed, 18 Aug 2010 08:42:34 +0200 From: Sascha Hauer Message-ID: <20100818064233.GY27749@pengutronix.de> References: <1282035337-25094-1-git-send-email-m.grzeschik@pengutronix.de> <4C6A6709.9080107@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <4C6A6709.9080107@gmail.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH|RFC] beagle: make nand ecc command based To: Nishanth Menon Cc: barebox@lists.infradead.org On Tue, Aug 17, 2010 at 05:40:09AM -0500, Nishanth Menon wrote: > On 08/17/2010 03:55 AM, Michael Grzeschik wrote: >> Signed-off-by: Michael Grzeschik >> --- >> This will only work once and bring the nand chip into a undefined state >> after a second call. Any ideas for doing this save? > looking at the gpmc logic, it does a reset in gpmc_cs_config by > disabling and re-enabling it -> so my guess is: > a) in the selection of ecc logic > b) reset of statemachines in mtd layers > c) nand chip not being reset from it's previous state (resetting the > controller does not mean nand chip is reset) (if i recollect sometime > back mtd used to do a 0xff and reset).. > > > personally, IMHO using s/w ecc has not much benefit other than being > "legacy enabled" It really seems odd to me that the omap internal ROM code expects hw ecc while xloader and kernel expect hw ecc. This way we always need two different ecc algorithms in place which is really inconvenient and hard for users to get it right. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox