Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SD Init Fail message only when we can detect the SD Card Insertion #19236

Conversation

rhapsodyv
Copy link
Member

Description

Only show sd init fail on card.mount if we can detect the card.

If we can't detect the sd card, the actual code shows "SD Init Fail" forever, because the card.mount is called in the marlin main loop always....

Benefits

Related Issues

#19064
#19226

…ther way will be showing SD Init Fail forever
@mikemerryguy
Copy link
Contributor

still sd init fail

@thisiskeithb
Copy link
Member

thisiskeithb commented Sep 2, 2020

I tested these changes on the following configs and no longer get "SD Init Fail" on boot:

  • BTT002/REPRAP_DISCOUNT_SMART_CONTROLLER (no motherboard-based SD card reader)
  • SKR Mini E3 V2/CR10_STOCKDISPLAY (no LCD-based SD card reader)
  • SKR Mini E3 V1.2/CR10_STOCKDISPLAY (no LCD-based SD card reader)
  • SKR 1.4/REPRAP_DISCOUNT_FULL_GRAPHIC_SMART_CONTROLLER/SDCARD_CONNECTION set to ONBOARD
  • SKR 1.4/REPRAP_DISCOUNT_FULL_GRAPHIC_SMART_CONTROLLER/SDCARD_CONNECTION set to LCD

Thank you!

@thisiskeithb
Copy link
Member

still sd init fail

I just tested the fix on an SKR Mini E3 1.2 with CR10_STOCKDISPLAY and it fixed the issue.

@mikemerryguy
Copy link
Contributor

@thisiskeithb @rhapsodyv
yup
I messed up a line, It is working for me as well!

Sorry!!!
( i didn't remove a removed line oops!!!!)

Glad we could all work through this

Thank you!!!
You guys Rock!!!!

@thinkyhead thinkyhead merged commit 1f25312 into MarlinFirmware:bugfix-2.0.x Sep 3, 2020
@rhapsodyv rhapsodyv deleted the sd-init-fail-only-when-sd-detect branch October 25, 2020 00:17
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
kageurufu pushed a commit to CR30-Users/Marlin-CR30 that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants