PCI: altera: Remove redundant platform_get_resource() return value check
authorBjorn Helgaas <bhelgaas@google.com>
Thu, 6 Oct 2016 18:29:02 +0000 (13:29 -0500)
committerBjorn Helgaas <bhelgaas@google.com>
Wed, 12 Oct 2016 00:54:23 +0000 (19:54 -0500)
devm_ioremap_resource() fails gracefully when given a NULL resource
pointer, so we don't need to check separately for failure from
platform_get_resource_byname().  Remove the redundant check.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/host/pcie-altera.c

index b26695c..931c51e 100644 (file)
@@ -74,7 +74,7 @@
 
 struct altera_pcie {
        struct platform_device  *pdev;
-       void __iomem            *cra_base;
+       void __iomem            *cra_base;      /* DT Cra */
        int                     irq;
        u8                      root_bus_nr;
        struct irq_domain       *irq_domain;
@@ -555,11 +555,6 @@ static int altera_pcie_parse_dt(struct altera_pcie *pcie)
        struct resource *cra;
 
        cra = platform_get_resource_byname(pdev, IORESOURCE_MEM, "Cra");
-       if (!cra) {
-               dev_err(dev, "no Cra memory resource defined\n");
-               return -ENODEV;
-       }
-
        pcie->cra_base = devm_ioremap_resource(dev, cra);
        if (IS_ERR(pcie->cra_base)) {
                dev_err(dev, "failed to map cra memory\n");