Merge branch 'remotes/lorenzo/pci/mediatek'
authorBjorn Helgaas <bhelgaas@google.com>
Tue, 4 May 2021 15:43:28 +0000 (10:43 -0500)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 4 May 2021 15:43:28 +0000 (10:43 -0500)
- Configure FC and FTS for functions other than 0 (Ryder Lee)

- Add missing MODULE_DEVICE_TABLE (Qiheng Lin)

- Add YAML schema for MediaTek (Jianjun Wang)

- Export pci_pio_to_address() for module use (Jianjun Wang)

- Add MediaTek MT8192 PCIe controller driver (Jianjun Wang)

- Add MediaTek MT8192 INTx support (Jianjun Wang)

- Add MediaTek MT8192 MSI support (Jianjun Wang)

- Add MediaTek MT8192 system power management support (Jianjun Wang)

* remotes/lorenzo/pci/mediatek:
  MAINTAINERS: Add Jianjun Wang as MediaTek PCI co-maintainer
  PCI: mediatek-gen3: Add system PM support
  PCI: mediatek-gen3: Add MSI support
  PCI: mediatek-gen3: Add INTx support
  PCI: mediatek-gen3: Add MediaTek Gen3 driver for MT8192
  PCI: Export pci_pio_to_address() for module use
  dt-bindings: PCI: mediatek-gen3: Add YAML schema
  PCI: mediatek: Add missing MODULE_DEVICE_TABLE
  PCI: mediatek: Configure FC and FTS for functions other than 0

Documentation/devicetree/bindings/pci/mediatek-pcie-gen3.yaml [new file with mode: 0644]
MAINTAINERS
drivers/pci/controller/Kconfig
drivers/pci/controller/Makefile
drivers/pci/controller/pcie-mediatek-gen3.c [new file with mode: 0644]
drivers/pci/controller/pcie-mediatek.c
drivers/pci/pci.c

diff --git a/Documentation/devicetree/bindings/pci/mediatek-pcie-gen3.yaml b/Documentation/devicetree/bindings/pci/mediatek-pcie-gen3.yaml
new file mode 100644 (file)
index 0000000..e7b1f98
--- /dev/null
@@ -0,0 +1,181 @@
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/pci/mediatek-pcie-gen3.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Gen3 PCIe controller on MediaTek SoCs
+
+maintainers:
+  - Jianjun Wang <jianjun.wang@mediatek.com>
+
+description: |+
+  PCIe Gen3 MAC controller for MediaTek SoCs, it supports Gen3 speed
+  and compatible with Gen2, Gen1 speed.
+
+  This PCIe controller supports up to 256 MSI vectors, the MSI hardware
+  block diagram is as follows:
+
+                    +-----+
+                    | GIC |
+                    +-----+
+                       ^
+                       |
+                   port->irq
+                       |
+               +-+-+-+-+-+-+-+-+
+               |0|1|2|3|4|5|6|7| (PCIe intc)
+               +-+-+-+-+-+-+-+-+
+                ^ ^           ^
+                | |    ...    |
+        +-------+ +------+    +-----------+
+        |                |                |
+  +-+-+---+--+--+  +-+-+---+--+--+  +-+-+---+--+--+
+  |0|1|...|30|31|  |0|1|...|30|31|  |0|1|...|30|31| (MSI sets)
+  +-+-+---+--+--+  +-+-+---+--+--+  +-+-+---+--+--+
+   ^ ^      ^  ^    ^ ^      ^  ^    ^ ^      ^  ^
+   | |      |  |    | |      |  |    | |      |  |  (MSI vectors)
+   | |      |  |    | |      |  |    | |      |  |
+
+    (MSI SET0)       (MSI SET1)  ...   (MSI SET7)
+
+  With 256 MSI vectors supported, the MSI vectors are composed of 8 sets,
+  each set has its own address for MSI message, and supports 32 MSI vectors
+  to generate interrupt.
+
+allOf:
+  - $ref: /schemas/pci/pci-bus.yaml#
+
+properties:
+  compatible:
+    const: mediatek,mt8192-pcie
+
+  reg:
+    maxItems: 1
+
+  reg-names:
+    items:
+      - const: pcie-mac
+
+  interrupts:
+    maxItems: 1
+
+  ranges:
+    minItems: 1
+    maxItems: 8
+
+  resets:
+    minItems: 1
+    maxItems: 2
+
+  reset-names:
+    minItems: 1
+    maxItems: 2
+    items:
+      - const: phy
+      - const: mac
+
+  clocks:
+    maxItems: 6
+
+  clock-names:
+    items:
+      - const: pl_250m
+      - const: tl_26m
+      - const: tl_96m
+      - const: tl_32k
+      - const: peri_26m
+      - const: top_133m
+
+  assigned-clocks:
+    maxItems: 1
+
+  assigned-clock-parents:
+    maxItems: 1
+
+  phys:
+    maxItems: 1
+
+  '#interrupt-cells':
+    const: 1
+
+  interrupt-controller:
+    description: Interrupt controller node for handling legacy PCI interrupts.
+    type: object
+    properties:
+      '#address-cells':
+        const: 0
+      '#interrupt-cells':
+        const: 1
+      interrupt-controller: true
+
+    required:
+      - '#address-cells'
+      - '#interrupt-cells'
+      - interrupt-controller
+
+    additionalProperties: false
+
+required:
+  - compatible
+  - reg
+  - reg-names
+  - interrupts
+  - ranges
+  - clocks
+  - '#interrupt-cells'
+  - interrupt-controller
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    #include <dt-bindings/interrupt-controller/arm-gic.h>
+    #include <dt-bindings/interrupt-controller/irq.h>
+
+    bus {
+        #address-cells = <2>;
+        #size-cells = <2>;
+
+        pcie: pcie@11230000 {
+            compatible = "mediatek,mt8192-pcie";
+            device_type = "pci";
+            #address-cells = <3>;
+            #size-cells = <2>;
+            reg = <0x00 0x11230000 0x00 0x4000>;
+            reg-names = "pcie-mac";
+            interrupts = <GIC_SPI 251 IRQ_TYPE_LEVEL_HIGH 0>;
+            bus-range = <0x00 0xff>;
+            ranges = <0x82000000 0x00 0x12000000 0x00
+                      0x12000000 0x00 0x1000000>;
+            clocks = <&infracfg 44>,
+                     <&infracfg 40>,
+                     <&infracfg 43>,
+                     <&infracfg 97>,
+                     <&infracfg 99>,
+                     <&infracfg 111>;
+            clock-names = "pl_250m", "tl_26m", "tl_96m",
+                          "tl_32k", "peri_26m", "top_133m";
+            assigned-clocks = <&topckgen 50>;
+            assigned-clock-parents = <&topckgen 91>;
+
+            phys = <&pciephy>;
+            phy-names = "pcie-phy";
+
+            resets = <&infracfg_rst 2>,
+                     <&infracfg_rst 3>;
+            reset-names = "phy", "mac";
+
+            #interrupt-cells = <1>;
+            interrupt-map-mask = <0 0 0 0x7>;
+            interrupt-map = <0 0 0 1 &pcie_intc 0>,
+                            <0 0 0 2 &pcie_intc 1>,
+                            <0 0 0 3 &pcie_intc 2>,
+                            <0 0 0 4 &pcie_intc 3>;
+            pcie_intc: interrupt-controller {
+                      #address-cells = <0>;
+                      #interrupt-cells = <1>;
+                      interrupt-controller;
+            };
+        };
+    };
index d92f85c..8050c14 100644 (file)
@@ -13919,6 +13919,7 @@ F:      drivers/pci/controller/dwc/pcie-histb.c
 
 PCIE DRIVER FOR MEDIATEK
 M:     Ryder Lee <ryder.lee@mediatek.com>
