Re: [PATCH][next] wcn36xx: remove redundant assignment to msg_body.min_ch_time

2017-12-29 Thread Colin Ian King
On 29/12/17 07:44, Loic Poulain wrote: > Hi Colin, Bjorn, > > On 26 December 2017 at 21:13, Bjorn Andersson > wrote: >> On Tue 19 Dec 09:04 PST 2017, Colin King wrote: >> >>> From: Colin Ian King >>> >>> msg_body.min_ch_time is being assigned twice; remove the redundant >>> first assignment. >>>

Re: [PATCH][next] wcn36xx: remove redundant assignment to msg_body.min_ch_time

2017-12-28 Thread Loic Poulain
Hi Colin, Bjorn, On 26 December 2017 at 21:13, Bjorn Andersson wrote: > On Tue 19 Dec 09:04 PST 2017, Colin King wrote: > >> From: Colin Ian King >> >> msg_body.min_ch_time is being assigned twice; remove the redundant >> first assignment. >> >> Detected by CoverityScan, CID#1463042 ("Unused Val

Re: [PATCH][next] wcn36xx: remove redundant assignment to msg_body.min_ch_time

2017-12-26 Thread Bjorn Andersson
On Tue 19 Dec 09:04 PST 2017, Colin King wrote: > From: Colin Ian King > > msg_body.min_ch_time is being assigned twice; remove the redundant > first assignment. > > Detected by CoverityScan, CID#1463042 ("Unused Value") > Happy to see Coverity working for us :) This should have had a: Fix