On 26 August 2014 11:05, Kalle Valo <[email protected]> wrote:
> + [email protected]
>
> Julia Lawall <[email protected]> writes:
>
>> On Tue, 26 Aug 2014, kbuild test robot wrote:
>>
>>> tree:   git://github.com/kvalo/ath ath-next-test
>>> head:   077256daf111b4db074ac602bed1e96e95e3a4a4
>>> commit: 6b7b27b12f4af628305940c1c083b8558d7618ac [47/51] ath10k: improve 
>>> logging to include dev id
>>> :::::: branch date: 16 hours ago
>>> :::::: commit date: 16 hours ago
>>>
>>> >> drivers/net/wireless/ath/ath10k/debug.c:169:5-8: Unneeded variable: 
>>> >> "ret". Return "0" on line 178
>>>
>>> Please consider folding the attached diff :-)
>>
>> The variable is indeed not needed.  I guess the function returns a value
>> for parallelism with ath10k_err.  But the function dev_err used in that
>> function returns a value, while the macro dev_warn_ratelimited used in
>> this function does not.
>
> Ok. If it's ok for Michal I'll fold the patch before I commit it.

I'm okay with that. Thanks.


MichaƂ

_______________________________________________
ath10k mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/ath10k

Reply via email to