+M:     Jianjun Wang <jianjun.wang@mediatek.com>
 L:     linux-pci@vger.kernel.org
 L:     linux-mediatek@lists.infradead.org
 S:     Supported
index 5aa8977..1e925ac 100644 (file)
@@ -233,6 +233,19 @@ config PCIE_MEDIATEK
          Say Y here if you want to enable PCIe controller support on
          MediaTek SoCs.
 
+config PCIE_MEDIATEK_GEN3
+       tristate "MediaTek Gen3 PCIe controller"
+       depends on ARCH_MEDIATEK || COMPILE_TEST
+       depends on PCI_MSI_IRQ_DOMAIN
+       help
+         Adds support for PCIe Gen3 MAC controller for MediaTek SoCs.
+         This PCIe controller is compatible with Gen3, Gen2 and Gen1 speed,
+         and support up to 256 MSI interrupt numbers for
+         multi-function devices.
+
+         Say Y here if you want to enable Gen3 PCIe controller support on
+         MediaTek SoCs.
+
 config VMD
        depends on PCI_MSI && X86_64 && SRCU
        tristate "Intel Volume Management Device Driver"
index 6d24a16..63e3880 100644 (file)
@@ -30,6 +30,7 @@ obj-$(CONFIG_PCIE_ROCKCHIP) += pcie-rockchip.o
 obj-$(CONFIG_PCIE_ROCKCHIP_EP) += pcie-rockchip-ep.o
 obj-$(CONFIG_PCIE_ROCKCHIP_HOST) += pcie-rockchip-host.o
 obj-$(CONFIG_PCIE_MEDIATEK) += pcie-mediatek.o
+obj-$(CONFIG_PCIE_MEDIATEK_GEN3) += pcie-mediatek-gen3.o
 obj-$(CONFIG_PCIE_MICROCHIP_HOST) += pcie-microchip-host.o
 obj-$(CONFIG_VMD) += vmd.o
 obj-$(CONFIG_PCIE_BRCMSTB) += pcie-brcmstb.o
