Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1715767

Re: [PATCH] PCI: dwc: make dw_pcie_host_ops const

Show all headers | View raw

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@gmail.com>

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
> 

Back to linux.kernel | Previous | Next | Find similar | Unroll thread

Thread

Re: [PATCH] PCI: dwc: make dw_pcie_host_ops const Bjorn Helgaas <helgaas@kernel.org> - 2017-08-19 23:30 +0200

csiph-web