On Wed, Aug 09, 2017 at 01:18:48PM +0530, Bhumika Goyal wrote: > Make these structures const as they are only stored in the ops field of > a pcie_port structure, which is of type const. > Done using Coccinelle. > > Signed-off-by: Bhumika Goyal <bhumirks@xxxxxxxxx> Applied to pci/host-hisi and pci/host-kirin for v4.14, thanks! > --- > drivers/pci/dwc/pcie-hisi.c | 2 +- > drivers/pci/dwc/pcie-kirin.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/dwc/pcie-hisi.c b/drivers/pci/dwc/pcie-hisi.c > index e51acee..518cd7e 100644 > --- a/drivers/pci/dwc/pcie-hisi.c > +++ b/drivers/pci/dwc/pcie-hisi.c > @@ -223,7 +223,7 @@ static int hisi_pcie_link_up(struct dw_pcie *pci) > return hisi_pcie->soc_ops->hisi_pcie_link_up(hisi_pcie); > } > > -static struct dw_pcie_host_ops hisi_pcie_host_ops = { > +static const struct dw_pcie_host_ops hisi_pcie_host_ops = { > .rd_own_conf = hisi_pcie_cfg_read, > .wr_own_conf = hisi_pcie_cfg_write, > }; > diff --git a/drivers/pci/dwc/pcie-kirin.c b/drivers/pci/dwc/pcie-kirin.c > index 33fddb9..a623f96 100644 > --- a/drivers/pci/dwc/pcie-kirin.c > +++ b/drivers/pci/dwc/pcie-kirin.c > @@ -441,7 +441,7 @@ static void kirin_pcie_host_init(struct pcie_port *pp) > .link_up = kirin_pcie_link_up, > }; > > -static struct dw_pcie_host_ops kirin_pcie_host_ops = { > +static const struct dw_pcie_host_ops kirin_pcie_host_ops = { > .rd_own_conf = kirin_pcie_rd_own_conf, > .wr_own_conf = kirin_pcie_wr_own_conf, > .host_init = kirin_pcie_host_init, > -- > 1.9.1 >