From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Thu, 18 Mar 2010 17:46:46 -0500
Subject: [U-Boot] [PATH] at91: add hwecc method for nand
In-Reply-To: <e24af7501003172355q6d247de1l67a2bf68cb601814@mail.gmail.com>
References: <e24af7501003172355q6d247de1l67a2bf68cb601814@mail.gmail.com>
Message-ID: <20100318224646.GA28540@loki.buserror.net>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Thu, Mar 18, 2010 at 11:55:29AM +0500, Nikolay Petukhov wrote:
> +	/* move to ECC position if needed */
> +	if (eccpos[0] != 0) {
> +		/* This only works on large pages
> +		 * because the ECC controller waits for
> +		 * NAND_CMD_RNDOUTSTART after the
> +		 * NAND_CMD_RNDOUT.
> +		 * anyway, for small pages, the eccpos[0] == 0
> +		 */
> +		chip->cmdfunc(mtd, NAND_CMD_RNDOUT,
> +				mtd->writesize + eccpos[0], -1);
> +	}
[snip]
> +	/* get back to oob start (end of page) */
> +	chip->cmdfunc(mtd, NAND_CMD_RNDOUT, mtd->writesize, -1);

Is the second RNDOUT OK on small page NAND?

> +	//struct atmel_nand_host *host = nand_chip->priv;

Just remove the line.

-Scott