Messages in this thread | | | From | Devesh Sharma <> | Date | Mon, 29 Jan 2018 17:38:48 +0530 | Subject | Re: [0/2] RDMA/bnxt_re: Adjustments for bnxt_qplib_alloc_dpi_tbl() |
| |
On Mon, Jan 29, 2018 at 3:01 PM, SF Markus Elfring <elfring@users.sourceforge.net> wrote: >> You don't need 2 patches when changing same lines of code. > > Are these really the same? > > >> Could you squash both and send your changes in a single patch. > > I prefer to keep the deletion of questionable error messages separate > from the refactoring for a bit of exception handling. >
Okay fair enough Series Acked-By: Devesh Sharma <devesh.sharma@broadcom.com>
> >> The patches look good to me otherwise. > > Thanks for your constructive feedback. > > Regards, > Markus > -- > 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
| |