diff options
author | Bjorn Helgaas <bhelgaas@google.com> | 2016-05-31 13:14:17 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2016-06-20 15:06:06 -0400 |
commit | 74462284bdda85216c3cf49d52920d4164f47e80 (patch) | |
tree | 7ad123346ba6763f6dd0d1a04839cdd8b2b7761d | |
parent | 9061f9bea4c995cea06c638606d1d56f95a759e0 (diff) |
PCI: altera: Request host bridge window resources with core function
Use devm_request_pci_bus_resources() to request host bridge window
resources instead of doing it by hand in the driver.
No functional change intended.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
-rw-r--r-- | drivers/pci/host/pcie-altera.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/pci/host/pcie-altera.c b/drivers/pci/host/pcie-altera.c index dbac6fb3f0bd..b97abbcdba33 100644 --- a/drivers/pci/host/pcie-altera.c +++ b/drivers/pci/host/pcie-altera.c | |||
@@ -432,21 +432,20 @@ static int altera_pcie_parse_request_of_pci_ranges(struct altera_pcie *pcie) | |||
432 | if (err) | 432 | if (err) |
433 | return err; | 433 | return err; |
434 | 434 | ||
435 | err = devm_request_pci_bus_resources(dev, &pcie->resources); | ||
436 | if (err) | ||
437 | goto out_release_res; | ||
438 | |||
435 | resource_list_for_each_entry(win, &pcie->resources) { | 439 | resource_list_for_each_entry(win, &pcie->resources) { |
436 | struct resource *parent, *res = win->res; | 440 | struct resource *res = win->res; |
437 | 441 | ||
438 | switch (resource_type(res)) { | 442 | switch (resource_type(res)) { |
439 | case IORESOURCE_MEM: | 443 | case IORESOURCE_MEM: |
440 | parent = &iomem_resource; | ||
441 | res_valid |= !(res->flags & IORESOURCE_PREFETCH); | 444 | res_valid |= !(res->flags & IORESOURCE_PREFETCH); |
442 | break; | 445 | break; |
443 | default: | 446 | default: |
444 | continue; | 447 | continue; |
445 | } | 448 | } |
446 | |||
447 | err = devm_request_resource(dev, parent, res); | ||
448 | if (err) | ||
449 | goto out_release_res; | ||
450 | } | 449 | } |
451 | 450 | ||
452 | if (!res_valid) { | 451 | if (!res_valid) { |