Re: [PATCH] [MTD] BLOCK_RO: Readonly Block Device Layer Over MTD

[Artem Bityutskiy <dedekind@infradead.org>]

On Mon, 2006-11-20 at 16:06 +0300, Vitaly Wool wrote:
> Artem Bityutskiy wrote:
> > I am talking about something like this:
> >
> > http://lists.infradead.org/pipermail/linux-mtd/2004-May/009683.html
> >   
> This one unconditionally changes mtdblock, doesn't it?

Yes.

Basically speaking, mtdblock was designed for NOR flashes and there were
no bad eraseblocks at all. Now you want to utilize it on NANDs. So you
add BB-awareness functionality. And I still think you should changd
mtdblock_ro instead of adding new driver.

Why? Because if you change mtdblock_ro:

1. you do not change semantics at all. in case of old BB-less flashes
all stays the same old way. So I do not understand why you need a new
device number.

2. You make mtdblock_ro workable on NAND. You extend it. You have right
to slightly change the semantics. 1:1 mapping makes no sense in this
case, so it is logical to skip bad blocks. Instead you add a new driver.
Strange.

-- 
Best regards,
Artem Bityutskiy (Битюцкий Артём)