Commit e19485dc authored by Xu Yilun's avatar Xu Yilun Committed by Moritz Fischer

fpga: dfl: pci: reduce the scope of variable 'ret'

This is to fix lkp cppcheck warnings:

 drivers/fpga/dfl-pci.c:230:6: warning: The scope of the variable 'ret' can be reduced. [variableScope]
    int ret = 0;
        ^

 drivers/fpga/dfl-pci.c:230:10: warning: Variable 'ret' is assigned a value that is never used. [unreadVariable]
    int ret = 0;
            ^

Fixes: 3c2760b7 ("fpga: dfl: pci: fix return value of cci_pci_sriov_configure")
Reported-by: default avatarkbuild test robot <lkp@intel.com>
Signed-off-by: default avatarXu Yilun <yilun.xu@intel.com>
Acked-by: default avatarWu Hao <hao.wu@intel.com>
Reviewed-by: default avatarTom Rix <trix@redhat.com>
Signed-off-by: default avatarMoritz Fischer <mdf@kernel.org>
parent 11ba4688
...@@ -227,7 +227,6 @@ static int cci_pci_sriov_configure(struct pci_dev *pcidev, int num_vfs) ...@@ -227,7 +227,6 @@ static int cci_pci_sriov_configure(struct pci_dev *pcidev, int num_vfs)
{ {
struct cci_drvdata *drvdata = pci_get_drvdata(pcidev); struct cci_drvdata *drvdata = pci_get_drvdata(pcidev);
struct dfl_fpga_cdev *cdev = drvdata->cdev; struct dfl_fpga_cdev *cdev = drvdata->cdev;
int ret = 0;
if (!num_vfs) { if (!num_vfs) {
/* /*
...@@ -239,6 +238,8 @@ static int cci_pci_sriov_configure(struct pci_dev *pcidev, int num_vfs) ...@@ -239,6 +238,8 @@ static int cci_pci_sriov_configure(struct pci_dev *pcidev, int num_vfs)
dfl_fpga_cdev_config_ports_pf(cdev); dfl_fpga_cdev_config_ports_pf(cdev);
} else { } else {
int ret;
/* /*
* before enable SRIOV, put released ports into VF access mode * before enable SRIOV, put released ports into VF access mode
* first of all. * first of all.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment