espressif/openthread
1
0
Fork 0
mirror of https://github.com/espressif/openthread.git synced 2026-08-14 14:47:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

[spinel] remove mIsCoprocessorReady flag check (#10144)

Browse Source 
- If we check in between SendReset() and WaitResponse(),
  mIsCoprocessorReady may become true before waitResponse() and if
  loop becomes false even successfully reset done.
- mIsCoprocessorReady is already checked before sendReset() to avoid
  resetting in case of Mulipan architecture. There is no need to check
  in between sendReset() and waitResponse().
- Removed mIsCoprocessorReady from if loop check between sendReset()
  and waitResponse().

Signed-off-by: Ashishkumar Vara <[email protected]>
This commit is contained in:
Ashish
2024-05-02 10:14:49 -07:00
committed by GitHub
parent 3517e6ce37
commit f404a228aa
1 changed files with 1 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/lib/spinel/spinel_driver.cpp
+1 -2
View File
@@ -129,8 +129,7 @@ void SpinelDriver::ResetCoprocessor(bool aSoftwareReset)
mWaitingKey = SPINEL_PROP_LAST_STATUS;
if (aSoftwareReset && (SendReset(SPINEL_RESET_STACK) == OT_ERROR_NONE) && (!mIsCoprocessorReady) &&
(WaitResponse() == OT_ERROR_NONE))
if (aSoftwareReset && (SendReset(SPINEL_RESET_STACK) == OT_ERROR_NONE) && (WaitResponse() == OT_ERROR_NONE))
{
VerifyOrExit(mIsCoprocessorReady, resetDone = false);
LogCrit("Software reset co-processor successfully");