[PATCH 1/2] net: davinci_cpdma: reduce time holding ctlr->lock in cpdma_control_set

2016-07-26 Thread Uwe Kleine-König
The only user of cpdma_control_set (i.e. cpsw_ndo_open) doesn't check the return code, so it doesn't matter, which error triggers. So the checks that are independant of the fields protected by ctlr->lock can be moved out of the critical section. Signed-off-by: Uwe Kleine-König --- drivers/net/et

Re: [PATCH 1/2] net: davinci_cpdma: reduce time holding ctlr->lock in cpdma_control_set

2016-08-04 Thread Grygorii Strashko
On 07/26/2016 03:02 PM, Uwe Kleine-König wrote: The only user of cpdma_control_set (i.e. cpsw_ndo_open) doesn't check the return code, so it doesn't matter, which error triggers. So the checks that are independant of the fields protected by ctlr->lock can be moved out of the critical section. Si

Re: [PATCH 1/2] net: davinci_cpdma: reduce time holding ctlr->lock in cpdma_control_set

2016-08-09 Thread Mugunthan V N
On Tuesday 26 July 2016 05:32 PM, Uwe Kleine-König wrote: > The only user of cpdma_control_set (i.e. cpsw_ndo_open) doesn't check > the return code, so it doesn't matter, which error triggers. So the > checks that are independant of the fields protected by ctlr->lock can be > moved out of the criti