diff --git a/drivers/pci/controller/pcie-mediatek-gen3.c b/drivers/pci/controller/pcie-mediatek-gen3.c
new file mode 100644 (file)
index 0000000..3c5b977
--- /dev/null
@@ -0,0 +1,1027 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * MediaTek PCIe host controller driver.
+ *
+ * Copyright (c) 2020 MediaTek Inc.
+ * Author: Jianjun Wang <jianjun.wang@mediatek.com>
+ */
+
+#include <linux/clk.h>
+#include <linux/delay.h>
+#include <linux/iopoll.h>
+#include <linux/irq.h>
+#include <linux/irqchip/chained_irq.h>
+#include <linux/irqdomain.h>
+#include <linux/kernel.h>
+#include <linux/module.h>
+#include <linux/msi.h>
+#include <linux/pci.h>
+#include <linux/phy/phy.h>
+#include <linux/platform_device.h>
+#include <linux/pm_domain.h>
+#include <linux/pm_runtime.h>
+#include <linux/reset.h>
+
+#include "../pci.h"
+
+#define PCIE_SETTING_REG               0x80
+#define PCIE_PCI_IDS_1                 0x9c
+#define PCI_CLASS(class)               (class << 8)
+#define PCIE_RC_MODE                   BIT(0)
+
+#define PCIE_CFGNUM_REG                        0x140
+#define PCIE_CFG_DEVFN(devfn)          ((devfn) & GENMASK(7, 0))
+#define PCIE_CFG_BUS(bus)              (((bus) << 8) & GENMASK(15, 8))
+#define PCIE_CFG_BYTE_EN(bytes)                (((bytes) << 16) & GENMASK(19, 16))
+#define PCIE_CFG_FORCE_BYTE_EN         BIT(20)
+#define PCIE_CFG_OFFSET_ADDR           0x1000
+#define PCIE_CFG_HEADER(bus, devfn) \
+       (PCIE_CFG_BUS(bus) | PCIE_CFG_DEVFN(devfn))
+
+#define PCIE_RST_CTRL_REG              0x148
+#define PCIE_MAC_RSTB                  BIT(0)
+#define PCIE_PHY_RSTB                  BIT(1)
+#define PCIE_BRG_RSTB                  BIT(2)
+#define PCIE_PE_RSTB                   BIT(3)
+
+#define PCIE_LTSSM_STATUS_REG          0x150
+#define PCIE_LTSSM_STATE_MASK          GENMASK(28, 24)
+#define PCIE_LTSSM_STATE(val)          ((val & PCIE_LTSSM_STATE_MASK) >> 24)
+#define PCIE_LTSSM_STATE_L2_IDLE       0x14
+
+#define PCIE_LINK_STATUS_REG           0x154
+#define PCIE_PORT_LINKUP               BIT(8)
+
+#define PCIE_MSI_SET_NUM               8
+#define PCIE_MSI_IRQS_PER_SET          32
+#define PCIE_MSI_IRQS_NUM \
+       (PCIE_MSI_IRQS_PER_SET * PCIE_MSI_SET_NUM)
+
+#define PCIE_INT_ENABLE_REG            0x180
+#define PCIE_MSI_ENABLE                        GENMASK(PCIE_MSI_SET_NUM + 8 - 1, 8)
+#define PCIE_MSI_SHIFT                 8
+#define PCIE_INTX_SHIFT                        24
+#define PCIE_INTX_ENABLE \
+       GENMASK(PCIE_INTX_SHIFT + PCI_NUM_INTX - 1, PCIE_INTX_SHIFT)
+
+#define PCIE_INT_STATUS_REG            0x184
+#define PCIE_MSI_SET_ENABLE_REG                0x190
+#define PCIE_MSI_SET_ENABLE            GENMASK(PCIE_MSI_SET_NUM - 1, 0)
+
+#define PCIE_MSI_SET_BASE_REG          0xc00
+#define PCIE_MSI_SET_OFFSET            0x10
+#define PCIE_MSI_SET_STATUS_OFFSET     0x04
+#define PCIE_MSI_SET_ENABLE_OFFSET     0x08
+
+#define PCIE_MSI_SET_ADDR_HI_BASE      0xc80
+#define PCIE_MSI_SET_ADDR_HI_OFFSET    0x04
+
+#define PCIE_ICMD_PM_REG               0x198
+#define PCIE_TURN_OFF_LINK             BIT(4)
+
+#define PCIE_TRANS_TABLE_BASE_REG      0x800
+#define PCIE_ATR_SRC_ADDR_MSB_OFFSET   0x4
+#define PCIE_ATR_TRSL_ADDR_LSB_OFFSET  0x8
+#define PCIE_ATR_TRSL_ADDR_MSB_OFFSET  0xc
+#define PCIE_ATR_TRSL_PARAM_OFFSET     0x10
+#define PCIE_ATR_TLB_SET_OFFSET                0x20
+
+#define PCIE_MAX_TRANS_TABLES          8
+#define PCIE_ATR_EN                    BIT(0)
+#define PCIE_ATR_SIZE(size) \
+       (((((size) - 1) << 1) & GENMASK(6, 1)) | PCIE_ATR_EN)
+#define PCIE_ATR_ID(id)                        ((id) & GENMASK(3, 0))
+#define PCIE_ATR_TYPE_MEM              PCIE_ATR_ID(0)
+#define PCIE_ATR_TYPE_IO               PCIE_ATR_ID(1)
+#define PCIE_ATR_TLP_TYPE(type)                (((type) << 16) & GENMASK(18, 16))
+#define PCIE_ATR_TLP_TYPE_MEM          PCIE_ATR_TLP_TYPE(0)
+#define PCIE_ATR_TLP_TYPE_IO           PCIE_ATR_TLP_TYPE(2)
+
+/**
+ * struct mtk_msi_set - MSI information for each set
+ * @base: IO mapped register base
+ * @msg_addr: MSI message address
+ * @saved_irq_state: IRQ enable state saved at suspend time
+ */
+struct mtk_msi_set {
+       void __iomem *base;
+       phys_addr_t msg_addr;
+       u32 saved_irq_state;
+};
+
+/**
+ * struct mtk_pcie_port - PCIe port information
+ * @dev: pointer to PCIe device
+ * @base: IO mapped register base
+ * @reg_base: physical register base
+ * @mac_reset: MAC reset control
+ * @phy_reset: PHY reset control
+ * @phy: PHY controller block
+ * @clks: PCIe clocks
+ * @num_clks: PCIe clocks count for this port
+ * @irq: PCIe controller interrupt number
+ * @saved_irq_state: IRQ enable state saved at suspend time
+ * @irq_lock: lock protecting IRQ register access
+ * @intx_domain: legacy INTx IRQ domain
+ * @msi_domain: MSI IRQ domain
+ * @msi_bottom_domain: MSI IRQ bottom domain
+ * @msi_sets: MSI sets information
+ * @lock: lock protecting IRQ bit map
+ * @msi_irq_in_use: bit map for assigned MSI IRQ
+ */
+struct mtk_pcie_port {
+       struct device *dev;
+       void __iomem *base;
+       phys_addr_t reg_base;
+       struct reset_control *mac_reset;
+       struct reset_control *phy_reset;
+       struct phy *phy;
+       struct clk_bulk_data *clks;
+       int num_clks;
+
+       int irq;
+       u32 saved_irq_state;
+       raw_spinlock_t irq_lock;
+       struct irq_domain *intx_domain;
+       struct irq_domain *msi_domain;
+       struct irq_domain *msi_bottom_domain;
+       struct mtk_msi_set msi_sets[PCIE_MSI_SET_NUM];
+       struct mutex lock;
+       DECLARE_BITMAP(msi_irq_in_use, PCIE_MSI_IRQS_NUM);
+};
+
+/**
+ * mtk_pcie_config_tlp_header() - Configure a configuration TLP header
+ * @bus: PCI bus to query
+ * @devfn: device/function number
+ * @where: offset in config space
+ * @size: data size in TLP header
+ *
+ * Set byte enable field and device information in configuration TLP header.
+ */
+static void mtk_pcie_config_tlp_header(struct pci_bus *bus, unsigned int devfn,
+                                       int where, int size)
+{
+       struct mtk_pcie_port *port = bus->sysdata;
+       int bytes;
+       u32 val;
+
+       bytes = (GENMASK(size - 1, 0) & 0xf) << (where & 0x3);
+
+       val = PCIE_CFG_FORCE_BYTE_EN | PCIE_CFG_BYTE_EN(bytes) |
+             PCIE_CFG_HEADER(bus->number, devfn);
+
+       writel_relaxed(val, port->base + PCIE_CFGNUM_REG);
+}
+
+static void __iomem *mtk_pcie_map_bus(struct pci_bus *bus, unsigned int devfn,
+                                     int where)
+{
+       struct mtk_pcie_port *port = bus->sysdata;
+
+       return port->base + PCIE_CFG_OFFSET_ADDR + where;
+}
+
+static int mtk_pcie_config_read(struct pci_bus *bus, unsigned int devfn,
+                               int where, int size, u32 *val)
+{
+       mtk_pcie_config_tlp_header(bus, devfn, where, size);
+
+       return pci_generic_config_read32(bus, devfn, where, size, val);
+}
+
+static int mtk_pcie_config_write(struct pci_bus *bus, unsigned int devfn,
+                                int where, int size, u32 val)
+{
+       mtk_pcie_config_tlp_header(bus, devfn, where, size);
+
+       if (size <= 2)
+               val <<= (where & 0x3) * 8;
+
+       return pci_generic_config_write32(bus, devfn, where, 4, val);
+}
+
+static struct pci_ops mtk_pcie_ops = {
+       .map_bus = mtk_pcie_map_bus,
+       .read  = mtk_pcie_config_read,
+       .write = mtk_pcie_config_write,
+};
+
+static int mtk_pcie_set_trans_table(struct mtk_pcie_port *port,
+                                   resource_size_t cpu_addr,
+                                   resource_size_t pci_addr,
+                                   resource_size_t size,
+                                   unsigned long type, int num)
+{
+       void __iomem *table;
+       u32 val;
+
+       if (num >= PCIE_MAX_TRANS_TABLES) {
+               dev_err(port->dev, "not enough translate table for addr: %#llx, limited to [%d]\n",
+                       (unsigned long long)cpu_addr, PCIE_MAX_TRANS_TABLES);
+               return -ENODEV;
+       }
+
+       table = port->base + PCIE_TRANS_TABLE_BASE_REG +
+               num * PCIE_ATR_TLB_SET_OFFSET;
+
+       writel_relaxed(lower_32_bits(cpu_addr) | PCIE_ATR_SIZE(fls(size) - 1),
+                      table);
+       writel_relaxed(upper_32_bits(cpu_addr),
+                      table + PCIE_ATR_SRC_ADDR_MSB_OFFSET);
+       writel_relaxed(lower_32_bits(pci_addr),
+                      table + PCIE_ATR_TRSL_ADDR_LSB_OFFSET);
+       writel_relaxed(upper_32_bits(pci_addr),
+                      table + PCIE_ATR_TRSL_ADDR_MSB_OFFSET);
+
+       if (type == IORESOURCE_IO)
+               val = PCIE_ATR_TYPE_IO | PCIE_ATR_TLP_TYPE_IO;
+       else
+               val = PCIE_ATR_TYPE_MEM | PCIE_ATR_TLP_TYPE_MEM;
+
+       writel_relaxed(val, table + PCIE_ATR_TRSL_PARAM_OFFSET);
+
+       return 0;
+}
+
+static void mtk_pcie_enable_msi(struct mtk_pcie_port *port)
+{
+       int i;
+       u32 val;
+
+       for (i = 0; i < PCIE_MSI_SET_NUM; i++) {
+               struct mtk_msi_set *msi_set = &port->msi_sets[i];
+
+               msi_set->base = port->base + PCIE_MSI_SET_BASE_REG +
+                               i * PCIE_MSI_SET_OFFSET;
+               msi_set->msg_addr = port->reg_base + PCIE_MSI_SET_BASE_REG +
+                                   i * PCIE_MSI_SET_OFFSET;
+
+               /* Configure the MSI capture address */
+               writel_relaxed(lower_32_bits(msi_set->msg_addr), msi_set->base);
+               writel_relaxed(upper_32_bits(msi_set->msg_addr),
+                              port->base + PCIE_MSI_SET_ADDR_HI_BASE +
+                              i * PCIE_MSI_SET_ADDR_HI_OFFSET);
+       }
+
+       val = readl_relaxed(port->base + PCIE_MSI_SET_ENABLE_REG);
+       val |= PCIE_MSI_SET_ENABLE;
+       writel_relaxed(val, port->base + PCIE_MSI_SET_ENABLE_REG);
+
+       val = readl_relaxed(port->base + PCIE_INT_ENABLE_REG);
+       val |= PCIE_MSI_ENABLE;
+       writel_relaxed(val, port->base + PCIE_INT_ENABLE_REG);
+}
+
+static int mtk_pcie_startup_port(struct mtk_pcie_port *port)
+{
+       struct resource_entry *entry;
+       struct pci_host_bridge *host = pci_host_bridge_from_priv(port);
+       unsigned int table_index = 0;
+       int err;
+       u32 val;
+
+       /* Set as RC mode */
+       val = readl_relaxed(port->base + PCIE_SETTING_REG);
+       val |= PCIE_RC_MODE;
+       writel_relaxed(val, port->base + PCIE_SETTING_REG);
+
+       /* Set class code */
+       val = readl_relaxed(port->base + PCIE_PCI_IDS_1);
+       val &= ~GENMASK(31, 8);
+       val |= PCI_CLASS(PCI_CLASS_BRIDGE_PCI << 8);
+       writel_relaxed(val, port->base + PCIE_PCI_IDS_1);
+
+       /* Mask all INTx interrupts */
+       val = readl_relaxed(port->base + PCIE_INT_ENABLE_REG);
+       val &= ~PCIE_INTX_ENABLE;
+       writel_relaxed(val, port->base + PCIE_INT_ENABLE_REG);
+
+       /* Assert all reset signals */
+       val = readl_relaxed(port->base + PCIE_RST_CTRL_REG);
+       val |= PCIE_MAC_RSTB | PCIE_PHY_RSTB | PCIE_BRG_RSTB | PCIE_PE_RSTB;
+       writel_relaxed(val, port->base + PCIE_RST_CTRL_REG);
+
+       /*
+        * Described in PCIe CEM specification setctions 2.2 (PERST# Signal)
+        * and 2.2.1 (Initial Power-Up (G3 to S0)).
+        * The deassertion of PERST# should be delayed 100ms (TPVPERL)
+        * for the power and clock to become stable.
+        */
+       msleep(100);
+
+       /* De-assert reset signals */
+       val &= ~(PCIE_MAC_RSTB | PCIE_PHY_RSTB | PCIE_BRG_RSTB | PCIE_PE_RSTB);
+       writel_relaxed(val, port->base + PCIE_RST_CTRL_REG);
+
+       /* Check if the link is up or not */
+       err = readl_poll_timeout(port->base + PCIE_LINK_STATUS_REG, val,
+                                !!(val & PCIE_PORT_LINKUP), 20,
+                                PCI_PM_D3COLD_WAIT * USEC_PER_MSEC);
+       if (err) {
+               val = readl_relaxed(port->base + PCIE_LTSSM_STATUS_REG);
+               dev_err(port->dev, "PCIe link down, ltssm reg val: %#x\n", val);
+               return err;
+       }
+
+       mtk_pcie_enable_msi(port);
+
+       /* Set PCIe translation windows */
+       resource_list_for_each_entry(entry, &host->windows) {
+               struct resource *res = entry->res;
+               unsigned long type = resource_type(res);
+               resource_size_t cpu_addr;
+               resource_size_t pci_addr;
+               resource_size_t size;
+               const char *range_type;
+
+               if (type == IORESOURCE_IO) {
+                       cpu_addr = pci_pio_to_address(res->start);
+                       range_type = "IO";
+               } else if (type == IORESOURCE_MEM) {
+                       cpu_addr = res->start;
+                       range_type = "MEM";
+               } else {
+                       continue;
+               }
+
+               pci_addr = res->start - entry->offset;
+               size = resource_size(res);
+               err = mtk_pcie_set_trans_table(port, cpu_addr, pci_addr, size,
+                                              type, table_index);
+               if (err)
+                       return err;
+
+               dev_dbg(port->dev, "set %s trans window[%d]: cpu_addr = %#llx, pci_addr = %#llx, size = %#llx\n",
+                       range_type, table_index, (unsigned long long)cpu_addr,
+                       (unsigned long long)pci_addr, (unsigned long long)size);
+
+               table_index++;
+       }
+
+       return 0;
+}
+
+static int mtk_pcie_set_affinity(struct irq_data *data,
+                                const struct cpumask *mask, bool force)
+{
+       return -EINVAL;
+}
+
+static void mtk_pcie_msi_irq_mask(struct irq_data *data)
+{
+       pci_msi_mask_irq(data);
+       irq_chip_mask_parent(data);
+}
+
+static void mtk_pcie_msi_irq_unmask(struct irq_data *data)
+{
+       pci_msi_unmask_irq(data);
+       irq_chip_unmask_parent(data);
+}
+
+static struct irq_chip mtk_msi_irq_chip = {
+       .irq_ack = irq_chip_ack_parent,
+       .irq_mask = mtk_pcie_msi_irq_mask,
+       .irq_unmask = mtk_pcie_msi_irq_unmask,
+       .name = "MSI",
+};
+
+static struct msi_domain_info mtk_msi_domain_info = {
+       .flags  = (MSI_FLAG_USE_DEF_DOM_OPS | MSI_FLAG_USE_DEF_CHIP_OPS |
+                  MSI_FLAG_PCI_MSIX | MSI_FLAG_MULTI_PCI_MSI),
+       .chip   = &mtk_msi_irq_chip,
+};
+
+static void mtk_compose_msi_msg(struct irq_data *data, struct msi_msg *msg)
+{
+       struct mtk_msi_set *msi_set = irq_data_get_irq_chip_data(data);
+       struct mtk_pcie_port *port = data->domain->host_data;
+       unsigned long hwirq;
+
+       hwirq = data->hwirq % PCIE_MSI_IRQS_PER_SET;
+
+       msg->address_hi = upper_32_bits(msi_set->msg_addr);
+       msg->address_lo = lower_32_bits(msi_set->msg_addr);
+       msg->data = hwirq;
+       dev_dbg(port->dev, "msi#%#lx address_hi %#x address_lo %#x data %d\n",
+               hwirq, msg->address_hi, msg->address_lo, msg->data);
+}
+
+static void mtk_msi_bottom_irq_ack(struct irq_data *data)
+{
+       struct mtk_msi_set *msi_set = irq_data_get_irq_chip_data(data);
+       unsigned long hwirq;
+
+       hwirq = data->hwirq % PCIE_MSI_IRQS_PER_SET;
+
+       writel_relaxed(BIT(hwirq), msi_set->base + PCIE_MSI_SET_STATUS_OFFSET);
+}
+
+static void mtk_msi_bottom_irq_mask(struct irq_data *data)
+{
+       struct mtk_msi_set *msi_set = irq_data_get_irq_chip_data(data);
+       struct mtk_pcie_port *port = data->domain->host_data;
+       unsigned long hwirq, flags;
+       u32 val;
+
+       hwirq = data->hwirq % PCIE_MSI_IRQS_PER_SET;
+
+       raw_spin_lock_irqsave(&port->irq_lock, flags);
+       val = readl_relaxed(msi_set->base + PCIE_MSI_SET_ENABLE_OFFSET);
+       val &= ~BIT(hwirq);
+       writel_relaxed(val, msi_set->base + PCIE_MSI_SET_ENABLE_OFFSET);
+       raw_spin_unlock_irqrestore(&port->irq_lock, flags);
+}
+
+static void mtk_msi_bottom_irq_unmask(struct irq_data *data)
+{
+       struct mtk_msi_set *msi_set = irq_data_get_irq_chip_data(data);
+       struct mtk_pcie_port *port = data->domain->host_data;
+       unsigned long hwirq, flags;
+       u32 val;
+
+       hwirq = data->hwirq % PCIE_MSI_IRQS_PER_SET;
+
+       raw_spin_lock_irqsave(&port->irq_lock, flags);
+       val = readl_relaxed(msi_set->base + PCIE_MSI_SET_ENABLE_OFFSET);
+       val |= BIT(hwirq);
+       writel_relaxed(val, msi_set->base + PCIE_MSI_SET_ENABLE_OFFSET);
+       raw_spin_unlock_irqrestore(&port->irq_lock, flags);
+}
+
+static struct irq_chip mtk_msi_bottom_irq_chip = {
+       .irq_ack                = mtk_msi_bottom_irq_ack,
+       .irq_mask               = mtk_msi_bottom_irq_mask,
+       .irq_unmask             = mtk_msi_bottom_irq_unmask,
+       .irq_compose_msi_msg    = mtk_compose_msi_msg,
+       .irq_set_affinity       = mtk_pcie_set_affinity,
+       .name                   = "MSI",
+};
+
+static int mtk_msi_bottom_domain_alloc(struct irq_domain *domain,
+                                      unsigned int virq, unsigned int nr_irqs,
+                                      void *arg)
+{
+       struct mtk_pcie_port *port = domain->host_data;
+       struct mtk_msi_set *msi_set;
+       int i, hwirq, set_idx;
+
+       mutex_lock(&port->lock);
+
+       hwirq = bitmap_find_free_region(port->msi_irq_in_use, PCIE_MSI_IRQS_NUM,
+                                       order_base_2(nr_irqs));
+
+       mutex_unlock(&port->lock);
+
+       if (hwirq < 0)
+               return -ENOSPC;
+
+       set_idx = hwirq / PCIE_MSI_IRQS_PER_SET;
+       msi_set = &port->msi_sets[set_idx];
+
+       for (i = 0; i < nr_irqs; i++)
+               irq_domain_set_info(domain, virq + i, hwirq + i,
+                                   &mtk_msi_bottom_irq_chip, msi_set,
+                                   handle_edge_irq, NULL, NULL);
+
+       return 0;
+}
+
+static void mtk_msi_bottom_domain_free(struct irq_domain *domain,
+                                      unsigned int virq, unsigned int nr_irqs)
+{
+       struct mtk_pcie_port *port = domain->host_data;
+       struct irq_data *data = irq_domain_get_irq_data(domain, virq);
+
+       mutex_lock(&port->lock);
+
+       bitmap_release_region(port->msi_irq_in_use, data->hwirq,
+                             order_base_2(nr_irqs));
+
+       mutex_unlock(&port->lock);
+
+       irq_domain_free_irqs_common(domain, virq, nr_irqs);
+}
+
+static const struct irq_domain_ops mtk_msi_bottom_domain_ops = {
+       .alloc = mtk_msi_bottom_domain_alloc,
+       .free = mtk_msi_bottom_domain_free,
+};
+
+static void mtk_intx_mask(struct irq_data *data)
+{
+       struct mtk_pcie_port *port = irq_data_get_irq_chip_data(data);
+       unsigned long flags;
+       u32 val;
+
+       raw_spin_lock_irqsave(&port->irq_lock, flags);
+       val = readl_relaxed(port->base + PCIE_INT_ENABLE_REG);
+       val &= ~BIT(data->hwirq + PCIE_INTX_SHIFT);
+       writel_relaxed(val, port->base + PCIE_INT_ENABLE_REG);
+       raw_spin_unlock_irqrestore(&port->irq_lock, flags);
+}
+
+static void mtk_intx_unmask(struct irq_data *data)
+{
+       struct mtk_pcie_port *port = irq_data_get_irq_chip_data(data);
+       unsigned long flags;
+       u32 val;
+
+       raw_spin_lock_irqsave(&port->irq_lock, flags);
+       val = readl_relaxed(port->base + PCIE_INT_ENABLE_REG);
+       val |= BIT(data->hwirq + PCIE_INTX_SHIFT);
+       writel_relaxed(val, port->base + PCIE_INT_ENABLE_REG);
+       raw_spin_unlock_irqrestore(&port->irq_lock, flags);
+}
+
+/**
+ * mtk_intx_eoi() - Clear INTx IRQ status at the end of interrupt
+ * @data: pointer to chip specific data
+ *
+ * As an emulated level IRQ, its interrupt status will remain
+ * until the corresponding de-assert message is received; hence that
+ * the status can only be cleared when the interrupt has been serviced.
+ */
+static void mtk_intx_eoi(struct irq_data *data)
+{
+       struct mtk_pcie_port *port = irq_data_get_irq_chip_data(data);
+       unsigned long hwirq;
+
+       hwirq = data->hwirq + PCIE_INTX_SHIFT;
+       writel_relaxed(BIT(hwirq), port->base + PCIE_INT_STATUS_REG);
+}
+
+static struct irq_chip mtk_intx_irq_chip = {
+       .irq_mask               = mtk_intx_mask,
+       .irq_unmask             = mtk_intx_unmask,
+       .irq_eoi                = mtk_intx_eoi,
+       .irq_set_affinity       = mtk_pcie_set_affinity,
+       .name                   = "INTx",
+};
+
+static int mtk_pcie_intx_map(struct irq_domain *domain, unsigned int irq,
+                            irq_hw_number_t hwirq)
+{
+       irq_set_chip_data(irq, domain->host_data);
+       irq_set_chip_and_handler_name(irq, &mtk_intx_irq_chip,
+                                     handle_fasteoi_irq, "INTx");
+       return 0;
+}
+
+static const struct irq_domain_ops intx_domain_ops = {
+       .map = mtk_pcie_intx_map,
+};
+
+static int mtk_pcie_init_irq_domains(struct mtk_pcie_port *port)
+{
+       struct device *dev = port->dev;
+       struct device_node *intc_node, *node = dev->of_node;
+       int ret;
+
+       raw_spin_lock_init(&port->irq_lock);
+
+       /* Setup INTx */
+       intc_node = of_get_child_by_name(node, "interrupt-controller");
+       if (!intc_node) {
+               dev_err(dev, "missing interrupt-controller node\n");
+               return -ENODEV;
+       }
+
+       port->intx_domain = irq_domain_add_linear(intc_node, PCI_NUM_INTX,
+                                                 &intx_domain_ops, port);
+       if (!port->intx_domain) {
+               dev_err(dev, "failed to create INTx IRQ domain\n");
+               return -ENODEV;
+       }
+
+       /* Setup MSI */
+       mutex_init(&port->lock);
+
+       port->msi_bottom_domain = irq_domain_add_linear(node, PCIE_MSI_IRQS_NUM,
+                                 &mtk_msi_bottom_domain_ops, port);
+       if (!port->msi_bottom_domain) {
+               dev_err(dev, "failed to create MSI bottom domain\n");
+               ret = -ENODEV;
+               goto err_msi_bottom_domain;
+       }
+
+       port->msi_domain = pci_msi_create_irq_domain(dev->fwnode,
+                                                    &mtk_msi_domain_info,
+                                                    port->msi_bottom_domain);
+       if (!port->msi_domain) {
+               dev_err(dev, "failed to create MSI domain\n");
+               ret = -ENODEV;
+               goto err_msi_domain;
+       }
+
+       return 0;
+
+err_msi_domain:
+       irq_domain_remove(port->msi_bottom_domain);
+err_msi_bottom_domain:
+       irq_domain_remove(port->intx_domain);
+
+       return ret;
+}
+
+static void mtk_pcie_irq_teardown(struct mtk_pcie_port *port)
+{
+       irq_set_chained_handler_and_data(port->irq, NULL, NULL);
+
+       if (port->intx_domain)
+               irq_domain_remove(port->intx_domain);
+
+       if (port->msi_domain)
+               irq_domain_remove(port->msi_domain);
+
+       if (port->msi_bottom_domain)
+               irq_domain_remove(port->msi_bottom_domain);
+
+       irq_dispose_mapping(port->irq);
+}
+
+static void mtk_pcie_msi_handler(struct mtk_pcie_port *port, int set_idx)
+{
+       struct mtk_msi_set *msi_set = &port->msi_sets[set_idx];
+       unsigned long msi_enable, msi_status;
+       unsigned int virq;
+       irq_hw_number_t bit, hwirq;
+
+       msi_enable = readl_relaxed(msi_set->base + PCIE_MSI_SET_ENABLE_OFFSET);
+
+       do {
+               msi_status = readl_relaxed(msi_set->base +
+                                          PCIE_MSI_SET_STATUS_OFFSET);
+               msi_status &= msi_enable;
+               if (!msi_status)
+                       break;
+
+               for_each_set_bit(bit, &msi_status, PCIE_MSI_IRQS_PER_SET) {
+                       hwirq = bit + set_idx * PCIE_MSI_IRQS_PER_SET;
+                       virq = irq_find_mapping(port->msi_bottom_domain, hwirq);
+                       generic_handle_irq(virq);
+               }
+       } while (true);
+}
+
+static void mtk_pcie_irq_handler(struct irq_desc *desc)
+{
+       struct mtk_pcie_port *port = irq_desc_get_handler_data(desc);
+       struct irq_chip *irqchip = irq_desc_get_chip(desc);
+       unsigned long status;
+       unsigned int virq;
+       irq_hw_number_t irq_bit = PCIE_INTX_SHIFT;
+
+       chained_irq_enter(irqchip, desc);
+
+       status = readl_relaxed(port->base + PCIE_INT_STATUS_REG);
+       for_each_set_bit_from(irq_bit, &status, PCI_NUM_INTX +
+                             PCIE_INTX_SHIFT) {
+               virq = irq_find_mapping(port->intx_domain,
+                                       irq_bit - PCIE_INTX_SHIFT);
+               generic_handle_irq(virq);
+       }
+
+       irq_bit = PCIE_MSI_SHIFT;
+       for_each_set_bit_from(irq_bit, &status, PCIE_MSI_SET_NUM +
+                             PCIE_MSI_SHIFT) {
+               mtk_pcie_msi_handler(port, irq_bit - PCIE_MSI_SHIFT);
+
+               writel_relaxed(BIT(irq_bit), port->base + PCIE_INT_STATUS_REG);
+       }
+
+       chained_irq_exit(irqchip, desc);
+}
+
+static int mtk_pcie_setup_irq(struct mtk_pcie_port *port)
+{
+       struct device *dev = port->dev;
+       struct platform_device *pdev = to_platform_device(dev);
+       int err;
+
+       err = mtk_pcie_init_irq_domains(port);
+       if (err)
+               return err;
+
+       port->irq = platform_get_irq(pdev, 0);
+       if (port->irq < 0)
+               return port->irq;
+
+       irq_set_chained_handler_and_data(port->irq, mtk_pcie_irq_handler, port);
+
+       return 0;
+}
+
+static int mtk_pcie_parse_port(struct mtk_pcie_port *port)
+{
+       struct device *dev = port->dev;
+       struct platform_device *pdev = to_platform_device(dev);
+       struct resource *regs;
+       int ret;
+
+       regs = platform_get_resource_byname(pdev, IORESOURCE_MEM, "pcie-mac");
+       if (!regs)
+               return -EINVAL;
+       port->base = devm_ioremap_resource(dev, regs);
+       if (IS_ERR(port->base)) {
+               dev_err(dev, "failed to map register base\n");
+               return PTR_ERR(port->base);
+       }
+
+       port->reg_base = regs->start;
+
+       port->phy_reset = devm_reset_control_get_optional_exclusive(dev, "phy");
+       if (IS_ERR(port->phy_reset)) {
+               ret = PTR_ERR(port->phy_reset);
+               if (ret != -EPROBE_DEFER)
+                       dev_err(dev, "failed to get PHY reset\n");
+
+               return ret;
+       }
+
+       port->mac_reset = devm_reset_control_get_optional_exclusive(dev, "mac");
+       if (IS_ERR(port->mac_reset)) {
+               ret = PTR_ERR(port->mac_reset);
+               if (ret != -EPROBE_DEFER)
+                       dev_err(dev, "failed to get MAC reset\n");
+
+               return ret;
+       }
+
+       port->phy = devm_phy_optional_get(dev, "pcie-phy");
+       if (IS_ERR(port->phy)) {
+               ret = PTR_ERR(port->phy);
+               if (ret != -EPROBE_DEFER)
+                       dev_err(dev, "failed to get PHY\n");
+
+               return ret;
+       }
+
+       port->num_clks = devm_clk_bulk_get_all(dev, &port->clks);
+       if (port->num_clks < 0) {
+               dev_err(dev, "failed to get clocks\n");
+               return port->num_clks;
+       }
+
+       return 0;
+}
+
+static int mtk_pcie_power_up(struct mtk_pcie_port *port)
+{
+       struct device *dev = port->dev;
+       int err;
+
+       /* PHY power on and enable pipe clock */
+       reset_control_deassert(port->phy_reset);
+
+       err = phy_init(port->phy);
+       if (err) {
+               dev_err(dev, "failed to initialize PHY\n");
+               goto err_phy_init;
+       }
+
+       err = phy_power_on(port->phy);
+       if (err) {
+               dev_err(dev, "failed to power on PHY\n");
+               goto err_phy_on;
+       }
+
+       /* MAC power on and enable transaction layer clocks */
+       reset_control_deassert(port->mac_reset);
+
+       pm_runtime_enable(dev);
+       pm_runtime_get_sync(dev);
+
+       err = clk_bulk_prepare_enable(port->num_clks, port->clks);
+       if (err) {
+               dev_err(dev, "failed to enable clocks\n");
+               goto err_clk_init;
+       }
+
+       return 0;
+
+err_clk_init:
+       pm_runtime_put_sync(dev);
+       pm_runtime_disable(dev);
+       reset_control_assert(port->mac_reset);
+       phy_power_off(port->phy);
+err_phy_on:
+       phy_exit(port->phy);
+err_phy_init:
+       reset_control_assert(port->phy_reset);
+
+       return err;
+}
+
+static void mtk_pcie_power_down(struct mtk_pcie_port *port)
+{
+       clk_bulk_disable_unprepare(port->num_clks, port->clks);
+
+       pm_runtime_put_sync(port->dev);
+       pm_runtime_disable(port->dev);
+       reset_control_assert(port->mac_reset);
+
+       phy_power_off(port->phy);
+       phy_exit(port->phy);
+       reset_control_assert(port->phy_reset);
+}
+
+static int mtk_pcie_setup(struct mtk_pcie_port *port)
+{
+       int err;
+
+       err = mtk_pcie_parse_port(port);
+       if (err)
+               return err;
+
+       /* Don't touch the hardware registers before power up */
+       err = mtk_pcie_power_up(port);
+       if (err)
+               return err;
+
+       /* Try link up */
+       err = mtk_pcie_startup_port(port);
+       if (err)
+               goto err_setup;
+
+       err = mtk_pcie_setup_irq(port);
+       if (err)
+               goto err_setup;
+
+       return 0;
+
+err_setup:
+       mtk_pcie_power_down(port);
+
+       return err;
+}
+
+static int mtk_pcie_probe(struct platform_device *pdev)
+{
+       struct device *dev = &pdev->dev;
+       struct mtk_pcie_port *port;
+       struct pci_host_bridge *host;
+       int err;
+
+       host = devm_pci_alloc_host_bridge(dev, sizeof(*port));
+       if (!host)
+               return -ENOMEM;
+
+       port = pci_host_bridge_priv(host);
+
+       port->dev = dev;
+       platform_set_drvdata(pdev, port);
+
+       err = mtk_pcie_setup(port);
+       if (err)
+               return err;
+
+       host->ops = &mtk_pcie_ops;
+       host->sysdata = port;
+
+       err = pci_host_probe(host);
+       if (err) {
+               mtk_pcie_irq_teardown(port);
+               mtk_pcie_power_down(port);
+               return err;
+       }
+
+       return 0;
+}
+
+static int mtk_pcie_remove(struct platform_device *pdev)
+{
+       struct mtk_pcie_port *port = platform_get_drvdata(pdev);
+       struct pci_host_bridge *host = pci_host_bridge_from_priv(port);
+
+       pci_lock_rescan_remove();
+       pci_stop_root_bus(host->bus);
+       pci_remove_root_bus(host->bus);
+       pci_unlock_rescan_remove();
+
+       mtk_pcie_irq_teardown(port);
+       mtk_pcie_power_down(port);
+
+       return 0;
+}
+
+static void __maybe_unused mtk_pcie_irq_save(struct mtk_pcie_port *port)
+{
+       int i;
+
+       raw_spin_lock(&port->irq_lock);
+
+       port->saved_irq_state = readl_relaxed(port->base + PCIE_INT_ENABLE_REG);
+
+       for (i = 0; i < PCIE_MSI_SET_NUM; i++) {
+               struct mtk_msi_set *msi_set = &port->msi_sets[i];
+
+               msi_set->saved_irq_state = readl_relaxed(msi_set->base +
+                                          PCIE_MSI_SET_ENABLE_OFFSET);
+       }
+
+       raw_spin_unlock(&port->irq_lock);
+}
+
+static void __maybe_unused mtk_pcie_irq_restore(struct mtk_pcie_port *port)
+{
+       int i;
+
+       raw_spin_lock(&port->irq_lock);
+
+       writel_relaxed(port->saved_irq_state, port->base + PCIE_INT_ENABLE_REG);
+
+       for (i = 0; i < PCIE_MSI_SET_NUM; i++) {
+               struct mtk_msi_set *msi_set = &port->msi_sets[i];
+
+               writel_relaxed(msi_set->saved_irq_state,
+                              msi_set->base + PCIE_MSI_SET_ENABLE_OFFSET);
+       }
+
+       raw_spin_unlock(&port->irq_lock);
+}
+
+static int __maybe_unused mtk_pcie_turn_off_link(struct mtk_pcie_port *port)
+{
+       u32 val;
+
+       val = readl_relaxed(port->base + PCIE_ICMD_PM_REG);
+       val |= PCIE_TURN_OFF_LINK;
+       writel_relaxed(val, port->base + PCIE_ICMD_PM_REG);
+
+       /* Check the link is L2 */
+       return readl_poll_timeout(port->base + PCIE_LTSSM_STATUS_REG, val,
+                                 (PCIE_LTSSM_STATE(val) ==
+                                  PCIE_LTSSM_STATE_L2_IDLE), 20,
+                                  50 * USEC_PER_MSEC);
+}
+
+static int __maybe_unused mtk_pcie_suspend_noirq(struct device *dev)
+{
+       struct mtk_pcie_port *port = dev_get_drvdata(dev);
+       int err;
+       u32 val;
+
+       /* Trigger link to L2 state */
+       err = mtk_pcie_turn_off_link(port);
+       if (err) {
+               dev_err(port->dev, "cannot enter L2 state\n");
+               return err;
+       }
+
+       /* Pull down the PERST# pin */
+       val = readl_relaxed(port->base + PCIE_RST_CTRL_REG);
+       val |= PCIE_PE_RSTB;
+       writel_relaxed(val, port->base + PCIE_RST_CTRL_REG);
+
+       dev_dbg(port->dev, "entered L2 states successfully");
+
+       mtk_pcie_irq_save(port);
+       mtk_pcie_power_down(port);
+
+       return 0;
+}
+
+static int __maybe_unused mtk_pcie_resume_noirq(struct device *dev)
+{
+       struct mtk_pcie_port *port = dev_get_drvdata(dev);
+       int err;
+
+       err = mtk_pcie_power_up(port);
+       if (err)
+               return err;
+
+       err = mtk_pcie_startup_port(port);
+       if (err) {
+               mtk_pcie_power_down(port);
+               return err;
+       }
+
+       mtk_pcie_irq_restore(port);
+
+       return 0;
+}
+
+static const struct dev_pm_ops mtk_pcie_pm_ops = {
+       SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(mtk_pcie_suspend_noirq,
+                                     mtk_pcie_resume_noirq)
+};
+
+static const struct of_device_id mtk_pcie_of_match[] = {
+       { .compatible = "mediatek,mt8192-pcie" },
+       {},
+};
+
+static struct platform_driver mtk_pcie_driver = {
+       .probe = mtk_pcie_probe,
+       .remove = mtk_pcie_remove,
+       .driver = {
+               .name = "mtk-pcie",
+               .of_match_table = mtk_pcie_of_match,
+               .pm = &mtk_pcie_pm_ops,
+       },
+};
+
+module_platform_driver(mtk_pcie_driver);
+MODULE_LICENSE("GPL v2");
index 23548b5..ddbd2f8 100644 (file)
@@ -760,7 +760,7 @@ static struct pci_ops mtk_pcie_ops = {
 static int mtk_pcie_startup_port(struct mtk_pcie_port *port)
 {
        struct mtk_pcie *pcie = port->pcie;
-       u32 func = PCI_FUNC(port->slot << 3);
+       u32 func = PCI_FUNC(port->slot);
        u32 slot = PCI_SLOT(port->slot << 3);
        u32 val;
        int err;
@@ -1210,6 +1210,7 @@ static const struct of_device_id mtk_pcie_ids[] = {
        { .compatible = "mediatek,mt7629-pcie", .data = &mtk_pcie_soc_mt7629 },
        {},
 };
+MODULE_DEVICE_TABLE(of, mtk_pcie_ids);
 
 static struct platform_driver mtk_pcie_driver = {
        .probe = mtk_pcie_probe,
index b1845e5..3123a4f 100644 (file)
@@ -4052,6 +4052,7 @@ phys_addr_t pci_pio_to_address(unsigned long pio)
 
        return address;
 }
+EXPORT_SYMBOL_GPL(pci_pio_to_address);
 
 unsigned long __weak pci_address_to_pio(phys_addr_t address)
 {