From: Michele De Candia (VT) <michele.decandia@valueteam.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] PATCH: bugfix for nand erase failure with bad blocks
Date: Wed, 17 Jun 2009 18:17:40 +0200 [thread overview]
Message-ID: <4A391724.2080207@valueteam.com> (raw)
In-Reply-To: <20090617155421.GB6333@loki.buserror.net>
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
>
next prev parent reply other threads:[~2009-06-17 16:17 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-16 13:06 [U-Boot] PATCH: bugfix for nand erase failure with bad blocks Michele De Candia
2009-06-16 18:10 ` Wolfgang Denk
2009-06-16 19:51 ` Michele De Candia
2009-06-16 20:09 ` Wolfgang Denk
2009-06-16 20:19 ` Scott Wood
2009-06-17 7:18 ` Michele De Candia
2009-06-17 7:43 ` Michele De Candia
2009-06-17 7:44 ` Michele De Candia
2009-06-17 15:54 ` Scott Wood
2009-06-17 16:17 ` Michele De Candia [this message]
2009-06-17 22:04 ` Scott Wood
2009-06-17 22:15 ` Wolfgang Denk
2009-06-17 22:34 ` Scott Wood
2009-06-19 7:01 ` Michele De Candia
2009-06-17 22:11 ` Wolfgang Denk
2009-06-17 9:18 ` Wolfgang Denk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A391724.2080207@valueteam.com \
--to=michele.decandia@valueteam.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.