[ath9k-devel] [PATCH 02/10] ath10k: decouple core init/deinit logic

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

On 10/06/13 18:53, Kalle Valo wrote:
> Michal Kazior <michal.kazior@tieto.com> writes:
>
>> Enables code reuse for proper hw reconfiguration
>> that is in turn required for proper
>> suspend/hibernation/wowlan support.
>>
>> Signed-off-by: Michal Kazior <michal.kazior@tieto.com>
>
> Let's have a terminology check. I have understood we use terms with
> meaning like this (maybe you have been thinking something else):
>
> init(): initilises the module in question from host side, allocates
>          memory etc. Called only once when driver is loaded.
>
> deinit(): reverse action for init(), frees allocated memory etc. Called
>            only once.
>
> start(): Initialises hw/fw. Called every time we start the firmware.
>
> stop(): Kills the running fw. Shuts down all power the device (when
>          possible)
>
> But here ath10k_core_init() seems to be initialising hw/fw. So should it
> be renamed to ath10k_core_start()?

You're right. I'll fix this and resend.


-- Pozdrawiam / Best regards, Michal Kazior.