PCI: dwc: Constify dw_pcie_host_ops structures
authorJisheng Zhang <jszhang@marvell.com>
Mon, 5 Jun 2017 08:53:46 +0000 (16:53 +0800)
committerBjorn Helgaas <bhelgaas@google.com>
Sun, 2 Jul 2017 23:38:50 +0000 (18:38 -0500)
The dw_pcie_host_ops structures are never modified.  Constify these
structures such that these can be write-protected.

Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Jingoo Han <jingoohan1@gmail.com>
Acked-by: Kishon Vijay Abraham I <kishon@ti.com>
drivers/pci/dwc/pci-dra7xx.c
drivers/pci/dwc/pci-exynos.c
drivers/pci/dwc/pci-imx6.c
drivers/pci/dwc/pci-keystone.c
drivers/pci/dwc/pci-layerscape.c
drivers/pci/dwc/pcie-armada8k.c
drivers/pci/dwc/pcie-artpec6.c
drivers/pci/dwc/pcie-designware-plat.c
drivers/pci/dwc/pcie-designware.h
drivers/pci/dwc/pcie-qcom.c
drivers/pci/dwc/pcie-spear13xx.c

index 8decf46cf5257228985ae722ec3ddaa3ca9b2184..e4166032b3c684d22e87dc662ca8c29fe7c2111e 100644 (file)
@@ -208,7 +208,7 @@ static void dra7xx_pcie_host_init(struct pcie_port *pp)
        dra7xx_pcie_enable_interrupts(dra7xx);
 }
 
-static struct dw_pcie_host_ops dra7xx_pcie_host_ops = {
+static const struct dw_pcie_host_ops dra7xx_pcie_host_ops = {
        .host_init = dra7xx_pcie_host_init,
 };
 
index 546082ad5a3f767d97af9d6015023aded654032e..c78c065525900d57127ee12a73373ce61e4d0b88 100644 (file)
@@ -590,7 +590,7 @@ static void exynos_pcie_host_init(struct pcie_port *pp)
        exynos_pcie_enable_interrupts(ep);
 }
 
-static struct dw_pcie_host_ops exynos_pcie_host_ops = {
+static const struct dw_pcie_host_ops exynos_pcie_host_ops = {
        .rd_own_conf = exynos_pcie_rd_own_conf,
        .wr_own_conf = exynos_pcie_wr_own_conf,
        .host_init = exynos_pcie_host_init,
index a98cba55c7f02e670436787c011a85a8772f5109..fb4816088a7aaf6bceebba0cb4f3c35697f7e7c5 100644 (file)
@@ -602,7 +602,7 @@ static int imx6_pcie_link_up(struct dw_pcie *pci)
                        PCIE_PHY_DEBUG_R1_XMLH_LINK_UP;
 }
 
-static struct dw_pcie_host_ops imx6_pcie_host_ops = {
+static const struct dw_pcie_host_ops imx6_pcie_host_ops = {
        .host_init = imx6_pcie_host_init,
 };
 
index fcc9723bad6e01ac7aee04ff5a406510aade5b6d..4783cec1f78d4f645d347602547215b7012a1dac 100644 (file)
@@ -291,7 +291,7 @@ static void __init ks_pcie_host_init(struct pcie_port *pp)
                        "Asynchronous external abort");
 }
 
