From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michele De Candia (VT) Date: Wed, 17 Jun 2009 18:17:40 +0200 Subject: [U-Boot] PATCH: bugfix for nand erase failure with bad blocks In-Reply-To: <20090617155421.GB6333@loki.buserror.net> References: <4A3798C4.8000303@valueteam.com> <20090616181035.52228832E416@gemini.denx.de> <4A37F7BF.2090101@valueteam.com> <20090616200940.DD093832E416@gemini.denx.de> <4A37FE47.3030203@freescale.com> <4A3898B5.8060309@valueteam.com> <4A389EE9.1000005@valueteam.com> <20090617155421.GB6333@loki.buserror.net> Message-ID: <4A391724.2080207@valueteam.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Scott Wood wrote: > On Wed, Jun 17, 2009 at 09:44:41AM +0200, Michele De Candia (VT) wrote: > >> Moreover, I think that if you want to erase a specific NAND area, the >> correct way to use 'nand erase' command would be: >> >> 'nand erase start end' >> >> If you want to erase an area but you want to be sure that 'size' bytes >> were erased, you should use: >> >> 'nand erase off size' >> > > How would the "nand erase" command reliably distinguish between the two alternatives? > > What we could do is extend the "plus" semantics (which currently allow > rounding the size up to a block boundary) so that if you have a plus sign > before the size it is interpreted the same as read/write. > As you has suggested we could use: 'nand erase start end' and 'nand erase off +size' > I'm a little uneasy about changing the normal erase command from size to end > -- it would break existing uses. Though, it would make it consistent with > the NOR erase command. Perhaps a period where it warns but accepts anyway a > size, if the second parameter is less than the first. > This doesn't work always: for example, when you erase at the NAND begin, second parameter could be greater than first one. It can always warn user when he uses the first erase way. > -Scott > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot >