[posix] fix rcp reset flow in ResetRcp (openthread#9314)

Remove call to `WaitResponse` after calling `HardwareReset` which didn't trigger any spinel transaction. `kOpenFileDelay` change form 500 to 50.
ferre111 · Aug 27, 2023 · 4078917 · 4078917
1 parent 4d2d950
commit 4078917
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/lib/spinel/radio_spinel_impl.hpp b/src/lib/spinel/radio_spinel_impl.hpp
@@ -259,7 +259,11 @@ template <typename InterfaceType> void RadioSpinel<InterfaceType>::ResetRcp(bool
     }
 
     hardwareReset = (mSpinelInterface.HardwareReset() == OT_ERROR_NONE);
-    SuccessOrExit(WaitResponse(false));
+
+    if (hardwareReset)
+    {
+        SuccessOrExit(WaitResponse(false));
+    }
 
     resetDone = true;
 

diff --git a/src/posix/platform/hdlc_interface.hpp b/src/posix/platform/hdlc_interface.hpp
@@ -235,7 +235,7 @@ class HdlcInterface : public ot::Spinel::SpinelInterface
     {
         kMaxWaitTime   = 2000, ///< Maximum wait time in Milliseconds for socket to become writable (see `SendFrame`).
         kResetTimeout  = 5000, ///< Maximum wait time in Milliseconds for file to become ready (see `ResetConnection`).
-        kOpenFileDelay = 500,  ///< Delay between open file calls, in Milliseconds (see `ResetConnection`).
+        kOpenFileDelay = 50,   ///< Delay between open file calls, in Milliseconds (see `ResetConnection`).
         kRemoveRcpDelay =
             2000, ///< Delay for removing RCP device from host OS after hard reset (see `ResetConnection`).
     };