[PATCH] mtd: nand: Rename create_bbt()'s 'len' variable to 'numpages'

Shmulik Ladkani shmulik.ladkani at gmail.com
Sat Aug 25 04:32:29 EDT 2012


On Fri, 24 Aug 2012 17:18:22 -0700 Brian Norris <computersforpeace at gmail.com> wrote:
> On Mon, Aug 20, 2012 at 6:29 AM, Shmulik Ladkani
> Thanks, this kind of fix is actually very helpful! It's kinda annoying
> that I always have to translate in my head the fact that 'len' means
> 'number of pages.'

Exactly.

It was triggered from reviewing some of the latest bbt patches; I was
summarizing on a _paper_: "len is numpages, scanlen is ... readlen is ..."
and I thought to myself, isn't this crazy?

So yes, naming is important ;-)

(BTW scanlen and readlen are annoying, too... but they don't have a
concise meaning)

Regards,
Shmulik



More information about the linux-mtd mailing list