[PATCH 2/2] ath11k: mhi: use mhi_sync_power_up()

[Kalle Valo]

Jeff Johnson <quic_jjohnson at quicinc.com> writes:

> On 1/27/2022 1:01 AM, Kalle Valo wrote:
>> From: Kalle Valo <quic_kvalo at quicinc.com>
>>
>> If amss.bin was missing ath11k would crash during 'rmmod ath11k_pci'. The
>> reason for that was that we were using mhi_async_power_up() which does not
>> check any errors. But mhi_sync_power_up() on the other hand does check for
>> errors so let's use that to fix the crash.
>>
>> I was not able to find a reason why an async version was used. This call ends
>> is called from ath11k_hif_power_up() which can sleep, so sync version should be
>> safe to use.
>
> I could not parse the 2nd sentence: This call ends is called

Heh, that is a pretty horrible sentence. How about this:

"ath11k_mhi_start() (which enables state ATH11K_MHI_POWER_ON) is called
from ath11k_hif_power_up(), which can sleep. So sync version should be
safe to use here."

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches