PCI: designware: Simplify pcie_host_ops.readl_rc() and .writel_rc() interfaces
authorBjorn Helgaas <bhelgaas@google.com>
Thu, 6 Oct 2016 18:25:46 +0000 (13:25 -0500)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 11 Oct 2016 13:29:25 +0000 (08:29 -0500)
The struct pcie_host_ops.readl_rc() and .writel_rc() function pointers
allow a driver to override the default DesignWare register accessors.

Make the signature of the override functions the same as the default
accessors.  This makes the default dw_pcie_readl_rc() and the corresponding
override more structurally similar: both will compute the final register
address with "pp->dbi_base + reg".  Previously dw_pcie_readl_rc() computed
the address and passed it to the override.

No functional change intended.

Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/host/pci-exynos.c
drivers/pci/host/pcie-designware.c
drivers/pci/host/pcie-designware.h

index 2e2d7f00b9e8ad0c7ed007d00d64db86f7b76f8f..b29e9d64d41b0ff9e3c1e452e3a49d20f20325ce 100644 (file)
@@ -425,22 +425,20 @@ static void exynos_pcie_enable_interrupts(struct pcie_port *pp)
                exynos_pcie_msi_init(pp);
 }
 
-static inline u32 exynos_pcie_readl_rc(struct pcie_port *pp,
-                                      void __iomem *dbi_base)
+static inline u32 exynos_pcie_readl_rc(struct pcie_port *pp, u32 reg)
 {
        u32 val;
 
        exynos_pcie_sideband_dbi_r_mode(pp, true);
-       val = readl(dbi_base);
+       val = readl(pp->dbi_base + reg);
        exynos_pcie_sideband_dbi_r_mode(pp, false);
        return val;
 }
 
-static inline void exynos_pcie_writel_rc(struct pcie_port *pp,
-                                       u32 val, void __iomem *dbi_base)
+static inline void exynos_pcie_writel_rc(struct pcie_port *pp, u32 val, u32 reg)
 {
        exynos_pcie_sideband_dbi_w_mode(pp, true);
-       writel(val, dbi_base);
+       writel(val, pp->dbi_base + reg);
        exynos_pcie_sideband_dbi_w_mode(pp, false);
 }
 
index 7ce4f7510bb5f041024c586413383fb0cc1fca75..6a28eb1e2ef9b8980db54aa283f29c5143c5fcdc 100644 (file)
@@ -144,7 +144,7 @@ int dw_pcie_cfg_write(void __iomem *addr, int size, u32 val)
 static inline u32 dw_pcie_readl_rc(struct pcie_port *pp, u32 reg)
 {
        if (pp->ops->readl_rc)
-               return pp->ops->readl_rc(pp, pp->dbi_base + reg);
+               return pp->ops->readl_rc(pp, reg);
 
        return readl(pp->dbi_base + reg);
 }
@@ -152,7 +152,7 @@ static inline u32 dw_pcie_readl_rc(struct pcie_port *pp, u32 reg)
 static inline void dw_pcie_writel_rc(struct pcie_port *pp, u32 val, u32 reg)
 {
        if (pp->ops->writel_rc)
-               pp->ops->writel_rc(pp, val, pp->dbi_base + reg);
+               pp->ops->writel_rc(pp, val, reg);
        else
                writel(val, pp->dbi_base + reg);
 }
index c8e5bc647f4958d0cf66643d8280bc7f41713ab7..60cbc682618f2f7a889b2dbafe65603937b48bd5 100644 (file)
@@ -54,9 +54,8 @@ struct pcie_port {
 };
 
 struct pcie_host_ops {
-       u32 (*readl_rc)(struct pcie_port *pp, void __iomem *dbi_base);
-       void (*writel_rc)(struct pcie_port *pp,
-                       u32 val, void __iomem *dbi_base);
+       u32 (*readl_rc)(struct pcie_port *pp, u32 reg);
+       void (*writel_rc)(struct pcie_port *pp, u32 val, u32 reg);
        int (*rd_own_conf)(struct pcie_port *pp, int where, int size, u32 *val);
        int (*wr_own_conf)(struct pcie_port *pp, int where, int size, u32 val);
        int (*rd_other_conf)(struct pcie_port *pp, struct pci_bus *bus,