forked from luck/tmp_suning_uos_patched
hsi: controllers:remove redundant code
Use devm_ioremap_resource() in order to make the code simpler, and remove redundant return value check of platform_get_resource() because this value is alreadytaken care by devm_ioremap_resource() Signed-off-by: Sanjeev Sharma <sanjeev_sharma@mentor.com> Signed-off-by: Sebastian Reichel <sre@kernel.org>
This commit is contained in:
parent
d2c85ac24e
commit
16bd5865cd
|
@ -295,27 +295,14 @@ static int __init ssi_get_iomem(struct platform_device *pd,
|
|||
const char *name, void __iomem **pbase, dma_addr_t *phy)
|
||||
{
|
||||
struct resource *mem;
|
||||
struct resource *ioarea;
|
||||
void __iomem *base;
|
||||
struct hsi_controller *ssi = platform_get_drvdata(pd);
|
||||
|
||||
mem = platform_get_resource_byname(pd, IORESOURCE_MEM, name);
|
||||
if (!mem) {
|
||||
dev_err(&pd->dev, "IO memory region missing (%s)\n", name);
|
||||
return -ENXIO;
|
||||
}
|
||||
ioarea = devm_request_mem_region(&ssi->device, mem->start,
|
||||
resource_size(mem), dev_name(&pd->dev));
|
||||
if (!ioarea) {
|
||||
dev_err(&pd->dev, "%s IO memory region request failed\n",
|
||||
mem->name);
|
||||
return -ENXIO;
|
||||
}
|
||||
base = devm_ioremap(&ssi->device, mem->start, resource_size(mem));
|
||||
if (!base) {
|
||||
dev_err(&pd->dev, "%s IO remap failed\n", mem->name);
|
||||
return -ENXIO;
|
||||
}
|
||||
base = devm_ioremap_resource(&ssi->device, mem);
|
||||
if (IS_ERR(base))
|
||||
return PTR_ERR(base);
|
||||
|
||||
*pbase = base;
|
||||
|
||||
if (phy)
|
||||
|
|
Loading…
Reference in New Issue
Block a user