🩹 Adjust manage_media for slow/late media init (USB FD) (#24015)

This commit is contained in:
Robby Candra 2022-05-23 09:13:17 +07:00 committed by Scott Lahteine
parent 28f8646aa6
commit 74596ad4d7

View file

@ -472,29 +472,32 @@ void CardReader::mount() {
#endif #endif
void CardReader::manage_media() { void CardReader::manage_media() {
static uint8_t prev_stat = 2; // First call, no prior state static uint8_t prev_stat = 2; // At boot we don't know if media is present or not
uint8_t stat = uint8_t(IS_SD_INSERTED()); uint8_t stat = uint8_t(IS_SD_INSERTED());
if (stat == prev_stat) return; if (stat == prev_stat) return; // Already checked and still no change?
DEBUG_SECTION(mm, "CardReader::manage_media", true); DEBUG_SECTION(cmm, "CardReader::manage_media()", true);
DEBUG_ECHOLNPGM("SD Status ", prev_stat, " -> ", stat); DEBUG_ECHOLNPGM("Media present: ", prev_stat, " -> ", stat);
flag.workDirIsRoot = true; // Return to root on mount/release
if (!ui.detected()) { if (!ui.detected()) {
DEBUG_ECHOLNPGM("SD: No UI Detected."); DEBUG_ECHOLNPGM("SD: No UI Detected.");
return; return;
} }
uint8_t old_stat = prev_stat; flag.workDirIsRoot = true; // Return to root on mount/release/init
prev_stat = stat; // Change now to prevent re-entry
const uint8_t old_stat = prev_stat;
prev_stat = stat; // Change now to prevent re-entry in safe_delay
if (stat) { // Media Inserted if (stat) { // Media Inserted
safe_delay(500); // Some boards need a delay to get settled safe_delay(500); // Some boards need a delay to get settled
// Try to mount the media (only later with SD_IGNORE_AT_STARTUP) // Try to mount the media (only later with SD_IGNORE_AT_STARTUP)
if (TERN1(SD_IGNORE_AT_STARTUP, old_stat != 2)) mount(); if (TERN1(SD_IGNORE_AT_STARTUP, old_stat != 2)) mount();
if (!isMounted()) stat = 0; // Not mounted? if (!isMounted()) { // Not mounted?
stat = 0;
IF_DISABLED(SD_IGNORE_AT_STARTUP, prev_stat = 0);
}
TERN_(RESET_STEPPERS_ON_MEDIA_INSERT, reset_stepper_drivers()); // Workaround for Cheetah bug TERN_(RESET_STEPPERS_ON_MEDIA_INSERT, reset_stepper_drivers()); // Workaround for Cheetah bug
} }
@ -504,16 +507,25 @@ void CardReader::manage_media() {
#endif #endif
} }
ui.media_changed(old_stat, stat); // Update the UI ui.media_changed(old_stat, stat); // Update the UI or flag an error
if (!stat) return; // Exit if no media is present if (!stat) return; // Exit if no media is present
TERN_(SDCARD_EEPROM_EMULATION, settings.first_load()); static bool did_first_insert = false;
if (did_first_insert) return; // Did a media insert already happen?
if (old_stat != 2) return; // First mount? did_first_insert = true; // Definitely handling this media insert...
DEBUG_ECHOLNPGM("First mount."); DEBUG_ECHOLNPGM("First mount.");
// Load settings the first time media is inserted (not just during init)
TERN_(SDCARD_EEPROM_EMULATION, settings.first_load());
#if HAS_USB_FLASH_DRIVE
const millis_t ms = millis();
DEBUG_ECHOLNPGM("USB mount waiting time = ", ms);
if (ms > 5000) return; // Too late to be considered "already inserted"?
#endif
bool do_auto = true; UNUSED(do_auto); bool do_auto = true; UNUSED(do_auto);
// Check for PLR file. // Check for PLR file.