mfd: asic3: use resource_size macro instead of local variable
This should make the code a little bit easier to read. Signed-off-by: Philipp Zabel <philipp.zabel@gmail.com> Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
This commit is contained in:
parent
01906d6d78
commit
be584bd5a4
|
@ -623,7 +623,6 @@ static int __init asic3_probe(struct platform_device *pdev)
|
||||||
struct asic3 *asic;
|
struct asic3 *asic;
|
||||||
struct resource *mem;
|
struct resource *mem;
|
||||||
unsigned long clksel;
|
unsigned long clksel;
|
||||||
int map_size;
|
|
||||||
int ret = 0;
|
int ret = 0;
|
||||||
|
|
||||||
asic = kzalloc(sizeof(struct asic3), GFP_KERNEL);
|
asic = kzalloc(sizeof(struct asic3), GFP_KERNEL);
|
||||||
|
@ -643,8 +642,7 @@ static int __init asic3_probe(struct platform_device *pdev)
|
||||||
goto out_free;
|
goto out_free;
|
||||||
}
|
}
|
||||||
|
|
||||||
map_size = mem->end - mem->start + 1;
|
asic->mapping = ioremap(mem->start, resource_size(mem));
|
||||||
asic->mapping = ioremap(mem->start, map_size);
|
|
||||||
if (!asic->mapping) {
|
if (!asic->mapping) {
|
||||||
ret = -ENOMEM;
|
ret = -ENOMEM;
|
||||||
dev_err(asic->dev, "Couldn't ioremap\n");
|
dev_err(asic->dev, "Couldn't ioremap\n");
|
||||||
|
@ -654,7 +652,7 @@ static int __init asic3_probe(struct platform_device *pdev)
|
||||||
asic->irq_base = pdata->irq_base;
|
asic->irq_base = pdata->irq_base;
|
||||||
|
|
||||||
/* calculate bus shift from mem resource */
|
/* calculate bus shift from mem resource */
|
||||||
asic->bus_shift = 2 - (map_size >> 12);
|
asic->bus_shift = 2 - (resource_size(mem) >> 12);
|
||||||
|
|
||||||
clksel = 0;
|
clksel = 0;
|
||||||
asic3_write_register(asic, ASIC3_OFFSET(CLOCK, SEL), clksel);
|
asic3_write_register(asic, ASIC3_OFFSET(CLOCK, SEL), clksel);
|
||||||
|
|
Loading…
Reference in New Issue