On 4/11/2024 11:06 PM, Baochen Qiang wrote: > Both the firmware reset feature and the power management > suspend/resume feature share common power-down and power-up > functionality. One aspect of the power-up functionality is > the handling of the ATH12K_QMI_EVENT_FW_INIT_DONE event. > When this event is received, a call is made to > ath12k_hal_dump_srng_stats(), with the purpose to collect > information that may be useful in debugging the cause of a > firmware reset. > > Unfortunately, since this functionality is shared between > both the firmware reset path and the power management > resume path, the kernel log is flooded with messages during > resume. Since these messages are not useful during resume, > and in fact can be confusing and can increase the time it > takes to resume, update the logic to only call > ath12k_hal_dump_srng_stats() during firmware reset. > > Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4 > Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Baochen Qiang <quic_bqiang@xxxxxxxxxxx> Acked-by: Jeff Johnson <quic_jjohnson@xxxxxxxxxxx>
