diff options
| author | Yuval Mintz <Yuval.Mintz@qlogic.com> | 2016-07-27 14:45:21 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-07-31 06:34:03 +0300 |
| commit | b21290b77f78f62e3f41ef876bfadacd65c83980 (patch) | |
| tree | 2d27aec9580578241c5b83b3e92317625300cd2e /drivers/net/ethernet/qlogic/qed/qed_vf.c | |
| parent | cb1fa088772e9e331841b10bea88d833ec8c01fa (diff) | |
| download | linux-b21290b77f78f62e3f41ef876bfadacd65c83980.tar.xz | |
qed: Don't over-do producer cleanup for Rx
Before requesting the firmware to start Rx queues,
driver goes and sets the queue producer in the device to 0.
But while the producer is 32-bit, the driver currently clears 64 bits,
effectively zeroing an additional CID's producer as well.
Signed-off-by: Yuval Mintz <Yuval.Mintz@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/qlogic/qed/qed_vf.c')
| -rw-r--r-- | drivers/net/ethernet/qlogic/qed/qed_vf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/qlogic/qed/qed_vf.c b/drivers/net/ethernet/qlogic/qed/qed_vf.c index 9819230947bf..9b780b31b15c 100644 --- a/drivers/net/ethernet/qlogic/qed/qed_vf.c +++ b/drivers/net/ethernet/qlogic/qed/qed_vf.c @@ -388,7 +388,7 @@ int qed_vf_pf_rxq_start(struct qed_hwfn *p_hwfn, /* Learn the address of the producer from the response */ if (pp_prod) { - u64 init_prod_val = 0; + u32 init_prod_val = 0; *pp_prod = (u8 __iomem *)p_hwfn->regview + resp->offset; DP_VERBOSE(p_hwfn, QED_MSG_IOV, @@ -396,7 +396,7 @@ int qed_vf_pf_rxq_start(struct qed_hwfn *p_hwfn, rx_qid, *pp_prod, resp->offset); /* Init the rcq, rx bd and rx sge (if valid) producers to 0 */ - __internal_ram_wr(p_hwfn, *pp_prod, sizeof(u64), + __internal_ram_wr(p_hwfn, *pp_prod, sizeof(u32), (u32 *)&init_prod_val); } |