Re: [PATCH] net: wireless: ath: ath10k: remove unnecessary code

2017-05-09 Thread Gustavo A. R. Silva
Hi Arend, Quoting Arend Van Spriel : On 9-5-2017 7:33, Kalle Valo wrote: "Gustavo A. R. Silva" writes: The name of an array used by itself will always return the array's address. So these tests will always evaluate as false and therefore the _return_ will never be executed. Signed-off-by:

Re: [PATCH] net: wireless: ath: ath10k: remove unnecessary code

2017-05-09 Thread Gustavo A. R. Silva
Hi Kalle, Quoting Kalle Valo : "Gustavo A. R. Silva" writes: The name of an array used by itself will always return the array's address. So these tests will always evaluate as false and therefore the _return_ will never be executed. Signed-off-by: Gustavo A. R. Silva I don't understand t

Re: [PATCH] net: wireless: ath: ath10k: remove unnecessary code

2017-05-09 Thread Kalle Valo
Arend Van Spriel writes: > On 9-5-2017 7:33, Kalle Valo wrote: >> "Gustavo A. R. Silva" writes: >> >>> The name of an array used by itself will always return the array's address. >>> So these tests will always evaluate as false and therefore the _return_ >>> will never be executed. >>> >>> Sign

Re: [PATCH] net: wireless: ath: ath10k: remove unnecessary code

2017-05-09 Thread Arend Van Spriel
On 9-5-2017 7:33, Kalle Valo wrote: > "Gustavo A. R. Silva" writes: > >> The name of an array used by itself will always return the array's address. >> So these tests will always evaluate as false and therefore the _return_ >> will never be executed. >> >> Signed-off-by: Gustavo A. R. Silva > >

Re: [PATCH] net: wireless: ath: ath10k: remove unnecessary code

2017-05-08 Thread Kalle Valo
"Gustavo A. R. Silva" writes: > The name of an array used by itself will always return the array's address. > So these tests will always evaluate as false and therefore the _return_ > will never be executed. > > Signed-off-by: Gustavo A. R. Silva I don't understand the commit log, especially wh