DEV-701 sd logging data not seen with usbx msc log and stream

[s-varna]

No description provided.

[Copilot]

Pull request overview

This PR addresses an issue where SD logging data is not observed when using USBX MSC in a “log and stream” workflow, by adjusting when dock/USB setup logic runs and by refining SD-card handoff behavior when USB is plugged in.

Changes:

• Removed LogAndStream_setupDockUndock() from the TASK_USB_SETUP execution path.
• Added an SD “ready/transfer state” wait in LogAndStream_setupDock() before changing SD access routing.
• When USB is plugged in (but not docked), explicitly routes SD access back to the MCU and invalidates DCache.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File	Description
TaskList/shimmer_taskList.c	Stops invoking dock/undock setup from the USB setup task.
log_and_stream_common.c	Adds SD readiness wait + USB-plug SD routing behavior; introduces DCache invalidation dependency.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 15 out of 15 changed files in this pull request and generated 12 comments.

Comments suppressed due to low confidence (1)

EEPROM/shimmer_eeprom.c:154

• The docstring for ShimEeprom_checkBtErrorCounts() says it resets invalid counters and has a side effect calling ShimEeprom_resetBtErrorCounts(), but the function currently only returns 1/0 and does not reset anything. Either update the documentation or add the described reset behavior to match the comment.

/**
 * Checks if any Bluetooth error count fields are set to the invalid value
 * (0xFFFF). If so, resets all error counts to zero.
 *
 * @return 1 if any error count was invalid and reset, 0 otherwise.
 * @sideeffect Calls ShimEeprom_resetBtErrorCounts() if any count is invalid.
 */
uint8_t ShimEeprom_checkBtErrorCounts(void)
{
  if (eepromSensorSettingsPage.btCntDisconnectWhileStreaming == 0xFFFF
      || eepromSensorSettingsPage.btCntUnsolicitedReboot == 0xFFFF
      || eepromSensorSettingsPage.btCntRtsLockup == 0xFFFF
      || eepromSensorSettingsPage.btCntDataRateTestBlockage == 0xFFFF)
  {
    return 1;
  }
  return 0;

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.