[PATCH] [MTD] UBI: Fix counting of ec value
Artem Bityutskiy
dedekind at infradead.org
Mon Feb 12 06:38:10 EST 2007
On Mon, 2007-02-12 at 11:48 +0100, Timo Lindhorst wrote:
> > So why have you moved this memory allocation here?
> Actually I have not moved up the allocation, but moved down the
> 'if (unlikely(ec > UBI_MAX_ERASECOUNTER)) { ... ' block, since we need
> to know the new ec value before we can do this test.
OK, fine, just free the memory there.
> > What's the point in the new 'ret' variable? Why ec += err does not work?
> I just thought adding an error code to a counter seems odd. Do you
> prefer this way?
Well, it's anyway better then a distinct variable IMO. Moreover, it is
used like this across the UBI code, so just assume it is UBI style :-)
--
Best regards,
Artem Bityutskiy (Битюцкий Артём)
More information about the linux-mtd
mailing list