Initializing SD card...initialization failed! with GIT but ok with 2.5

[luc-github]

SD support looks broken in GIT version when it was Ok in 2.5

Basic Infos

• [ X] This issue complies with the issue POLICY doc.
• [X ] I have read the documentation at readthedocs and the issue is not addressed there.
• [ X] I have tested that the issue is present in current master branch (aka latest git).
• [ X] I have searched the issue tracker for a similar issue.
• [ X] If there is a stack dump, I have decoded it.
• [ X] I have filled out all fields below.

Platform

• Hardware: ESP-12
• Core Version: latest git 04/27/2019
• Development Env: Arduino IDE
• Operating System: Windows 10

Settings in IDE

• Module: Generic ESP8266 Module
• Flash Mode: DOUT
• Flash Size: 4MB
• lwip Variant: v2 Lower Memory
• Reset Method: nodemcu
• Flash Frequency: 40Mhz
• CPU Frequency: 160MHz
• Upload Using: SERIAL
• Upload Speed: 115200

Problem Description

SD card init failed with git version
using sample : https://github.com/esp8266/Arduino/blob/master/libraries/SD/examples/listfiles/listfiles.ino

Initializing SD card...initialization failed!

on same hardware with core 2.5 same code works

Also tried the new library :
https://github.com/earlephilhower/ESP8266SdFat/blob/92ab4b59eb920bf3134347dbe07e7c059eb64d46/examples/ReadWrite/ReadWrite.ino modifing
#define SD_CS_PIN 4 to not change the wiring
same error:
Initializing SD card...initialization failed!

[earlephilhower]

