diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2020-09-18 16:33:11 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-09-21 22:55:13 +0200 |
| commit | 58ed68b592377f7483bcbff937567d8cbaab38ed (patch) | |
| tree | 9f6c7db2bc45a543deed0dcff2aab54194644a79 /drivers | |
| parent | Merge branch 'bnxt_en-Bug-fixes' (diff) | |
| download | linux-58ed68b592377f7483bcbff937567d8cbaab38ed.tar.xz linux-58ed68b592377f7483bcbff937567d8cbaab38ed.zip | |
sfc: Fix error code in probe
This failure path should return a negative error code but it currently
returns success.
Fixes: 51b35a454efd ("sfc: skeleton EF100 PF driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Edward Cree <ecree@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/sfc/ef100.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/sfc/ef100.c b/drivers/net/ethernet/sfc/ef100.c index c54b7f8243f3..ffdb36715a49 100644 --- a/drivers/net/ethernet/sfc/ef100.c +++ b/drivers/net/ethernet/sfc/ef100.c @@ -490,6 +490,7 @@ static int ef100_pci_probe(struct pci_dev *pci_dev, if (fcw.offset > pci_resource_len(efx->pci_dev, fcw.bar) - ESE_GZ_FCW_LEN) { netif_err(efx, probe, efx->net_dev, "Func control window overruns BAR\n"); + rc = -EIO; goto fail; } |