| Message ID | 20200725091945.75176-1-weiyongjun1@huawei.com |
|---|---|
| State | New |
| Headers | show |
| Series | [-next] PCI: cadence: Fix unused-but-set-variable warning | expand |
On Sat, Jul 25, 2020 at 05:19:45PM +0800, Wei Yongjun wrote: > Gcc report warning as followings: > > drivers/pci/controller/cadence/pcie-cadence-ep.c:390:33: warning: > variable 'vec_ctrl' set but not used [-Wunused-but-set-variable] > 390 | u32 tbl_offset, msg_data, reg, vec_ctrl; > | ^~~~~~~~ > > This variable is not used so this commit removing it. > > Fixes: dae15ff2c7a9 ("PCI: cadence: Add MSI-X support to Endpoint driver") > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> > --- > drivers/pci/controller/cadence/pcie-cadence-ep.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Squashed in the original commit, thanks. Lorenzo > diff --git a/drivers/pci/controller/cadence/pcie-cadence-ep.c b/drivers/pci/controller/cadence/pcie-cadence-ep.c > index 87c76341eab4..ec1306da301f 100644 > --- a/drivers/pci/controller/cadence/pcie-cadence-ep.c > +++ b/drivers/pci/controller/cadence/pcie-cadence-ep.c > @@ -387,7 +387,7 @@ static int cdns_pcie_ep_send_msix_irq(struct cdns_pcie_ep *ep, u8 fn, > u16 interrupt_num) > { > u32 cap = CDNS_PCIE_EP_FUNC_MSIX_CAP_OFFSET; > - u32 tbl_offset, msg_data, reg, vec_ctrl; > + u32 tbl_offset, msg_data, reg; > struct cdns_pcie *pcie = &ep->pcie; > struct pci_epf_msix_tbl *msix_tbl; > struct cdns_pcie_epf *epf; > @@ -410,7 +410,6 @@ static int cdns_pcie_ep_send_msix_irq(struct cdns_pcie_ep *ep, u8 fn, > msix_tbl = epf->epf_bar[bir]->addr + tbl_offset; > msg_addr = msix_tbl[(interrupt_num - 1)].msg_addr; > msg_data = msix_tbl[(interrupt_num - 1)].msg_data; > - vec_ctrl = msix_tbl[(interrupt_num - 1)].vector_ctrl; > > /* Set the outbound region if needed. */ > if (ep->irq_pci_addr != (msg_addr & ~pci_addr_mask) || >
diff --git a/drivers/pci/controller/cadence/pcie-cadence-ep.c b/drivers/pci/controller/cadence/pcie-cadence-ep.c index 87c76341eab4..ec1306da301f 100644 --- a/drivers/pci/controller/cadence/pcie-cadence-ep.c +++ b/drivers/pci/controller/cadence/pcie-cadence-ep.c @@ -387,7 +387,7 @@ static int cdns_pcie_ep_send_msix_irq(struct cdns_pcie_ep *ep, u8 fn, u16 interrupt_num) { u32 cap = CDNS_PCIE_EP_FUNC_MSIX_CAP_OFFSET; - u32 tbl_offset, msg_data, reg, vec_ctrl; + u32 tbl_offset, msg_data, reg; struct cdns_pcie *pcie = &ep->pcie; struct pci_epf_msix_tbl *msix_tbl; struct cdns_pcie_epf *epf; @@ -410,7 +410,6 @@ static int cdns_pcie_ep_send_msix_irq(struct cdns_pcie_ep *ep, u8 fn, msix_tbl = epf->epf_bar[bir]->addr + tbl_offset; msg_addr = msix_tbl[(interrupt_num - 1)].msg_addr; msg_data = msix_tbl[(interrupt_num - 1)].msg_data; - vec_ctrl = msix_tbl[(interrupt_num - 1)].vector_ctrl; /* Set the outbound region if needed. */ if (ep->irq_pci_addr != (msg_addr & ~pci_addr_mask) ||
Gcc report warning as followings: drivers/pci/controller/cadence/pcie-cadence-ep.c:390:33: warning: variable 'vec_ctrl' set but not used [-Wunused-but-set-variable] 390 | u32 tbl_offset, msg_data, reg, vec_ctrl; | ^~~~~~~~ This variable is not used so this commit removing it. Fixes: dae15ff2c7a9 ("PCI: cadence: Add MSI-X support to Endpoint driver") Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com> --- drivers/pci/controller/cadence/pcie-cadence-ep.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)