Commit e3538f40 authored by Jaehoon Chung's avatar Jaehoon Chung Committed by Bjorn Helgaas

PCI: exynos: Remove unnecessary local variables

Remove unnecessary local variables: elbi_base, phy_base, block_base.  We
need one resource structure for assigning each resource.  Reuse the single
'res' variable for all.
Signed-off-by: default avatarJaehoon Chung <jh80.chung@samsung.com>
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
Reviewed-by: default avatarPankaj Dubey <pankaj.dubey@samsung.com>
Acked-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
Acked-by: default avatarJingoo Han <jingoohan1@gmail.com>
parent d6da7d90
...@@ -507,9 +507,7 @@ static int __init exynos_pcie_probe(struct platform_device *pdev) ...@@ -507,9 +507,7 @@ static int __init exynos_pcie_probe(struct platform_device *pdev)
struct exynos_pcie *ep; struct exynos_pcie *ep;
struct pcie_port *pp; struct pcie_port *pp;
struct device_node *np = dev->of_node; struct device_node *np = dev->of_node;
struct resource *elbi_base; struct resource *res;
struct resource *phy_base;
struct resource *block_base;
int ret; int ret;
ep = devm_kzalloc(dev, sizeof(*ep), GFP_KERNEL); ep = devm_kzalloc(dev, sizeof(*ep), GFP_KERNEL);
...@@ -540,22 +538,22 @@ static int __init exynos_pcie_probe(struct platform_device *pdev) ...@@ -540,22 +538,22 @@ static int __init exynos_pcie_probe(struct platform_device *pdev)
if (ret) if (ret)
goto fail_clk; goto fail_clk;
elbi_base = platform_get_resource(pdev, IORESOURCE_MEM, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
ep->elbi_base = devm_ioremap_resource(dev, elbi_base); ep->elbi_base = devm_ioremap_resource(dev, res);
if (IS_ERR(ep->elbi_base)) { if (IS_ERR(ep->elbi_base)) {
ret = PTR_ERR(ep->elbi_base); ret = PTR_ERR(ep->elbi_base);
goto fail_bus_clk; goto fail_bus_clk;
} }
phy_base = platform_get_resource(pdev, IORESOURCE_MEM, 1); res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
ep->phy_base = devm_ioremap_resource(dev, phy_base); ep->phy_base = devm_ioremap_resource(dev, res);
if (IS_ERR(ep->phy_base)) { if (IS_ERR(ep->phy_base)) {
ret = PTR_ERR(ep->phy_base); ret = PTR_ERR(ep->phy_base);
goto fail_bus_clk; goto fail_bus_clk;
} }
block_base = platform_get_resource(pdev, IORESOURCE_MEM, 2); res = platform_get_resource(pdev, IORESOURCE_MEM, 2);
ep->block_base = devm_ioremap_resource(dev, block_base); ep->block_base = devm_ioremap_resource(dev, res);
if (IS_ERR(ep->block_base)) { if (IS_ERR(ep->block_base)) {
ret = PTR_ERR(ep->block_base); ret = PTR_ERR(ep->block_base);
goto fail_bus_clk; goto fail_bus_clk;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment