Messages in this thread | | | Subject | Re: [0/2] RDMA/bnxt_re: Adjustments for bnxt_qplib_alloc_dpi_tbl() | From | SF Markus Elfring <> | Date | Mon, 29 Jan 2018 10:31:04 +0100 |
| |
> 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.
> The patches look good to me otherwise.
Thanks for your constructive feedback.
Regards, Markus
| |