[PATCH] ath10k: Fix un-initialized debug objects.
Ben Greear
greearb at candelatech.com
Fri Oct 25 10:59:46 EDT 2013
On 10/25/2013 12:49 AM, Kalle Valo wrote:
> greearb at candelatech.com writes:
>> INIT_WORK(&ar->restart_work, ath10k_core_restart);
>>
>> + ath10k_debug_init(ar);
>
> For symmetry would it make more sense to move ath10k_debug_destroy() to
> ath10k_core_unregister()? That way we could avoid adding a new function.
>
In my opinion, you should initialize such things as early as possible
so you don't have to worry so much about the various error cases leaving
things un-initialized. I believe my patch accomplished that.
What new function are you planning to avoid?
Thanks,
Ben
--
Ben Greear <greearb at candelatech.com>
Candela Technologies Inc http://www.candelatech.com
More information about the ath10k
mailing list