Bug in physmap driver?

Lennert Buytenhek buytenh at wantstofly.org
Thu Apr 16 11:06:30 EDT 2009


On Thu, Apr 16, 2009 at 04:00:19PM +0100, David Howells wrote:

> In physmap_flash_probe(), isn't the '+ 1' in:
> 
> 		info->map[i].size = dev->resource[i].end - dev->resource[i].start + 1;
> 
> incorrect?  It results in an artificially inflated size:
> 
> (gdb) p info->map[0]
> $7 = {name = 0xc08aa520 "physmap-flash.0", size = 0x200001,

That would suggest that dev->resource[i].end is off-by-one?



More information about the linux-mtd mailing list