I've just run the ReadFiles.ino example on 2 cards, changing the one line to D0 from 4 as that's the way I'm wired up:

    if (!SD.begin(D0)) {

On the 128MB Canon SDC-128MH card I get the expected output (Debug enabled):

SDK:2.2.1(cfd48f3)/Core:2.5.0-100-gcdb54957=20500100/lwIP:STABLE-2_1_2_RELEASE/glue:1.1-7-g82abda3/BearSSL:a143020
Initializing SD card...initialization done.
Writing to test.txt...done.
test.txt:
testing 1, 2, 3.

And the new Samsung EVO Select U1 32GB I get the right output, too:

SDK:2.2.1(cfd48f3)/Core:2.5.0-100-gcdb54957=20500100/lwIP:STABLE-2_1_2_RELEASE/glue:1.1-7-g82abda3/BearSSL:a143020
Initializing SD card...initialization done.
Writing to test.txt...done.
test.txt:
testing 1, 2, 3.

Assuming it is a card related issue, what can you tell us about the card? Is it a very old MMC one? What size, brand, type, any other info?

[earlephilhower]

Also, calling @TD-er. I think I remember you going to the GIT head for your ESPEasy. Does it support SD cards, and if so have you or any of your users tried it or had issues like this? Thx!

[luc-github]

My card is Micro SD Kingston 4Gb microSDHC – Classe 4 -
I did not expect it was a SD card issue as it work on 3D printers and on version 2.5
It works also on ESP32 with SD library

I have also tried an ADATA 16GBMicro SD class 10 with same result

[earlephilhower]

..edit..since I can't spell PIN...and can't spell "since" either, it seems...

Also tried the new library ReadWrite.ino modifing
#define SD_CS_PIN 4 to not change the wiring
same error:
Initializing SD card...initialization failed!

The SD_CS_PIN define isn't used anywhere in the SD library:

earle@server:~/Arduino/hardware/esp8266com/esp8266/libraries/SD$ grep -r SD_CS_PIN
earle@server:~/Arduino/hardware/esp8266com/esp8266/libraries/SD$ 

SD_CS_PIN is used in some SdFat examples, but there is no actual use in the low-level libs, so defining it is a no-op, I think.

Can you edit the readfiles.ino example and change the same line I showed SD.begin(4) -> SD.begin(D0) to your pin (making sure to use appropriate D4 or whatever CS pin macro, or the raw integer if you're very sure of the mapping between 8266 internal GPI numbering and your board)?

[luc-github]

I am not sure to understand what you mean SD_CS_PIN is used in file

#include <SPI.h>
//#include <SD.h>
#include "SdFat.h"

using namespace sdfat;

SdFat SD;

#define SD_CS_PIN 4
File myFile;

void setup() {
  // Open serial communications and wait for port to open:
  Serial.begin(9600);
  while (!Serial) {
    ; // wait for serial port to connect. Needed for native USB port only
  }


  Serial.print("Initializing SD card...");

  if (!SD.begin(SD_CS_PIN)) {
    Serial.println("initialization failed!");
    return;
  }
  Serial.println("initialization done.");

[earlephilhower]

Ahha, I think I see the difference:

I am using the SD library example ReadFiles.ino, which is the same as was always included in the ESP8266 ARduino tree.

You are using the ESP8266SdFat example ReadFiles.ino which is different and much updated by the SdFat maintainer. That library was just added by me since we threw out the unmaintained, cut-n-pasted version from Arduino.cc

So, there is something in the SdFat examples which is incorrect, but the SD ones are fine. LEt me see. It's probably a simple define/namespace/config issue.

[luc-github]

and about SD library listfiles.ino :

Serial.print("Initializing SD card...");

  if (!SD.begin(4)) {
    Serial.println("initialization failed!");
    return;
  }
  Serial.println("initialization done.");

sample is not good anymore ?

[earlephilhower]

Think we cross-posted. You're not using the SD examples, but the SdFat ones...

[luc-github]

read my first post I tried both

[earlephilhower]

So, what's going on is the ESP8266SdFat examples have an updated, incompatible with the older SD SD.h where they make an ESP8266-incompatible File (defined in a private header) and do other badness. It's fine for other systems where they're the only filesystem, just not good for us. And not needed since our SD.h is compatible with all our other Filesystems (SPIFFS, LittleFS, etc.).

Basically, I think the only answer is to ignore the SdFat examples and use the 8266-compatible SD ones. I can blow the examples away from the repo to avoid this issue.

[earlephilhower]

read my first post I tried both

For the SD one, did you modify the SD.begin() line or set the #define? The #define would be unused. I'm unable to repro any failure, so we'll need some others to chime in to get any other idea.

[luc-github]

the sample code from SD :
https://github.com/esp8266/Arduino/blob/master/libraries/SD/examples/listfiles/listfiles.ino

I did not change anything - same code in 2.5 works and same code failed in GIT

both are identical sample and identical hardware and identical SD card

[luc-github]

I changed code like you:

 if (!SD.begin(D0)) {
...

and I get

C:\Users\user\Desktop\listfiles\listfiles.ino: In function 'void setup()':

listfiles:38:17: error: 'D0' was not declared in this scope

   if (!SD.begin(D0)) {

                 ^

Multiple libraries were found for "SD.h"
 Used: C:\Users\user\Documents\Arduino\hardware\esp8266com\esp8266\libraries\SD
 Not used: C:\Program Files\WindowsApps\ArduinoLLC.ArduinoIDE_1.8.21.0_x86__mdqgnx93n4wtt\libraries\SD
Using library SPI at version 1.0 in folder: C:\Users\user\Documents\Arduino\hardware\esp8266com\esp8266\libraries\SPI 
Using library SD at version 2.0.0 in folder: C:\Users\user\Documents\Arduino\hardware\esp8266com\esp8266\libraries\SD 
Using library SDFS at version 0.1.0 in folder: C:\Users\user\Documents\Arduino\hardware\esp8266com\esp8266\libraries\SDFS 
Using library ESP8266SdFat at version 1.0.16 in folder: C:\Users\user\Documents\Arduino\hardware\esp8266com\esp8266\libraries\ESP8266SdFat 
exit status 1
'D0' was not declared in this scope

is that normal ?

[earlephilhower]

That's definitely not normal.

Do you maybe have the GIT and the IDE version installed at the same time? Need to uninstall one to use the other successfully. The lack of a D0 define means even basic Arduino.h isn't working for you. Assuming that there is a "D0" on your board, of course. :)