[ath9k-devel] [PATCH 1/5] ath10k: replace debug_mtx with conf_mutex

[Michal Kazior <michal.kazior@tieto.com>]

On 18/04/13 11:44, Kalle Valo wrote:
> Michal Kazior <michal.kazior@tieto.com> writes:
>
>> It's not a good to have too many mutexes. The
>> patch simplifies locking of debufs to use
>> conf_mutex.
>>
>> Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
>
> [...]
>
>> @@ -328,7 +328,7 @@ int ath10k_debug_create(struct ath10k *ar)
>>   		return -ENOMEM;
>>
>>   	init_completion(&ar->debug.event_stats_compl);
>> -	mutex_init(&ar->debug.debug_mtx);
>> +	mutex_init(&ar->conf_mutex);
>
> Can this be right? We already initialise conf_mutex in core.c.

Good catch, thanks! The mutex_init() is already initialized elsewhere. 
No need to re-initialize it here.


-- Pozdrawiam / Best regards, Michal Kazior.