[PATCH v2 1/3] wifi: ath12k: configure RDDM size to MHI for device recovery

Wen Gong quic_wgong at quicinc.com
Thu Aug 3 02:40:39 PDT 2023


On 8/3/2023 5:24 PM, Kalle Valo wrote:
> Wen Gong <quic_wgong at quicinc.com> wrote:
>
>> RDDM is RAM DUMP DEBUG module, it is used to debug issues when firmware
>> encounters an error.
>>
>> The rddm_size is needed by firmware while MHI enter RDDM state. Add it
>> to support device recovery when ath12k receive MHI_CB_EE_RDDM message.
>>
>> Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4
>>
>> Signed-off-by: Wen Gong <quic_wgong at quicinc.com>
>> Signed-off-by: Kalle Valo <quic_kvalo at quicinc.com>
> I'm not sure what "support device recovery" means exactly. How does this patch
> change functionality from user's point of view?
>
> No need to resend because of this, I can add that to the commit log.
Device recovery means SSR(subsystem restart), when firmware happen 
crash, ath12k
will receive the RDDM event, and then ath12k/mac80211 begin to re-start 
wifi/firmware,
after that, the wifi become normal again.

This patch is to let firmware report RDDM event correctly to ath12k. 
Without this patch,
firmware will not report RDDM event to ath12k correctly, then ath12k 
will not begin SSR
process.

I think it should be changed like this:
The rddm_size is needed by firmware while MHI enter RDDM state. Add it
and then firmware will report MHI_CB_EE_RDDM correctly while firmware
encounters an error, then ath12k could start the device recovery process.



More information about the ath12k mailing list