The a2dp example for vs1053 does not work

Author: pschatzmann

examples/examples-vs1053/basic-a2dp-vs1053/basic-a2dp-vs1053.ino

@@ -15,21 +15,23 @@
 
 BluetoothA2DPSink a2dp_sink;
 VS1053Stream out; // final output
-
+bool active = false;
 // Write data to SPDIF in callback
 void read_data_stream(const uint8_t *data, uint32_t length) {
-  out.write(data, length);
+  if (active) out.write(data, length);
 }
 
-// Start and Stop VS1053Stream to make sure that WAV header is generated
+// for esp_a2d_audio_state_t see https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/bluetooth/esp_a2dp.html#_CPPv421esp_a2d_audio_state_t
 void audio_state_changed(esp_a2d_audio_state_t state, void *ptr){
   Serial.println(a2dp_sink.to_str(state));
     switch(state){
       case ESP_A2D_AUDIO_STATE_STARTED:
         out.begin();
+        active = true;
         break;
       case ESP_A2D_AUDIO_STATE_STOPPED:
       case ESP_A2D_AUDIO_STATE_REMOTE_SUSPEND:
+        active = false;
         out.end();
         break;
     }
@@ -38,7 +40,7 @@ void audio_state_changed(esp_a2d_audio_state_t state, void *ptr){
 
 void setup() {
   Serial.begin(115200);
-  AudioLogger::instance().begin(Serial, AudioLogger::Warning);
+  AudioLogger::instance().begin(Serial, AudioLogger::Info);
 
   // register callbacks
   a2dp_sink.set_stream_reader(read_data_stream, false);
@@ -48,7 +50,7 @@ void setup() {
   a2dp_sink.set_auto_reconnect(false);
   a2dp_sink.start("a2dp-vs1053");
 
-  // setup output
+  // setup VS1053
   auto cfg = out.defaultConfig();
   cfg.sample_rate = a2dp_sink.sample_rate();
   cfg.channels = 2;
@@ -60,6 +62,7 @@ void setup() {
   //cfg.reset_pin = VS1053_RESET;
 
   out.begin(cfg);
+  out.end();
 }
 
-void loop() { delay(1000); }
+void loop() { delay(1000); }

examples/sandbox/basic-a2dp-vs1053/basic-a2dp-vs1053.ino

@@ -24,7 +24,7 @@ class VS1053Config : public AudioBaseInfo {
         channels = 2;
         bits_per_sample = 16;
     }
-    RxTxMode mode;
+    RxTxMode mode = TX_MODE;
     /// set to false if it is a pcm stream
     uint8_t cs_pin = VS1053_CS; 
     uint8_t dcs_pin = VS1053_DCS;
@@ -83,10 +83,14 @@ class VS1053Stream : public AudioStreamX {
     }
 
     /// defines the default configuration that is used with the next begin()
-    void setConfig(VS1053Config c){
+    void setAudioInfo(VS1053Config c){
         cfg = c;
     }
 
+    void setAudioInfo(AudioBaseInfo c){
+        cfg.copyFrom(c);
+    }
+
     /// Starts with the default config or restarts
     bool begin() {
         return begin(cfg);
@@ -102,6 +106,7 @@ class VS1053Stream : public AudioStreamX {
         }
         this->cfg = cfg;
         setAudioInfo(cfg);
+        cfg.logInfo();
         LOGI("is_encoded_data: %s", cfg.is_encoded_data?"true":"false");
         LOGI("is_midi: %s", cfg.is_midi?"true":"false");
         LOGI("cs_pin: %d", cfg.cs_pin);