On Mon, Sep 16, 2019 at 4:27 AM Gavin Hu <[email protected]> wrote: > As there is an inclusive rte_io_wmb within the following rte_write32() > API who rings the doorbell, this makes the above rte_wmb unnecessary and > remove it. > > Fixes: 1cd45aeb3270 ("net/bnxt: support Stratus VF device") > Cc: [email protected] > > Signed-off-by: Gavin Hu <[email protected]> > Reviewed-by: Steve Capper <[email protected]> > Reviewed-by: Ruifeng Wang <[email protected]> > Reviewed-by: Phil Yang <[email protected]> >
Acked-by: Ajit Khaparde <[email protected]> > --- > drivers/net/bnxt/bnxt_hwrm.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/net/bnxt/bnxt_hwrm.c b/drivers/net/bnxt/bnxt_hwrm.c > index 9883fb5..1d9fb17 100644 > --- a/drivers/net/bnxt/bnxt_hwrm.c > +++ b/drivers/net/bnxt/bnxt_hwrm.c > @@ -115,9 +115,6 @@ static int bnxt_hwrm_send_message(struct bnxt *bp, > void *msg, > data = (uint32_t *)&short_input; > msg_len = sizeof(short_input); > > - /* Sync memory write before updating doorbell */ > - rte_wmb(); > - > max_req_len = BNXT_HWRM_SHORT_REQ_LEN; > } > > -- > 2.7.4 > >
