[spinel] add coprocessor reset failure callback

Author: gytxxsy

src/lib/spinel/openthread-spinel-config.h

@@ -163,4 +163,13 @@
 #define OPENTHREAD_SPINEL_CONFIG_RCP_TX_WAIT_TIME_SECS 5
 #endif
 
+/**
+ * @def OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE
+ *
+ * Enables Co-processor reset failure callback in Spinel driver
+ */
+#ifndef OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE
+#define OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE 0
+#endif
+
 #endif // OPENTHREAD_SPINEL_CONFIG_H_

src/lib/spinel/spinel_driver.cpp

@@ -52,6 +52,10 @@ SpinelDriver::SpinelDriver(void)
     , mSpinelVersionMajor(-1)
     , mSpinelVersionMinor(-1)
     , mIsCoprocessorReady(false)
+#if OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE
+    , mCoprocessorResetFailureCallback(nullptr)
+    , mCoprocessorResetFailureContext(nullptr)
+#endif
 {
     memset(mVersion, 0, sizeof(mVersion));
 
@@ -119,6 +123,15 @@ otError SpinelDriver::SendReset(uint8_t aResetType)
     return error;
 }
 
+#if OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE
+void SpinelDriver::SetCoprocessorResetFailureCallback(otSpinelDriverCoprocessorResetFailureCallback aCallback,
+                                                      void                                         *aContext)
+{
+    mCoprocessorResetFailureCallback = aCallback;
+    mCoprocessorResetFailureContext  = aContext;
+}
+#endif
+
 void SpinelDriver::ResetCoprocessor(bool aSoftwareReset)
 {
     bool hardwareReset;
@@ -129,7 +142,13 @@ void SpinelDriver::ResetCoprocessor(bool aSoftwareReset)
     VerifyOrExit(!mIsCoprocessorReady, resetDone = true);
 #endif
 
-    mWaitingKey = SPINEL_PROP_LAST_STATUS;
+    // Before resetting the co-processor, mIsCoprocessorReady must be set to false.
+    // Otherwise, in some cases, the host may misinterpret the reset result.
+    // For example, if mIsCoprocessorReady is true when calling ResetCoprocessor,
+    // and the reset fails but the co-processor sends some invalid data,`WaitResponse`
+    // may mistakenly return OT_ERROR_NONE, causing the host to believe the reset succeeded.
+    mIsCoprocessorReady = false;
+    mWaitingKey         = SPINEL_PROP_LAST_STATUS;
 
     if (aSoftwareReset && (SendReset(SPINEL_RESET_STACK) == OT_ERROR_NONE) && (WaitResponse() == OT_ERROR_NONE))
     {
@@ -160,6 +179,12 @@ void SpinelDriver::ResetCoprocessor(bool aSoftwareReset)
     if (!resetDone)
     {
         LogCrit("Failed to reset co-processor!");
+#if OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE
+        if (mCoprocessorResetFailureCallback)
+        {
+            mCoprocessorResetFailureCallback(mCoprocessorResetFailureContext);
+        }
+#endif
         DieNow(OT_EXIT_FAILURE);
     }
 }

src/lib/spinel/spinel_driver.hpp

@@ -215,6 +215,27 @@ class SpinelDriver : public otSpinelDriver, public Logger
      */
     spinel_iid_t GetIid(void) { return mIid; }
 
+#if OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE
+    /**
+     * A callback type for handling Co-processor reset failure of spinel driver.
+     *
+     * @param[in] aContext  A pointer to the user context.
+     */
+    typedef void (*otSpinelDriverCoprocessorResetFailureCallback)(void *aContext);
+
+    /**
+     * Registers a callback to handle Co-processor reset failure of Spinel driver.
+     *
+     * This function is used to register a callback to handle Co-processor reset failure.
+     * When the Spinel driver fails to reset the Co-processor through both software and
+     * hardware resets, the user can handle the error through the callback(such as OTA).
+     *
+     * @param[in] aCallback The callback.
+     * @param[in] aContext  A pointer to the user context.
+     */
+    void SetCoprocessorResetFailureCallback(otSpinelDriverCoprocessorResetFailureCallback aCallback, void *aContext);
+#endif
+
 private:
     static constexpr uint16_t kMaxSpinelFrame    = SPINEL_FRAME_MAX_SIZE;
     static constexpr uint16_t kVersionStringSize = 128;
@@ -314,6 +335,11 @@ class SpinelDriver : public otSpinelDriver, public Logger
     char mVersion[kVersionStringSize];
 
     Array<unsigned int, kCapsBufferSize> mCoprocessorCaps;
+
+#if OPENTHREAD_SPINEL_CONFIG_COPROCESSOR_RESET_FAILURE_CALLBACK_ENABLE
+    otSpinelDriverCoprocessorResetFailureCallback mCoprocessorResetFailureCallback;
+    void                                         *mCoprocessorResetFailureContext;
+#endif
 };
 
 } // namespace Spinel