Messages in this thread Patch in this message | | | From | "Zouwei (Samuel)" <> | Subject | 答复: [PATCH -next] PCI: dwc: Make hisi pcie platform ops static | Date | Thu, 23 Apr 2020 03:09:20 +0000 |
| |
Hi Wang,
Thanks for your review and reply. I will modify and send the v2 soon.
Thanks Zou Wei -----邮件原件----- 发件人: Wangzhou (B) 发送时间: 2020年4月23日 10:59 收件人: Zouwei (Samuel) <zou_wei@huawei.com>; lorenzo.pieralisi@arm.com; amurray@thegoodpenguin.co.uk; bhelgaas@google.com 抄送: linux-pci@vger.kernel.org; linux-kernel@vger.kernel.org 主题: Re: [PATCH -next] PCI: dwc: Make hisi_pcie_platform_ops static
On 2020/4/22 17:47, Zou Wei wrote: > Fix the following sparse warning: > > drivers/pci/controller/dwc/pcie-hisi.c:365:21: warning: > symbol 'hisi_pcie_platform_ops' was not declared. Should it be static? > > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Zou Wei <zou_wei@huawei.com> > --- > drivers/pci/controller/dwc/pcie-hisi.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/pci/controller/dwc/pcie-hisi.c b/drivers/pci/controller/dwc/pcie-hisi.c > index 6d9e1b2..b440f40 100644 > --- a/drivers/pci/controller/dwc/pcie-hisi.c > +++ b/drivers/pci/controller/dwc/pcie-hisi.c > @@ -362,7 +362,8 @@ static int hisi_pcie_platform_init(struct pci_config_window *cfg) > return 0; > } > > -struct pci_ecam_ops hisi_pcie_platform_ops = { > +static struct pci_ecam_ops hisi_pcie_platform_ops = { > + }
why adding "}"? BTW, static is OK here.
> .bus_shift = 20, > .init = hisi_pcie_platform_init, > .pci_ops = { >
| |