mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-12 05:48:40 +00:00
mtd: ixp4xx map: use resource_size
Use the resource_size inline function instead of manually calculating the resource size. This reduces the chance of introducing off-by-one errors. Signed-off-by: Tobias Klauser <tklauser@distanz.ch> Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com> Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
This commit is contained in:
parent
2af7c65399
commit
d6587feaf0
1 changed files with 3 additions and 3 deletions
|
@ -210,7 +210,7 @@ static int ixp4xx_flash_probe(struct platform_device *dev)
|
||||||
* not attempt to do a direct access on us.
|
* not attempt to do a direct access on us.
|
||||||
*/
|
*/
|
||||||
info->map.phys = NO_XIP;
|
info->map.phys = NO_XIP;
|
||||||
info->map.size = dev->resource->end - dev->resource->start + 1;
|
info->map.size = resource_size(dev->resource);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* We only support 16-bit accesses for now. If and when
|
* We only support 16-bit accesses for now. If and when
|
||||||
|
@ -224,7 +224,7 @@ static int ixp4xx_flash_probe(struct platform_device *dev)
|
||||||
info->map.copy_from = ixp4xx_copy_from,
|
info->map.copy_from = ixp4xx_copy_from,
|
||||||
|
|
||||||
info->res = request_mem_region(dev->resource->start,
|
info->res = request_mem_region(dev->resource->start,
|
||||||
dev->resource->end - dev->resource->start + 1,
|
resource_size(dev->resource),
|
||||||
"IXP4XXFlash");
|
"IXP4XXFlash");
|
||||||
if (!info->res) {
|
if (!info->res) {
|
||||||
printk(KERN_ERR "IXP4XXFlash: Could not reserve memory region\n");
|
printk(KERN_ERR "IXP4XXFlash: Could not reserve memory region\n");
|
||||||
|
@ -233,7 +233,7 @@ static int ixp4xx_flash_probe(struct platform_device *dev)
|
||||||
}
|
}
|
||||||
|
|
||||||
info->map.virt = ioremap(dev->resource->start,
|
info->map.virt = ioremap(dev->resource->start,
|
||||||
dev->resource->end - dev->resource->start + 1);
|
resource_size(dev->resource));
|
||||||
if (!info->map.virt) {
|
if (!info->map.virt) {
|
||||||
printk(KERN_ERR "IXP4XXFlash: Failed to ioremap region\n");
|
printk(KERN_ERR "IXP4XXFlash: Failed to ioremap region\n");
|
||||||
err = -EIO;
|
err = -EIO;
|
||||||
|
|
Loading…
Reference in a new issue