pci: pcie_dw_ti: migrate to common Designware PCIe functions
authorNeil Armstrong <narmstrong@baylibre.com>
Thu, 25 Mar 2021 14:49:19 +0000 (15:49 +0100)
committerBin Meng <bmeng.cn@gmail.com>
Thu, 15 Apr 2021 02:43:17 +0000 (10:43 +0800)
Migrate the dw_ti driver to use the common DW PCIe helpers.

Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
drivers/pci/Kconfig
drivers/pci/pcie_dw_ti.c

index ab5a5e7..318d8fa 100644 (file)
@@ -264,7 +264,7 @@ config PCIE_DW_COMMON
 
 config PCI_KEYSTONE
        bool "TI Keystone PCIe controller"
-       depends on DM_PCI
+       select PCIE_DW_COMMON
        help
          Say Y here if you want to enable PCI controller support on AM654 SoC.
 
index 33a5c3c..4195a02 100644 (file)
 #include <linux/delay.h>
 #include <linux/err.h>
 
+#include "pcie_dw_common.h"
+
 DECLARE_GLOBAL_DATA_PTR;
 
 #define PCIE_VENDORID_MASK     GENMASK(15, 0)
 #define PCIE_DEVICEID_SHIFT    16
 
-/* PCI DBICS registers */
-#define PCIE_CONFIG_BAR0               0x10
-#define PCIE_LINK_STATUS_REG           0x80
-#define PCIE_LINK_STATUS_SPEED_OFF     16
-#define PCIE_LINK_STATUS_SPEED_MASK    (0xf << PCIE_LINK_STATUS_SPEED_OFF)
-#define PCIE_LINK_STATUS_WIDTH_OFF     20
-#define PCIE_LINK_STATUS_WIDTH_MASK    (0xf << PCIE_LINK_STATUS_WIDTH_OFF)
-
 #define PCIE_LINK_CAPABILITY           0x7c
 #define PCIE_LINK_CTL_2                        0xa0
 #define TARGET_LINK_SPEED_MASK         0xf
@@ -47,46 +41,12 @@ DECLARE_GLOBAL_DATA_PTR;
 #define PORT_LOGIC_LTSSM_STATE_MASK    0x1f
 #define PORT_LOGIC_LTSSM_STATE_L0      0x11
 
-#define PCIE_LINK_WIDTH_SPEED_CONTROL  0x80c
-#define PORT_LOGIC_SPEED_CHANGE                (0x1 << 17)
-
 #define PCIE_LINK_UP_TIMEOUT_MS                100
 
-/*
- * iATU Unroll-specific register definitions
- * From 4.80 core version the address translation will be made by unroll.
- * The registers are offset from atu_base
- */
-#define PCIE_ATU_UNR_REGION_CTRL1      0x00
-#define PCIE_ATU_UNR_REGION_CTRL2      0x04
-#define PCIE_ATU_UNR_LOWER_BASE                0x08
-#define PCIE_ATU_UNR_UPPER_BASE                0x0c
-#define PCIE_ATU_UNR_LIMIT             0x10
-#define PCIE_ATU_UNR_LOWER_TARGET      0x14
-#define PCIE_ATU_UNR_UPPER_TARGET      0x18
-
-#define PCIE_ATU_REGION_INDEX1         (0x1 << 0)
-#define PCIE_ATU_REGION_INDEX0         (0x0 << 0)
-#define PCIE_ATU_TYPE_MEM              (0x0 << 0)
-#define PCIE_ATU_TYPE_IO               (0x2 << 0)
-#define PCIE_ATU_TYPE_CFG0             (0x4 << 0)
-#define PCIE_ATU_TYPE_CFG1             (0x5 << 0)
-#define PCIE_ATU_ENABLE                        (0x1 << 31)
-#define PCIE_ATU_BAR_MODE_ENABLE       (0x1 << 30)
-#define PCIE_ATU_BUS(x)                        (((x) & 0xff) << 24)
-#define PCIE_ATU_DEV(x)                        (((x) & 0x1f) << 19)
-#define PCIE_ATU_FUNC(x)               (((x) & 0x7) << 16)
-
-/* Register address builder */
-#define PCIE_GET_ATU_OUTB_UNR_REG_OFFSET(region)       ((region) << 9)
-
 /* Offsets from App base */
 #define PCIE_CMD_STATUS                        0x04
 #define LTSSM_EN_VAL                   BIT(0)
 
