Re: [PATCH 23/26] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-03-09 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:28:10PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 10:58:34 +0100 > > 1. Return zero in one case directly. > > 2. Return the value from a call of the function "_ocrdma_dealloc_pd" >without using an

Re: [PATCH 23/26] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-03-09 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:28:10PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 10:58:34 +0100 > > 1. Return zero in one case directly. > > 2. Return the value from a call of the function "_ocrdma_dealloc_pd" >without using an extra assignment for the local

Re: [PATCH 23/26] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-03-08 Thread Devesh Sharma
Acked-By: Devesh Sharma On Wed, Mar 8, 2017 at 6:58 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 10:58:34 +0100 > > 1. Return zero in one case directly. > > 2. Return the

Re: [PATCH 23/26] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-03-08 Thread Devesh Sharma
Acked-By: Devesh Sharma On Wed, Mar 8, 2017 at 6:58 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 10:58:34 +0100 > > 1. Return zero in one case directly. > > 2. Return the value from a call of the function "_ocrdma_dealloc_pd" >without using an extra

[PATCH 23/26] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 10:58:34 +0100 1. Return zero in one case directly. 2. Return the value from a call of the function "_ocrdma_dealloc_pd" without using an extra assignment for the local variable. 3. Remove the variable "status" in

[PATCH 23/26] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 10:58:34 +0100 1. Return zero in one case directly. 2. Return the value from a call of the function "_ocrdma_dealloc_pd" without using an extra assignment for the local variable. 3. Remove the variable "status" in this function then.