wifi: ath12k: do not process consecutive RDDM event

Currently we do reset for each RDDM event from MHI, however there are
cases, see below log, that we get two or more consecutive events, and
it is pointless to do reset for the subsequent ones. What's more, it
makes reset process more likely to fail.

[  103.289864] mhi mhi0: System error detected
[  103.289871] ath12k_pci 0000:03:00.0: mhi notify status reason MHI_CB_EE_RDDM
[  103.293144] mhi mhi0: System error detected
[  103.293150] ath12k_pci 0000:03:00.0: mhi notify status reason MHI_CB_EE_RDDM

Add a check to avoid reset again and again. This is done by tracking previous
MHI status: if we receive a new RDDM event while the previous event is
also the same, we treat it as duplicate and ignore it, because normally
we should at least receive a MHI_CB_EE_MISSION_MODE event between them.

Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0.c5-00481-QCAHMTSWPL_V1.0_V2.0_SILICONZ-3

Signed-off-by: Baochen Qiang <quic_bqiang@quicinc.com>
Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://msgid.link/20240529015357.9846-1-quic_bqiang@quicinc.com

diff --git a/drivers/net/wireless/ath/ath12k/mhi.c b/drivers/net/wireless/ath/ath12k/mhi.c
index fef2f76220336d8817f657cbdefff55f4956192d..df96b0f91f54c1f05126663e1c23f8d31f1de769 100644 (file)
--- a/drivers/net/wireless/ath/ath12k/mhi.c
+++ b/drivers/net/wireless/ath/ath12k/mhi.c
@@ -16,6 +16,7 @@
 #define MHI_TIMEOUT_DEFAULT_MS 90000
 #define OTP_INVALID_BOARD_ID   0xFFFF
 #define OTP_VALID_DUALMAC_BOARD_ID_MASK                0x1000
+#define MHI_CB_INVALID 0xff
 
 static const struct mhi_channel_config ath12k_mhi_channels_qcn9274[] = {
        {
@@ -268,6 +269,7 @@ static void ath12k_mhi_op_status_cb(struct mhi_controller *mhi_cntrl,
                                    enum mhi_callback cb)
 {
        struct ath12k_base *ab = dev_get_drvdata(mhi_cntrl->cntrl_dev);
+       struct ath12k_pci *ab_pci = ath12k_pci_priv(ab);
 
        ath12k_dbg(ab, ATH12K_DBG_BOOT, "mhi notify status reason %s\n",
                   ath12k_mhi_op_callback_to_str(cb));
@@ -277,12 +279,20 @@ static void ath12k_mhi_op_status_cb(struct mhi_controller *mhi_cntrl,
                ath12k_warn(ab, "firmware crashed: MHI_CB_SYS_ERROR\n");
                break;
        case MHI_CB_EE_RDDM:
+               if (ab_pci->mhi_pre_cb == MHI_CB_EE_RDDM) {
+                       ath12k_dbg(ab, ATH12K_DBG_BOOT,
+                                  "do not queue again for consecutive RDDM event\n");
+                       break;
+               }
+
                if (!(test_bit(ATH12K_FLAG_UNREGISTERING, &ab->dev_flags)))
                        queue_work(ab->workqueue_aux, &ab->reset_work);
                break;
        default:
                break;
        }
+
+       ab_pci->mhi_pre_cb = cb;
 }
 
 static int ath12k_mhi_op_read_reg(struct mhi_controller *mhi_cntrl,
@@ -313,6 +323,7 @@ int ath12k_mhi_register(struct ath12k_pci *ab_pci)
        if (!mhi_ctrl)
                return -ENOMEM;
 
+       ab_pci->mhi_pre_cb = MHI_CB_INVALID;
        ab_pci->mhi_ctrl = mhi_ctrl;
        mhi_ctrl->cntrl_dev = ab->dev;
        mhi_ctrl->regs = ab->mem;

diff --git a/drivers/net/wireless/ath/ath12k/pci.h b/drivers/net/wireless/ath/ath12k/pci.h
index 6186a78038cf6e48be0d7d7ebafeee7187d075e8..31584a7ad80eb9f55423905039050eac247aa1d8 100644 (file)
--- a/drivers/net/wireless/ath/ath12k/pci.h
+++ b/drivers/net/wireless/ath/ath12k/pci.h
@@ -104,6 +104,7 @@ struct ath12k_pci {
        struct mhi_controller *mhi_ctrl;
        const struct ath12k_msi_config *msi_config;
        unsigned long mhi_state;
+       enum mhi_callback mhi_pre_cb;
        u32 register_window;
 
        /* protects register_window above */