diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2024-03-26 11:07:45 +0100 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2024-04-04 08:14:05 +0200 |
commit | 4c3116e852873de659203f04fcf3aabffc2f9aaa (patch) | |
tree | 5b3cad8d943de518be6739e0552244fa0fe83d4d | |
parent | 32145911666d8f39a1ede28b6f0cd12468a140c0 (diff) | |
download | barebox-4c3116e852873de659203f04fcf3aabffc2f9aaa.tar.gz barebox-4c3116e852873de659203f04fcf3aabffc2f9aaa.tar.xz |
pci: pcie-designware: iterate over windows
The ranges property has already been parsed in pci_controller_init(), so
instead of iterating over the ranges again, use the result stored in
the resource_entry list to initialize the controller base addresses.
Link: https://lore.barebox.org/20240326100746.471532-16-s.hauer@pengutronix.de
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
-rw-r--r-- | drivers/pci/pcie-designware-host.c | 27 |
1 files changed, 13 insertions, 14 deletions
diff --git a/drivers/pci/pcie-designware-host.c b/drivers/pci/pcie-designware-host.c index 87842bf4ee..cd1d9433aa 100644 --- a/drivers/pci/pcie-designware-host.c +++ b/drivers/pci/pcie-designware-host.c @@ -72,9 +72,9 @@ int __init dw_pcie_host_init(struct pcie_port *pp) struct dw_pcie *pci = to_dw_pcie_from_pp(pp); struct device *dev = pci->dev; struct device_node *np = dev->of_node; - struct of_pci_range range; struct of_pci_range_parser parser; struct resource *cfg_res; + struct resource_entry *window; int ret; pp->pci.parent = dev; @@ -96,19 +96,18 @@ int __init dw_pcie_host_init(struct pcie_port *pp) return -EINVAL; } - /* Get the I/O and memory ranges from DT */ - for_each_of_pci_range(&parser, &range) { - unsigned long restype = range.flags & IORESOURCE_TYPE_BITS; - - if (restype == IORESOURCE_IO) { - pp->io_size = range.size; - pp->io_bus_addr = range.pci_addr; - pp->io_base = range.cpu_addr; - } - if (restype == IORESOURCE_MEM) { - pp->mem_size = range.size; - pp->mem_bus_addr = range.pci_addr; - pp->mem_base = range.cpu_addr; + resource_list_for_each_entry(window, &pp->pci.windows) { + switch (resource_type(window->res)) { + case IORESOURCE_IO: + pp->io_size = resource_size(window->res); + pp->io_bus_addr = window->res->start - window->offset; + pp->io_base = window->res->start; + break; + case IORESOURCE_MEM: + pp->mem_size = resource_size(window->res); + pp->mem_bus_addr = window->res->start - window->offset; + pp->mem_base = window->res->start; + break; } } |