diff options
| author |   Michal Kalderon <michal.kalderon@marvell.com> | 2019-05-30 15:20:40 +0300 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2019-05-30 15:01:07 -0700 |
| commit | 8e2ea3ea962535c64800e82e03e3e1b6cfd9c64f (patch) | |
| tree | 72e31d70dbb7f4499967dd81513444fd2dd2e6b5 /drivers | |
| parent | net: dsa: Add error path handling in dsa_tree_setup() (diff) | |
| download | linux-dev-8e2ea3ea962535c64800e82e03e3e1b6cfd9c64f.tar.xz linux-dev-8e2ea3ea962535c64800e82e03e3e1b6cfd9c64f.zip | |
qed: Fix static checker warning
In some cases abs_ppfid could be printed without being initialized.
Fixes: 79284adeb99e ("qed: Add llh ppfid interface and 100g support for offload protocols")
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Michal Kalderon <michal.kalderon@marvell.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/ethernet/qlogic/qed/qed_dev.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/drivers/net/ethernet/qlogic/qed/qed_dev.c b/drivers/net/ethernet/qlogic/qed/qed_dev.c index a971418755e9..eec7cb65c7e6 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_dev.c +++ b/drivers/net/ethernet/qlogic/qed/qed_dev.c @@ -1138,12 +1138,12 @@ qed_llh_add_protocol_filter(struct qed_dev *cdev, if (rc) goto err; + rc = qed_llh_abs_ppfid(cdev, ppfid, &abs_ppfid); + if (rc) + goto err; + /* Configure the LLH only in case of a new the filter */ if (ref_cnt == 1) { - rc = qed_llh_abs_ppfid(cdev, ppfid, &abs_ppfid); - if (rc) - goto err; - rc = qed_llh_protocol_filter_to_hilo(cdev, type, source_port_or_eth_type, dest_port, &high, &low); @@ -1195,12 +1195,12 @@ void qed_llh_remove_mac_filter(struct qed_dev *cdev, if (rc) goto err; + rc = qed_llh_abs_ppfid(cdev, ppfid, &abs_ppfid); + if (rc) + goto err; + /* Remove from the LLH in case the filter is not in use */ if (!ref_cnt) { - rc = qed_llh_abs_ppfid(cdev, ppfid, &abs_ppfid); - if (rc) - goto err; - rc = qed_llh_remove_filter(p_hwfn, p_ptt, abs_ppfid, filter_idx); if (rc) @@ -1253,12 +1253,12 @@ void qed_llh_remove_protocol_filter(struct qed_dev *cdev, if (rc) goto err; + rc = qed_llh_abs_ppfid(cdev, ppfid, &abs_ppfid); + if (rc) + goto err; + /* Remove from the LLH in case the filter is not in use */ if (!ref_cnt) { - rc = qed_llh_abs_ppfid(cdev, ppfid, &abs_ppfid); - if (rc) - goto err; - rc = qed_llh_remove_filter(p_hwfn, p_ptt, abs_ppfid, filter_idx); if (rc) |