Fix broken AudioDriverLyratMiniClass

Author: pschatzmann

src/Driver.h

@@ -1414,26 +1414,28 @@ class AudioDriverLyratMiniClass : public AudioDriver {
     // setup SPI for SD
     // pins.setSPIActiveForSD(codecCfg.sd_active);
 
-    bool ok = true;
-
     // Start ES8311
     AD_LOGI("starting ES8311");
+    pins.begin();
     dac.setPins(this->pins());
-    ok = dac.setConfig(codecCfg);
+    if (!dac.setConfig(codecCfg)) {
+      AD_LOGE("setConfig failed");
+      return false;
+    }
     setPAPower(true);
     setVolume(DRIVER_DEFAULT_VOLUME);
 
     // Start ES7243
     if (codecCfg.input_device != ADC_INPUT_NONE) {
       AD_LOGI("starting ES7243");
       adc.setPins(this->pins());
-      ok = ok && adc.setConfig(codecCfg);
+      if (!adc.setConfig(codecCfg)) {
+        AD_LOGE("adc.begin failed");
+        return false;
+      }
     }
 
-    if (!ok) {
-      AD_LOGI("AudioDriverLyratMiniClass::begin failed");
-    }
-    return ok;
+    return true;
   }
   bool end(void) {
     int rc = 0;

src/DriverPins.h

@@ -192,7 +192,7 @@ struct PinsI2C : public I2CConfig {
   bool pinsAvailable() { return scl != -1 && sda != -1 && frequency != 0; }
   operator bool() { return pinsAvailable(); }
 
-  bool begin() {
+  bool begin() {    
     if (set_active) {
       AD_LOGD("PinsI2C::begin for function %d on port %d", (int)function, port);
       return i2c_bus_create(this) == RESULT_OK;

src/Platforms/API_I2C_Arduino.cpp

@@ -8,6 +8,7 @@
 #include "Platforms/API_I2C.h"
 
 error_t i2c_bus_create(struct I2CConfig *config) {
+  AD_LOGI("i2c_bus_create");
   assert(config != nullptr);
   I2CConfig &pins = *config;
   TwoWire *p_wire = (TwoWire *)pins.p_wire;
@@ -21,7 +22,7 @@ error_t i2c_bus_create(struct I2CConfig *config) {
     // begin with defined pins, if supported
 #if defined(ESP32)
     AD_LOGI("setting up I2C scl: %d, sda: %d", pins.scl, pins.sda);
-    p_wire->begin(pins.sda, pins.scl);
+    p_wire->begin(pins.sda,pins.scl);
 #elif defined(ARDUINO_ARCH_RP2040) || defined(ARDUINO_ARCH_STM32)
     AD_LOGI("setting up I2C scl: %d, sda: %d", pins.scl, pins.sda);
     p_wire->setSCL(pins.scl);