Merge pull request #4624 from paroga/usb_dfu_reset

embassy-usb-dfu: fix: do net reset in GetStatus request

2 files changed, 8 insertions, 1 deletions

diff --git a/embassy-usb-dfu/CHANGELOG.md b/embassy-usb-dfu/CHANGELOG.md
index ef842945d..0088e66fe 100644
--- a/embassy-usb-dfu/CHANGELOG.md
+++ b/embassy-usb-dfu/CHANGELOG.md
@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 <!-- next-header -->
 ## Unreleased - ReleaseDate
 
+- changed: Do not reset in the GetStatus request
 - Allow enabling the `application` and `dfu` feature at the same time
 
 ## 0.2.0 - 2025-08-27
diff --git a/embassy-usb-dfu/src/dfu.rs b/embassy-usb-dfu/src/dfu.rs
index be28890bb..7c28d04cf 100644
--- a/embassy-usb-dfu/src/dfu.rs
+++ b/embassy-usb-dfu/src/dfu.rs
@@ -183,7 +183,7 @@ impl<'d, DFU: NorFlash, STATE: NorFlash, RST: Reset, const BLOCK_SIZE: usize> Ha
             Ok(Request::GetStatus) => {
                 match self.state {
                     State::DlSync => self.state = State::Download,
-                    State::ManifestSync => self.reset.sys_reset(),
+                    State::ManifestSync => self.state = State::ManifestWaitReset,
                     _ => {}
                 }
 
@@ -202,6 +202,12 @@ impl<'d, DFU: NorFlash, STATE: NorFlash, RST: Reset, const BLOCK_SIZE: usize> Ha
             _ => None,
         }
     }
+
+    fn reset(&mut self) {
+        if matches!(self.state, State::ManifestSync | State::ManifestWaitReset) {
+            self.reset.sys_reset()
+        }
+    }
 }
 
 /// An implementation of the USB DFU 1.1 protocol