[PATCH] [MTD] devices/mtd_dataflash: Add SW sector protection support

Mike McTernan mmcternan at airvana.com
Tue Mar 31 18:08:26 EDT 2009


On Monday 23 March 2009 20:36, David Brownell wrote:

> I took a really quick look at this. 

Many thanks.

> The terminology is confusing ... there are now two kinds 
> of "sector" in the driver.  Call one of them "lock sector"
> or somesuch,

It's as per the datasheet figure 4-1; the device has pages, blocks and
sectors.  Blocks aren't very interesting so are not in the driver, but
pages and sectors are.  But I see only one use of sectors in this
driver, what am I missing?

> so it makes sense to check the count of such sectors

Could you clarify what you mean here?

> stylistically, it'd be better not to #ifdef the guts of
> the lock/unlock methods.  #define them as NULL when
> SWPROT is not configured, and don't even set those
> methods for devices that don't support those ops.

That's easily changed.

Kind Regards,

Mike




More information about the linux-mtd mailing list