feat: retry failed usb operations

Author: FabulousCodingFox

src/ledmatrix/ledmatrix.cpp

@@ -47,14 +47,11 @@ namespace fw16led::ledmatrix
     // Send the data via bulk OUT transfer
     int actual_length = 0;
     int ret = libusb_bulk_transfer(device.get(), ENDPOINT_OUT, outData.data(), static_cast<int>(outData.size()), &actual_length, TRANSFER_TIMEOUT_MS);
-    if (ret != LIBUSB_SUCCESS)
-    {
-      LOG_WARN("Bulk OUT transfer failed: {}", libusb_strerror(static_cast<libusb_error>(ret)));
-    }
-
-    if (actual_length != static_cast<int>(outData.size()))
+    if (ret != LIBUSB_SUCCESS || actual_length != static_cast<int>(outData.size()))
     {
-      LOG_WARN("Bulk OUT transfer size does not match. Expected {}; Got {}", outData.size(), actual_length);
+      LOG_WARN("Bulk OUT transfer failed or size mismatch: {}", libusb_strerror(static_cast<libusb_error>(ret)));
+      std::this_thread::sleep_for(std::chrono::milliseconds(100));
+      return send_command(command, parameters);
     }
   }
 
@@ -72,15 +69,11 @@ namespace fw16led::ledmatrix
     // Send the data via bulk OUT transfer
     int actual_length = 0;
     int ret = libusb_bulk_transfer(device.get(), ENDPOINT_OUT, outData.data(), static_cast<int>(outData.size()), &actual_length, TRANSFER_TIMEOUT_MS);
-    if (ret != LIBUSB_SUCCESS)
-    {
-      LOG_WARN("Bulk OUT transfer failed: {}", libusb_strerror(static_cast<libusb_error>(ret)));
-      return {};
-    }
-
-    if (actual_length != static_cast<int>(outData.size()))
+    if (ret != LIBUSB_SUCCESS || actual_length != static_cast<int>(outData.size()))
     {
-      LOG_WARN("Bulk OUT transfer size does not match. Expected {}; Got {}", outData.size(), actual_length);
+      LOG_WARN("Bulk OUT transfer failed or size mismatch: {}", libusb_strerror(static_cast<libusb_error>(ret)));
+      std::this_thread::sleep_for(std::chrono::milliseconds(100));
+      return send_command_with_response(command, parameters);
     }
 
     std::vector<uint8_t> inData(RESPONSE_SIZE, 0);
@@ -90,7 +83,8 @@ namespace fw16led::ledmatrix
     if (ret != LIBUSB_SUCCESS)
     {
       LOG_WARN("Bulk IN transfer failed: {}", libusb_strerror(static_cast<libusb_error>(ret)));
-      return {};
+      std::this_thread::sleep_for(std::chrono::milliseconds(100));
+      return send_command_with_response(command, parameters);
     }
 
     // Shrink the buffer to the actual number of bytes read

src/managers/usb.cpp

@@ -81,26 +81,6 @@ namespace fw16led::managers
 
             LOG_DEBUG("-> Successfully claimed interface 1");
 
-            // Wait for the device to be ready
-            bool ready = false;
-            std::vector<uint8_t> dummy = {0x32, 0xAC, 0x00, 0x00};
-            while (!ready)
-            {
-              int actual_length = 0;
-              int ret = libusb_bulk_transfer(handle, 0x01, dummy.data(), 4, &actual_length, 100);
-              if (ret == LIBUSB_SUCCESS && actual_length == 4)
-              {
-                ready = true;
-              }
-              else
-              {
-                LOG_WARN("-> Waiting for device to be ready...");
-                std::this_thread::sleep_for(std::chrono::milliseconds(100));
-              }
-            }
-
-            LOG_DEBUG("-> Device is ready");
-
             // Store the handle
             ledpanels.push_back(std::make_shared<LedPanel>(std::make_shared<ledmatrix::LedMatrix>(handle)));
           }