Messages in this thread |  | | From | Devesh Sharma <> | Date | Mon, 29 Jan 2018 09:14:13 +0530 | Subject | Re: [PATCH 0/2] RDMA/bnxt_re: Adjustments for bnxt_qplib_alloc_dpi_tbl() |
| |
On Sun, Jan 28, 2018 at 1:45 AM, SF Markus Elfring <elfring@users.sourceforge.net> wrote: > From: Markus Elfring <elfring@users.sourceforge.net> > Date: Sat, 27 Jan 2018 21:10:12 +0100 > > Two update suggestions were taken into account > from static source code analysis.
You don't need 2 patches when changing same lines of code. Could you squash both and send your changes in a single patch. The patches look good to me otherwise. -Thanks
Reviewed-By: Devesh Sharma <devesh.sharma@broadcom.com>
> > Markus Elfring (2): > Delete two error messages for a failed memory allocation > Use common error handling code > > drivers/infiniband/hw/bnxt_re/qplib_res.c | 18 +++++++----------- > 1 file changed, 7 insertions(+), 11 deletions(-) > > -- > 2.16.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
|  |