ESP32 Korvo 1.1 SD card doesn't work

[FlorianB99]

I am experiencing a new problem: I am always getting the error [E] AudioSourceSD.h : 60 - SD.begin cs=23 failed.
The pins I am using are tested with my SD card test program, they are 100% correct.

This is my code:

/**
 * @file player-sd-i2s.ino
 * @brief example using the SD library
 * 
 * @author Phil Schatzmann
 * @copyright GPLv3
 */

#include "AudioTools.h"
#include "AudioTools/AudioLibs/AudioSourceSD.h"
#include "AudioTools/AudioCodecs/CodecMP3Helix.h"
#include "AudioTools/AudioLibs/I2SCodecStream.h"
#include "SPI.h"

int sck = 5;
int miso = 21;
int mosi = 18;
int cs = 23;
const char* startFilePath = "/";
const char* ext = "mp3";
SPIClass mySPI;
AudioSourceSD source(startFilePath, ext, cs, mySPI, true);
I2SCodecStream i2s(KorvoV1);
MP3DecoderHelix decoder;
AudioPlayer player(source, i2s, decoder);


void printMetaData(MetaDataType type, const char* str, int len) {
  Serial.print("==> ");
  Serial.print(toStr(type));
  Serial.print(": ");
  Serial.println(str);
}

void setup() {
  Serial.begin(115200);
  mySPI.begin(sck, miso, mosi, cs);
  //AudioToolsLogger.begin(Serial, AudioToolsLogLevel::Info);
AudioDriverLogger.begin(Serial,AudioDriverLogLevel::Info);
  // setup output
  auto cfg = i2s.defaultConfig(TX_MODE);
  i2s.begin(cfg);
  bool volSet = i2s.setVolume(0.6);
  // setup player
  //source.setFileFilter("*Bob Dylan*");
  player.setMetadataCallback(printMetaData);
  player.begin();
}

In Driverpins.h I added:

class PinsKorvoV1Class : public DriverPins {
 public:
  PinsKorvoV1Class() {
	  
    // sd pins
    addSPI(ESP32PinsSD);
    // add i2c codec pins: scl, sda, port, frequency
    addI2C(PinFunction::CODEC, 32, 19);
    // add i2s pins: mclk, bck, ws,data_out, data_in ,(port)
    addI2S(PinFunction::CODEC,     0, 25, 22, 13, -1);
    addI2S(PinFunction::CODEC_ADC, 0, 27, 26, -1, 36);
    // add other pins
    addPin(PinFunction::KEY, 39, PinLogic::Input, 0);
    addPin(PinFunction::PA, 12, PinLogic::Output);
    addPin(PinFunction::LED, 33, PinLogic::Output);
  }
};

I am getting this in log, confirming the pins I set:

Info:    AudioDriver::begin
Info:    setupI2CAddress: 0x18
Info:    ==> Updating address: 0x18
Info:    setting up SPI miso:21,mosi:18, clk:5, cs:23
Info:    setting up I2C scl: 32, sda: 19
Info:    Setting i2c clock: 100000
Info:    AudioDriver::setConfig
Info:    MCLK_SOURCE: 0
Info:    ES8311 in Slave mode
Info:    ES8311 clock source: SCLK
Info:    setPAPower pin 12 -> 1
[E] AudioSourceSD.h : 60 - SD.begin cs=23 failed

Where else could I look? Thank you in advance.

[pschatzmann]

I suggest you remove all your drive related custom logic from your sketch and let the driver do the job. Use e.g. this example.

With this sketch you can also check if your buttons are working...

[FlorianB99]

I did and SD card worked after I modified the default pins. I need to implement that in the Driver section of course.

// Added to be compatible with the AudioKitStream.h
#ifndef PIN_AUDIO_KIT_SD_CARD_CS
#define PIN_AUDIO_KIT_SD_CARD_CS 23
#define PIN_AUDIO_KIT_SD_CARD_MISO 21
#define PIN_AUDIO_KIT_SD_CARD_MOSI 18
#define PIN_AUDIO_KIT_SD_CARD_CLK 5

I am having trouble with the keys though. I am getting messages about pin ignored because it is already defined. I assume that the logic of this code does not work:

  kit.addDefaultActions();
  kit.addAction(kit.getKey(1), startStop);
  kit.addAction(kit.getKey(4), next);
  kit.addAction(kit.getKey(3), previous);

Because the GPIO for getKey() is always the same. Therefore, all my keys will just mute/unmute. Any suggestions? I don't see how this would work with the Lyrat Mini.

[pschatzmann]

Check the isKeyPressed() metho in PinsLyratMiniClass. It uses analogRead() to determine the voltage and assigns the keys to voltage ranges

[FlorianB99]

Yes, I have that and modified to the voltages that I measured. I just don't see how it is compatible to addAction() as this seems to be based on actual GPIOs

[pschatzmann]

The AudioBoardAction which is defined in AudioBoardStream is calling p_board->isKeyPressed(key);

[FlorianB99]

Yes I get that, but all I was doing, was copying the LyratMini Code and modified the values. I am able to use the isKeyPressed() function, it just doesn't work with the player_sd_audiokit example. Anyway, since I don't need the programmed actions, it is not a showstopper. I am moving on to using the microphone.

I will be happy to send you the code and a board, so you can try and possibly include the board into your examples.

[pschatzmann]

Mybe you can double check with the LyraT-Mini buttons example

[FlorianB99]

Done. It works but it's missing many key presses. I think I would have to adjust the range limits.

[FlorianB99]

I will not continue the key issue. As far as I am concerned, this is solved and working.