-/* Parameters for the waiting for iATU enabled routine */
-#define LINK_WAIT_MAX_IATU_RETRIES     5
-#define LINK_WAIT_IATU                 10000
 
 #define AM654_PCIE_DEV_TYPE_MASK       0x3
 #define EP                             0x0
@@ -96,29 +56,13 @@ DECLARE_GLOBAL_DATA_PTR;
 /**
  * struct pcie_dw_ti - TI DW PCIe controller state
  *
+ * @pci: The common PCIe DW structure
  * @app_base: The base address of application register space
- * @dbics_base: The base address of dbics register space
- * @cfg_base: The base address of configuration space
- * @atu_base: The base address of ATU space
- * @cfg_size: The size of the configuration space which is needed
- *            as it gets written into the PCIE_ATU_LIMIT register
- * @first_busno: This driver supports multiple PCIe controllers.
- *               first_busno stores the bus number of the PCIe root-port
- *               number which may vary depending on the PCIe setup
- *               (PEX switches etc).
  */
 struct pcie_dw_ti {
+       /* Must be first member of the struct */
+       struct pcie_dw dw;
        void *app_base;
-       void *dbi_base;
-       void *cfg_base;
-       void *atu_base;
-       fdt_size_t cfg_size;
-       int first_busno;
-       struct udevice *dev;
-
-       /* IO and MEM PCI regions */
-       struct pci_region io;
-       struct pci_region mem;
 };
 
 enum dw_pcie_device_mode {
@@ -128,261 +72,6 @@ enum dw_pcie_device_mode {
        DW_PCIE_RC_TYPE,
 };
 
-static int pcie_dw_get_link_speed(struct pcie_dw_ti *pci)
-{
-       return (readl(pci->dbi_base + PCIE_LINK_STATUS_REG) &
-               PCIE_LINK_STATUS_SPEED_MASK) >> PCIE_LINK_STATUS_SPEED_OFF;
-}
-
-static int pcie_dw_get_link_width(struct pcie_dw_ti *pci)
-{
-       return (readl(pci->dbi_base + PCIE_LINK_STATUS_REG) &
-               PCIE_LINK_STATUS_WIDTH_MASK) >> PCIE_LINK_STATUS_WIDTH_OFF;
-}
-
-static void dw_pcie_writel_ob_unroll(struct pcie_dw_ti *pci, u32 index, u32 reg,
-                                    u32 val)
-{
-       u32 offset = PCIE_GET_ATU_OUTB_UNR_REG_OFFSET(index);
-       void __iomem *base = pci->atu_base;
-
-       writel(val, base + offset + reg);
-}
-
-static u32 dw_pcie_readl_ob_unroll(struct pcie_dw_ti *pci, u32 index, u32 reg)
-{
-       u32 offset = PCIE_GET_ATU_OUTB_UNR_REG_OFFSET(index);
-       void __iomem *base = pci->atu_base;
-
-       return readl(base + offset + reg);
-}
-
-/**
- * pcie_dw_prog_outbound_atu_unroll() - Configure ATU for outbound accesses
- *
- * @pcie: Pointer to the PCI controller state
- * @index: ATU region index
- * @type: ATU accsess type
- * @cpu_addr: the physical address for the translation entry
- * @pci_addr: the pcie bus address for the translation entry
- * @size: the size of the translation entry
- */
-static void pcie_dw_prog_outbound_atu_unroll(struct pcie_dw_ti *pci, int index,
-                                            int type, u64 cpu_addr,
-                                            u64 pci_addr, u32 size)
-{
-       u32 retries, val;
-
-       debug("ATU programmed with: index: %d, type: %d, cpu addr: %8llx, pci addr: %8llx, size: %8x\n",
-             index, type, cpu_addr, pci_addr, size);
-
-       dw_pcie_writel_ob_unroll(pci, index, PCIE_ATU_UNR_LOWER_BASE,
-                                lower_32_bits(cpu_addr));
-       dw_pcie_writel_ob_unroll(pci, index, PCIE_ATU_UNR_UPPER_BASE,
-                                upper_32_bits(cpu_addr));
-       dw_pcie_writel_ob_unroll(pci, index, PCIE_ATU_UNR_LIMIT,
-                                lower_32_bits(cpu_addr + size - 1));
-       dw_pcie_writel_ob_unroll(pci, index, PCIE_ATU_UNR_LOWER_TARGET,
-                                lower_32_bits(pci_addr));
-       dw_pcie_writel_ob_unroll(pci, index, PCIE_ATU_UNR_UPPER_TARGET,
-                                upper_32_bits(pci_addr));
-       dw_pcie_writel_ob_unroll(pci, index, PCIE_ATU_UNR_REGION_CTRL1,
-                                type);
-       dw_pcie_writel_ob_unroll(pci, index, PCIE_ATU_UNR_REGION_CTRL2,
-                                PCIE_ATU_ENABLE);
-
-       /*
-        * Make sure ATU enable takes effect before any subsequent config
-        * and I/O accesses.
-        */
-       for (retries = 0; retries < LINK_WAIT_MAX_IATU_RETRIES; retries++) {
-               val = dw_pcie_readl_ob_unroll(pci, index,
-                                             PCIE_ATU_UNR_REGION_CTRL2);
-               if (val & PCIE_ATU_ENABLE)
-                       return;
-
-               udelay(LINK_WAIT_IATU);
-       }
-       dev_err(pci->dev, "outbound iATU is not being enabled\n");
-}
-
-/**
- * set_cfg_address() - Configure the PCIe controller config space access
- *
- * @pcie: Pointer to the PCI controller state
- * @d: PCI device to access
- * @where: Offset in the configuration space
- *
- * Configures the PCIe controller to access the configuration space of
- * a specific PCIe device and returns the address to use for this
- * access.
- *
- * Return: Address that can be used to access the configation space
- *         of the requested device / offset
- */
-static uintptr_t set_cfg_address(struct pcie_dw_ti *pcie,
-                                pci_dev_t d, uint where)
-{
-       int bus = PCI_BUS(d) - pcie->first_busno;
-       uintptr_t va_address;
-       u32 atu_type;
-
-       /* Use dbi_base for own configuration read and write */
-       if (!bus) {
-               va_address = (uintptr_t)pcie->dbi_base;
-               goto out;
-       }
-
-       if (bus == 1)
-               /* For local bus, change TLP Type field to 4. */
-               atu_type = PCIE_ATU_TYPE_CFG0;
-       else
-               /* Otherwise, change TLP Type field to 5. */
-               atu_type = PCIE_ATU_TYPE_CFG1;
-
-       /*
-        * Not accessing root port configuration space?
-        * Region #0 is used for Outbound CFG space access.
-        * Direction = Outbound
-        * Region Index = 0
-        */
-       d = PCI_MASK_BUS(d);
-       d = PCI_ADD_BUS(bus, d);
-       pcie_dw_prog_outbound_atu_unroll(pcie, PCIE_ATU_REGION_INDEX1,
-                                        atu_type, (u64)pcie->cfg_base,
-                                        d << 8, pcie->cfg_size);
-
-       va_address = (uintptr_t)pcie->cfg_base;
-
-out:
-       va_address += where & ~0x3;
-
-       return va_address;
-}
-
-/**
- * pcie_dw_addr_valid() - Check for valid bus address
- *
- * @d: The PCI device to access
- * @first_busno: Bus number of the PCIe controller root complex
- *
- * Return 1 (true) if the PCI device can be accessed by this controller.
- *
- * Return: 1 on valid, 0 on invalid
- */
-static int pcie_dw_addr_valid(pci_dev_t d, int first_busno)
-{
-       if ((PCI_BUS(d) == first_busno) && (PCI_DEV(d) > 0))
-               return 0;
-       if ((PCI_BUS(d) == first_busno + 1) && (PCI_DEV(d) > 0))
-               return 0;
-
-       return 1;
-}
-
-/**
- * pcie_dw_ti_read_config() - Read from configuration space
- *
- * @bus: Pointer to the PCI bus
- * @bdf: Identifies the PCIe device to access
- * @offset: The offset into the device's configuration space
- * @valuep: A pointer at which to store the read value
- * @size: Indicates the size of access to perform
- *
- * Read a value of size @size from offset @offset within the configuration
- * space of the device identified by the bus, device & function numbers in @bdf
- * on the PCI bus @bus.
- *
- * Return: 0 on success
- */
-static int pcie_dw_ti_read_config(const struct udevice *bus, pci_dev_t bdf,
-                                 uint offset, ulong *valuep,
-                                 enum pci_size_t size)
-{
-       struct pcie_dw_ti *pcie = dev_get_priv(bus);
-       uintptr_t va_address;
-       ulong value;
-
-       debug("PCIE CFG read: bdf=%2x:%2x:%2x ",
-             PCI_BUS(bdf), PCI_DEV(bdf), PCI_FUNC(bdf));
-
-       if (!pcie_dw_addr_valid(bdf, pcie->first_busno)) {
-               debug("- out of range\n");
-               *valuep = pci_get_ff(size);
-               return 0;
-       }
-
-       va_address = set_cfg_address(pcie, bdf, offset);
-
-       value = readl(va_address);
-
-       debug("(addr,val)=(0x%04x, 0x%08lx)\n", offset, value);
-       *valuep = pci_conv_32_to_size(value, offset, size);
-
-       pcie_dw_prog_outbound_atu_unroll(pcie, PCIE_ATU_REGION_INDEX1,
-                                        PCIE_ATU_TYPE_IO, pcie->io.phys_start,
-                                        pcie->io.bus_start, pcie->io.size);
-
-       return 0;
-}
-
-/**
- * pcie_dw_ti_write_config() - Write to configuration space
- *
- * @bus: Pointer to the PCI bus
- * @bdf: Identifies the PCIe device to access
- * @offset: The offset into the device's configuration space
- * @value: The value to write
- * @size: Indicates the size of access to perform
- *
- * Write the value @value of size @size from offset @offset within the
- * configuration space of the device identified by the bus, device & function
- * numbers in @bdf on the PCI bus @bus.
- *
- * Return: 0 on success
- */
-static int pcie_dw_ti_write_config(struct udevice *bus, pci_dev_t bdf,
-                                  uint offset, ulong value,
-                                  enum pci_size_t size)
-{
-       struct pcie_dw_ti *pcie = dev_get_priv(bus);
-       uintptr_t va_address;
-       ulong old;
-
-       debug("PCIE CFG write: (b,d,f)=(%2d,%2d,%2d) ",
-             PCI_BUS(bdf), PCI_DEV(bdf), PCI_FUNC(bdf));
-       debug("(addr,val)=(0x%04x, 0x%08lx)\n", offset, value);
-
-       if (!pcie_dw_addr_valid(bdf, pcie->first_busno)) {
-               debug("- out of range\n");
-               return 0;
-       }
-
-       va_address = set_cfg_address(pcie, bdf, offset);
-
-       old = readl(va_address);
-       value = pci_conv_size_to_32(old, value, offset, size);
-       writel(value, va_address);
-
-       pcie_dw_prog_outbound_atu_unroll(pcie, PCIE_ATU_REGION_INDEX1,
-                                        PCIE_ATU_TYPE_IO, pcie->io.phys_start,
-                                        pcie->io.bus_start, pcie->io.size);
-
-       return 0;
-}
-
-static inline void dw_pcie_dbi_write_enable(struct pcie_dw_ti *pci, bool en)
-{
-       u32 val;
-
-       val = readl(pci->dbi_base + PCIE_MISC_CONTROL_1_OFF);
-       if (en)
-               val |= PCIE_DBI_RO_WR_EN;
-       else
-               val &= ~PCIE_DBI_RO_WR_EN;
-       writel(val, pci->dbi_base + PCIE_MISC_CONTROL_1_OFF);
-}
-
 /**
  * pcie_dw_configure() - Configure link capabilities and speed
  *
@@ -395,19 +84,19 @@ static void pcie_dw_configure(struct pcie_dw_ti *pci, u32 cap_speed)
 {
        u32 val;
 
-       dw_pcie_dbi_write_enable(pci, true);
+       dw_pcie_dbi_write_enable(&pci->dw, true);
 
-       val = readl(pci->dbi_base + PCIE_LINK_CAPABILITY);
+       val = readl(pci->dw.dbi_base + PCIE_LINK_CAPABILITY);
        val &= ~TARGET_LINK_SPEED_MASK;
        val |= cap_speed;
-       writel(val, pci->dbi_base + PCIE_LINK_CAPABILITY);
+       writel(val, pci->dw.dbi_base + PCIE_LINK_CAPABILITY);
 
-       val = readl(pci->dbi_base + PCIE_LINK_CTL_2);
+       val = readl(pci->dw.dbi_base + PCIE_LINK_CTL_2);
        val &= ~TARGET_LINK_SPEED_MASK;
        val |= cap_speed;
-       writel(val, pci->dbi_base + PCIE_LINK_CTL_2);
+       writel(val, pci->dw.dbi_base + PCIE_LINK_CTL_2);
 
-       dw_pcie_dbi_write_enable(pci, false);
+       dw_pcie_dbi_write_enable(&pci->dw, false);
 }
 
 /**
@@ -421,7 +110,7 @@ static int is_link_up(struct pcie_dw_ti *pci)
 {
        u32 val;
 
-       val = readl(pci->dbi_base + PCIE_PORT_DEBUG0);
+       val = readl(pci->dw.dbi_base + PCIE_PORT_DEBUG0);
        val &= PORT_LOGIC_LTSSM_STATE_MASK;
 
        return (val == PORT_LOGIC_LTSSM_STATE_L0);
@@ -477,56 +166,6 @@ static int pcie_dw_ti_pcie_link_up(struct pcie_dw_ti *pci, u32 cap_speed)
        return 1;
 }
 
-/**
- * pcie_dw_setup_host() - Setup the PCIe controller for RC opertaion
- *
- * @pcie: Pointer to the PCI controller state
- *
- * Configure the host BARs of the PCIe controller root port so that
- * PCI(e) devices may access the system memory.
- */
-static void pcie_dw_setup_host(struct pcie_dw_ti *pci)
-{
-       u32 val;
-
-       /* setup RC BARs */
-       writel(PCI_BASE_ADDRESS_MEM_TYPE_64,
-              pci->dbi_base + PCI_BASE_ADDRESS_0);
-       writel(0x0, pci->dbi_base + PCI_BASE_ADDRESS_1);
-
-       /* setup interrupt pins */
-       dw_pcie_dbi_write_enable(pci, true);
-       val = readl(pci->dbi_base + PCI_INTERRUPT_LINE);
-       val &= 0xffff00ff;
-       val |= 0x00000100;
-       writel(val, pci->dbi_base + PCI_INTERRUPT_LINE);
-       dw_pcie_dbi_write_enable(pci, false);
-
-       /* setup bus numbers */
-       val = readl(pci->dbi_base + PCI_PRIMARY_BUS);
-       val &= 0xff000000;
-       val |= 0x00ff0100;
-       writel(val, pci->dbi_base + PCI_PRIMARY_BUS);
-
-       /* setup command register */
-       val = readl(pci->dbi_base + PCI_COMMAND);
-       val &= 0xffff0000;
-       val |= PCI_COMMAND_IO | PCI_COMMAND_MEMORY |
-               PCI_COMMAND_MASTER | PCI_COMMAND_SERR;
-       writel(val, pci->dbi_base + PCI_COMMAND);
-
-       /* Enable write permission for the DBI read-only register */
-       dw_pcie_dbi_write_enable(pci, true);
-       /* program correct class for RC */
-       writew(PCI_CLASS_BRIDGE_PCI, pci->dbi_base + PCI_CLASS_DEVICE);
-       /* Better disable write permission right after the update */
-       dw_pcie_dbi_write_enable(pci, false);
-
-       val = readl(pci->dbi_base + PCIE_LINK_WIDTH_SPEED_CONTROL);
-       val |= PORT_LOGIC_SPEED_CHANGE;
-       writel(val, pci->dbi_base + PCIE_LINK_WIDTH_SPEED_CONTROL);
-}
-
 static int pcie_am654_set_mode(struct pcie_dw_ti *pci,
                               enum dw_pcie_device_mode mode)
 {
@@ -535,7 +174,7 @@ static int pcie_am654_set_mode(struct pcie_dw_ti *pci,
        u32 mask;
        int ret;
 
-       syscon = syscon_regmap_lookup_by_phandle(pci->dev,
+       syscon = syscon_regmap_lookup_by_phandle(pci->dw.dev,
                                                 "ti,syscon-pcie-mode");
        if (IS_ERR(syscon))
                return 0;
@@ -550,13 +189,13 @@ static int pcie_am654_set_mode(struct pcie_dw_ti *pci,
                val = EP;
                break;
        default:
-               dev_err(pci->dev, "INVALID device type %d\n", mode);
+               dev_err(pci->dw.dev, "INVALID device type %d\n", mode);
                return -EINVAL;
        }
 
        ret = regmap_update_bits(syscon, 0, mask, val);
        if (ret) {
-               dev_err(pci->dev, "failed to set pcie mode\n");
+               dev_err(pci->dw.dev, "failed to set pcie mode\n");
                return ret;
        }
 
@@ -569,7 +208,7 @@ static int pcie_dw_init_id(struct pcie_dw_ti *pci)
        unsigned int id;
        int ret;
 
-       devctrl_regs = syscon_regmap_lookup_by_phandle(pci->dev,
+       devctrl_regs = syscon_regmap_lookup_by_phandle(pci->dw.dev,
                                                       "ti,syscon-pcie-id");
        if (IS_ERR(devctrl_regs))
                return PTR_ERR(devctrl_regs);
@@ -578,10 +217,10 @@ static int pcie_dw_init_id(struct pcie_dw_ti *pci)
        if (ret)
                return ret;
 
-       dw_pcie_dbi_write_enable(pci, true);
-       writew(id & PCIE_VENDORID_MASK, pci->dbi_base + PCI_VENDOR_ID);
-       writew(id >> PCIE_DEVICEID_SHIFT, pci->dbi_base + PCI_DEVICE_ID);
-       dw_pcie_dbi_write_enable(pci, false);
+       dw_pcie_dbi_write_enable(&pci->dw, true);
+       writew(id & PCIE_VENDORID_MASK, pci->dw.dbi_base + PCI_VENDOR_ID);
+       writew(id >> PCIE_DEVICEID_SHIFT, pci->dw.dbi_base + PCI_DEVICE_ID);
+       dw_pcie_dbi_write_enable(&pci->dw, false);
 
        return 0;
 }
@@ -635,10 +274,10 @@ static int pcie_dw_ti_probe(struct udevice *dev)
        generic_phy_init(&phy1);
        generic_phy_power_on(&phy1);
 
-       pci->first_busno = dev_seq(dev);
-       pci->dev = dev;
+       pci->dw.first_busno = dev_seq(dev);
+       pci->dw.dev = dev;
 
-       pcie_dw_setup_host(pci);
+       pcie_dw_setup_host(&pci->dw);
        pcie_dw_init_id(pci);
 
        if (device_is_compatible(dev, "ti,am654-pcie-rc"))
@@ -650,23 +289,14 @@ static int pcie_dw_ti_probe(struct udevice *dev)
        }
 
        printf("PCIE-%d: Link up (Gen%d-x%d, Bus%d)\n", dev_seq(dev),
-              pcie_dw_get_link_speed(pci),
-              pcie_dw_get_link_width(pci),
+              pcie_dw_get_link_speed(&pci->dw),
+              pcie_dw_get_link_width(&pci->dw),
               hose->first_busno);
 
-       /* Store the IO and MEM windows settings for future use by the ATU */
-       pci->io.phys_start = hose->regions[0].phys_start; /* IO base */
-       pci->io.bus_start  = hose->regions[0].bus_start;  /* IO_bus_addr */
-       pci->io.size        = hose->regions[0].size;       /* IO size */
-
-       pci->mem.phys_start = hose->regions[1].phys_start; /* MEM base */
-       pci->mem.bus_start  = hose->regions[1].bus_start;  /* MEM_bus_addr */
-       pci->mem.size        = hose->regions[1].size;       /* MEM size */
-
-       pcie_dw_prog_outbound_atu_unroll(pci, PCIE_ATU_REGION_INDEX0,
+       pcie_dw_prog_outbound_atu_unroll(&pci->dw, PCIE_ATU_REGION_INDEX0,
                                         PCIE_ATU_TYPE_MEM,
-                                        pci->mem.phys_start,
-                                        pci->mem.bus_start, pci->mem.size);
+                                        pci->dw.mem.phys_start,
+                                        pci->dw.mem.bus_start, pci->dw.mem.size);
 
        return 0;
 }
@@ -687,19 +317,19 @@ static int pcie_dw_ti_of_to_plat(struct udevice *dev)
        struct pcie_dw_ti *pcie = dev_get_priv(dev);
 
        /* Get the controller base address */
-       pcie->dbi_base = (void *)dev_read_addr_name(dev, "dbics");
-       if ((fdt_addr_t)pcie->dbi_base == FDT_ADDR_T_NONE)
+       pcie->dw.dbi_base = (void *)dev_read_addr_name(dev, "dbics");
+       if ((fdt_addr_t)pcie->dw.dbi_base == FDT_ADDR_T_NONE)
                return -EINVAL;
 
        /* Get the config space base address and size */
-       pcie->cfg_base = (void *)dev_read_addr_size_name(dev, "config",
-                                                        &pcie->cfg_size);
-       if ((fdt_addr_t)pcie->cfg_base == FDT_ADDR_T_NONE)
+       pcie->dw.cfg_base = (void *)dev_read_addr_size_name(dev, "config",
+                                                        &pcie->dw.cfg_size);
+       if ((fdt_addr_t)pcie->dw.cfg_base == FDT_ADDR_T_NONE)
                return -EINVAL;
 
        /* Get the iATU base address and size */
-       pcie->atu_base = (void *)dev_read_addr_name(dev, "atu");
-       if ((fdt_addr_t)pcie->atu_base == FDT_ADDR_T_NONE)
+       pcie->dw.atu_base = (void *)dev_read_addr_name(dev, "atu");
+       if ((fdt_addr_t)pcie->dw.atu_base == FDT_ADDR_T_NONE)
                return -EINVAL;
 
        /* Get the app base address and size */
@@ -711,8 +341,8 @@ static int pcie_dw_ti_of_to_plat(struct udevice *dev)
 }
 
 static const struct dm_pci_ops pcie_dw_ti_ops = {
-       .read_config    = pcie_dw_ti_read_config,
-       .write_config   = pcie_dw_ti_write_config,
+       .read_config    = pcie_dw_read_config,
+       .write_config   = pcie_dw_write_config,
 };
 
 static const struct udevice_id pcie_dw_ti_ids[] = {