-static struct dw_pcie_host_ops keystone_pcie_host_ops = {
+static const struct dw_pcie_host_ops keystone_pcie_host_ops = {
        .rd_other_conf = ks_dw_pcie_rd_other_conf,
        .wr_other_conf = ks_dw_pcie_wr_other_conf,
        .host_init = ks_pcie_host_init,
index 27d638c4e134316676f9984a4596b6275c23f133..fd861289ad8bc53a3e474ca2b922060bfddb7493 100644 (file)
@@ -39,7 +39,7 @@ struct ls_pcie_drvdata {
        u32 lut_offset;
        u32 ltssm_shift;
        u32 lut_dbg;
-       struct dw_pcie_host_ops *ops;
+       const struct dw_pcie_host_ops *ops;
        const struct dw_pcie_ops *dw_pcie_ops;
 };
 
@@ -185,12 +185,12 @@ static int ls_pcie_msi_host_init(struct pcie_port *pp,
        return 0;
 }
 
-static struct dw_pcie_host_ops ls1021_pcie_host_ops = {
+static const struct dw_pcie_host_ops ls1021_pcie_host_ops = {
        .host_init = ls1021_pcie_host_init,
        .msi_host_init = ls_pcie_msi_host_init,
 };
 
-static struct dw_pcie_host_ops ls_pcie_host_ops = {
+static const struct dw_pcie_host_ops ls_pcie_host_ops = {
        .host_init = ls_pcie_host_init,
        .msi_host_init = ls_pcie_msi_host_init,
 };
index 495b023042b3ac2b861515aea4245be91ecaefeb..ea8f34af6a851a5b0e721f8aca8dfb32c181a827 100644 (file)
@@ -160,7 +160,7 @@ static irqreturn_t armada8k_pcie_irq_handler(int irq, void *arg)
        return IRQ_HANDLED;
 }
 
-static struct dw_pcie_host_ops armada8k_pcie_host_ops = {
+static const struct dw_pcie_host_ops armada8k_pcie_host_ops = {
        .host_init = armada8k_pcie_host_init,
 };
 
index 82a04acc42fdc228e8e500efd4fa082494b7930c..01c6f7823672520bc6e4e18f9afc4a1d90c32924 100644 (file)
@@ -184,7 +184,7 @@ static void artpec6_pcie_host_init(struct pcie_port *pp)
        artpec6_pcie_enable_interrupts(artpec6_pcie);
 }
 
-static struct dw_pcie_host_ops artpec6_pcie_host_ops = {
+static const struct dw_pcie_host_ops artpec6_pcie_host_ops = {
        .host_init = artpec6_pcie_host_init,
 };
 
index a9865d91b43c4dfcc4a74d45b754da6bb9a5c9a0..091b4e7ad05959e27757326e592b78c9796bb709 100644 (file)
@@ -46,7 +46,7 @@ static void dw_plat_pcie_host_init(struct pcie_port *pp)
                dw_pcie_msi_init(pp);
 }
 
-static struct dw_pcie_host_ops dw_plat_pcie_host_ops = {
+static const struct dw_pcie_host_ops dw_plat_pcie_host_ops = {
        .host_init = dw_plat_pcie_host_init,
 };
 
index c6a840575796ff1bcc2617c4eb40f003668a3a4e..b4d2a89f8e58591c6a08ea4e5204d56ddfc00466 100644 (file)
@@ -162,7 +162,7 @@ struct pcie_port {
        struct resource         *mem;
        struct resource         *busn;
        int                     irq;
-       struct dw_pcie_host_ops *ops;
+       const struct dw_pcie_host_ops *ops;
        int                     msi_irq;
        struct irq_domain       *irq_domain;
        unsigned long           msi_data;
index 96ee527c1ed2e0344b3bf88506e3faaf9d0838ea..39e8c009571587ab20bccc5059920bbb3503e2c7 100644 (file)
@@ -634,7 +634,7 @@ static int qcom_pcie_rd_own_conf(struct pcie_port *pp, int where, int size,
        return dw_pcie_read(pci->dbi_base + where, size, val);
 }
 
-static struct dw_pcie_host_ops qcom_pcie_dw_ops = {
+static const struct dw_pcie_host_ops qcom_pcie_dw_ops = {
        .host_init = qcom_pcie_host_init,
        .rd_own_conf = qcom_pcie_rd_own_conf,
 };
index 8ff36b3dbbdf7ebdb8f06fdaea334033467bf31d..80897291e0fbf976f4d6ad1760f06c08152a9f6c 100644 (file)
@@ -186,7 +186,7 @@ static void spear13xx_pcie_host_init(struct pcie_port *pp)
        spear13xx_pcie_enable_interrupts(spear13xx_pcie);
 }
 
-static struct dw_pcie_host_ops spear13xx_pcie_host_ops = {
+static const struct dw_pcie_host_ops spear13xx_pcie_host_ops = {
        .host_init = spear13xx_pcie_host_init,
 };