On Wed, Dec 26, 2018 at 12:56:22PM -0600, Aditya Pakki wrote: > In bnxt_qplib_map_tc2cos(), bnxt_qplib_rcfw_send_message() can return > an error value. The fix returns the error from the latter function > upstream. > > Signed-off-by: Aditya Pakki <pakki001@xxxxxxx> > --- > drivers/infiniband/hw/bnxt_re/qplib_sp.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/infiniband/hw/bnxt_re/qplib_sp.c b/drivers/infiniband/hw/bnxt_re/qplib_sp.c > index 5216b5f844cc..adb8eae5193c 100644 > --- a/drivers/infiniband/hw/bnxt_re/qplib_sp.c > +++ b/drivers/infiniband/hw/bnxt_re/qplib_sp.c > @@ -778,9 +778,8 @@ int bnxt_qplib_map_tc2cos(struct bnxt_qplib_res *res, u16 *cids) > req.cos0 = cpu_to_le16(cids[0]); > req.cos1 = cpu_to_le16(cids[1]); > > - bnxt_qplib_rcfw_send_message(rcfw, (void *)&req, (void *)&resp, NULL, > - 0); > - return 0; > + return bnxt_qplib_rcfw_send_message(rcfw, (void *)&req, (void *)&resp, > + NULL, 0); > } AFAIK, there is no need to do casting to/from void*. Thanks > > int bnxt_qplib_get_roce_stats(struct bnxt_qplib_rcfw *rcfw, > -- > 2.17.1 >
Attachment:
signature.asc
Description: PGP signature
