[PATCH] nanddump: dump rigth length when jumping bad blocks

Richard Genoud richard.genoud at gmail.com
Thu Jun 19 08:02:59 UTC 2014


2014-06-18 21:53 GMT+02:00 Baruch Siach <baruch at tkos.co.il>:
> Hi Richard,
>
> On Wed, Jun 18, 2014 at 04:59:49PM +0200, Richard Genoud wrote:
>> 2014-06-18 16:08 GMT+02:00 Baruch Siach <baruch at tkos.co.il>:
>> > On Wed, Jun 18, 2014 at 03:44:30PM +0200, Richard Genoud wrote:
>> >> With the "skip bad block" option, nanddump is not dumping the right
>> >> length if there's bad blocks.
>> >>
>> >> This patch moves the end read address each time a bad block is
>> >> encountered.
>> >
>> > This changes nanddump behaviour from upstream historical --omitbad (-b;
>> > removed in upstream d8b8f780ec3c), to existing skipbad, for which upstream
>> > uses the --bb=skipbad command line parameter. To avoid confusion, we should
>> > adapt to upstream command line parameter names, IMO.
>>
>> yeah, you are right, I verified with upstream nanddump, and it doesn't
>> update the length on bad block.
>> so, please ignore this patch.
>
> I'm have no objection to this change. It matches the change made in upstream
> mtd-utils code (commits 5dfb5a6adc7311 and d8b8f780ec3c916). I just think that
> we should also change the command line parameters name to match (from -b to
> --bb=skipbad; see mtd-utils commit c2c3cb928935c).

ok, I see.
I'll come back with a new patch matching mtd-utils' 1.5.1 nanddump
behaviour --bb=skipbad.

Thanks !


More information about the busybox mailing list