Commit ec98e9ab authored by Minghuan Lian's avatar Minghuan Lian Committed by Bjorn Helgaas

PCI: designware: Fix configuration base address when using 'reg'

The code has calculated cfg0_base and cfg1_base when parsing 'reg' or
'ranges' property of PCI DTS node, so remove duplicate calculation.  When
using 'reg', resource cfg is not used, so this code computed an incorrect
configuration base.
Signed-off-by: default avatarMinghuan Lian <Minghuan.Lian@freescale.com>
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
Acked-by: default avatarMohit KUMAR <mohit.kumar@st.com>
parent 9f0dbe08
...@@ -510,7 +510,6 @@ int __init dw_pcie_host_init(struct pcie_port *pp) ...@@ -510,7 +510,6 @@ int __init dw_pcie_host_init(struct pcie_port *pp)
pp->mem_base = pp->mem.start; pp->mem_base = pp->mem.start;
if (!pp->va_cfg0_base) { if (!pp->va_cfg0_base) {
pp->cfg0_base = pp->cfg.start;
pp->va_cfg0_base = devm_ioremap(pp->dev, pp->cfg0_base, pp->va_cfg0_base = devm_ioremap(pp->dev, pp->cfg0_base,
pp->cfg0_size); pp->cfg0_size);
if (!pp->va_cfg0_base) { if (!pp->va_cfg0_base) {
...@@ -520,7 +519,6 @@ int __init dw_pcie_host_init(struct pcie_port *pp) ...@@ -520,7 +519,6 @@ int __init dw_pcie_host_init(struct pcie_port *pp)
} }
if (!pp->va_cfg1_base) { if (!pp->va_cfg1_base) {
pp->cfg1_base = pp->cfg.start + pp->cfg0_size;
pp->va_cfg1_base = devm_ioremap(pp->dev, pp->cfg1_base, pp->va_cfg1_base = devm_ioremap(pp->dev, pp->cfg1_base,
pp->cfg1_size); pp->cfg1_size);
if (!pp->va_cfg1_base) { if (!pp->va_cfg1_base) {
......
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