From 46e25cbc5ff62e24f86574d7ae5d872aa0c2595d Mon Sep 17 00:00:00 2001
From: Gerhard de Clercq <11624490+Gerharddc@users.noreply.github.com>
Date: Tue, 13 May 2025 15:09:53 +0200
Subject: [embassy-usb-dfu] correct comment about composite devices

---
 examples/boot/application/stm32wb-dfu/src/main.rs | 10 ++++------
 examples/boot/bootloader/stm32wb-dfu/src/main.rs  | 10 ++++------
 2 files changed, 8 insertions(+), 12 deletions(-)

(limited to 'examples/boot')

diff --git a/examples/boot/application/stm32wb-dfu/src/main.rs b/examples/boot/application/stm32wb-dfu/src/main.rs
index 68e9bc3f6..6236dfe52 100644
--- a/examples/boot/application/stm32wb-dfu/src/main.rs
+++ b/examples/boot/application/stm32wb-dfu/src/main.rs
@@ -59,13 +59,11 @@ async fn main(_spawner: Spawner) {
 
     // We add MSOS headers so that the device automatically gets assigned the WinUSB driver on Windows.
     // Otherwise users need to do this manually using a tool like Zadig.
-    builder.msos_descriptor(msos::windows_version::WIN8_1, 2);
-
-    // In the case of non-composite devices, it seems that feature headers need to be on the device level.
-    // (As is implemented here)
     //
-    // For composite devices however, they should be on the function level instead.
-    // (This is achieved by passing a GUID to the "usb_dfu" function)
+    // It seems these always need to be at added at the device level for this to work and for
+    // composite devices they also need to be added on the function level (as shown later).
+    //
+    builder.msos_descriptor(msos::windows_version::WIN8_1, 2);
     builder.msos_feature(msos::CompatibleIdFeatureDescriptor::new("WINUSB", ""));
     builder.msos_feature(msos::RegistryPropertyFeatureDescriptor::new(
         "DeviceInterfaceGUIDs",
diff --git a/examples/boot/bootloader/stm32wb-dfu/src/main.rs b/examples/boot/bootloader/stm32wb-dfu/src/main.rs
index 2cd7f859d..8cfd4daa7 100644
--- a/examples/boot/bootloader/stm32wb-dfu/src/main.rs
+++ b/examples/boot/bootloader/stm32wb-dfu/src/main.rs
@@ -67,13 +67,11 @@ fn main() -> ! {
 
         // We add MSOS headers so that the device automatically gets assigned the WinUSB driver on Windows.
         // Otherwise users need to do this manually using a tool like Zadig.
-        builder.msos_descriptor(msos::windows_version::WIN8_1, 2);
-
-        // In the case of non-composite devices, it seems that feature headers need to be on the device level.
-        // (As is implemented here)
         //
-        // For composite devices however, they should be on the function level instead.
-        // (This is achieved by passing a GUID to the "usb_dfu" function)
+        // It seems these always need to be at added at the device level for this to work and for
+        // composite devices they also need to be added on the function level (as shown later).
+        //
+        builder.msos_descriptor(msos::windows_version::WIN8_1, 2);
         builder.msos_feature(msos::CompatibleIdFeatureDescriptor::new("WINUSB", ""));
         builder.msos_feature(msos::RegistryPropertyFeatureDescriptor::new(
             "DeviceInterfaceGUIDs",
-- 
cgit