Feature/unified eth with player state

README.md

@@ -108,23 +108,11 @@ Update third party code (opus, flac, esp-dsp, improv_wifi):
 ```
 git submodule update --init
 ```
-Copy one of the template sdkconfig files and rename it to sdkconfig...
 
-...on Linux:
-```
-cp sdkconfig_lyrat_v4.3 sdkconfig
-```
-
-...on Windows:
-```
-copy sdkconfig_lyrat_v4.3 sdkconfig
-```
-
-### ESP-IDF environment setup (required for configuration, compiling and flashing)
+### ESP-IDF environnement configuration
 - <b>If you're on Windows :</b> Install [ESP-IDF v5.5.1](https://github.com/espressif/esp-idf/releases/tag/v5.5.1) locally ([More info](https://docs.espressif.com/projects/esp-idf/en/latest/esp32/get-started/windows-setup-update.html)).
-- <b>If you're on Linux (docker) :</b> Use the image for ESP-IDF by following [docker build](doc/docker_build.md) doc (you won't need any of the remaining commands/steps below up until the <b>Test</b> section then) 
+- <b>If you're on Linux (docker) :</b> Use the image for ESP-IDF by following [docker build](doc/docker_build.md) doc
 - <b>If you're on Linux :</b> follow [official Espressif](https://docs.espressif.com/projects/esp-idf/en/stable/esp32/get-started/linux-macos-setup.html) instructions
-
   For debian based systems you'll need to do the following:
   ```
   sudo apt-get install git wget flex bison gperf python3 python3-pip python3-venv cmake ninja-build ccache libffi-dev libssl-dev dfu-util libusb-1.0-0
@@ -136,16 +124,24 @@ copy sdkconfig_lyrat_v4.3 sdkconfig
   . ./export.sh
   ```
 
-### Snapcast ESP Configuration (Non-Docker-Linux and Windows)
+<a name="config"></a>
+### Snapcast ESP Configuration
+Start with the default config (remove any existing sdkconfig file) or copy one of the template sdkconfig files and rename it to sdkconfig
 
-Configure your platform:
+```
+rm sdkconfig
+```
+or
+```
+cp sdkconfig_lyrat_v4.3 sdkconfig
+```
+
+then configure your platform:
 
 ```
 idf.py menuconfig
 ```
-
-<a name="config"></a>
-Choose configuration options to match your setup
+Configure to match your setup
   - <b>Audio HAL :</b> Choose your audio board
     - Lyrat (4.3, 4.2)
     - Lyrat TD (2.2, 2.1)
@@ -235,7 +231,7 @@ Replace `snapclient.local` with your clients IP address. If you have multiple cl
 
 You are very welcome to help and provide [Pull
 Requests](https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-requests)
-to the project. Use [develop](https://github.com/CarlosDerSeher/snapclient/tree/develop) branch for your PRs as this is the place where new features will go.
+to the project. Use [develop](https://github.com/jorgenkraghjakobsen/snapclient/tree/develop) branch for your PRs as this is the place where new features will go.
 
 We strongly suggest you activate [pre-commit](https://pre-commit.com) hooks in
 this git repository before starting to hack and make commits.

components/lightsnapcast/player.c

@@ -499,12 +499,17 @@ int init_player(i2s_std_gpio_config_t pin_config0_, i2s_port_t i2sNum_, void (*s
 
   // create message queue to inform task of changed settings
   snapcastSettingQueueHandle = xQueueCreate(1, sizeof(playerSetting_t));
+  if (snapcastSettingQueueHandle == NULL) {
+    ESP_LOGE(TAG, "Failed to create snapcast settings queue");
+    return -1;
+  }
 
   if (playerStateMux == NULL) {
     playerStateMux = xSemaphoreCreateMutex();
     xSemaphoreGive(playerStateMux);
   }
 
+
   // create semaphore for time diff buffer to server
   if (latencyBufSemaphoreHandle == NULL) {
     latencyBufSemaphoreHandle = xSemaphoreCreateMutex();
@@ -593,6 +598,15 @@ int start_player() {
     entries -= ((i2sDmaBufMaxLen * i2sDmaBufCnt) / scSet->chkInFrames);
 
     pcmChkQHdl = xQueueCreate(entries, sizeof(pcm_chunk_message_t *));
+    if (pcmChkQHdl == NULL) {
+      ESP_LOGE(TAG, "Failed to create pcm chunk queue (%d entries)", entries);
+      tg0_timer_deinit();
+#if CONFIG_PM_ENABLE
+      esp_pm_lock_release(player_pm_lock_handle);
+#endif
+      playerStarted = false;
+      return -1;
+    }
 
     ESP_LOGI(TAG, "created new queue with %d", entries);
   }
@@ -1547,8 +1561,11 @@ static void player_task(void *pvParameters) {
           queueCreatedWithChkInFrames = __scSet.chkInFrames;
 
           pcmChkQHdl = xQueueCreate(entries, sizeof(pcm_chunk_message_t *));
-
-          ESP_LOGI(TAG, "created new queue with %d", entries);
+          if (pcmChkQHdl == NULL) {
+            ESP_LOGE(TAG, "Failed to create pcm chunk queue (%d entries)", entries);
+          } else {
+            ESP_LOGI(TAG, "created new queue with %d", entries);
+          }
         }
 
         if ((scSet.sr != __scSet.sr) || (scSet.bits != __scSet.bits) ||

components/lightsnapcast/snapcast_protocol_parser.c

@@ -193,7 +193,7 @@ parser_return_state_t parse_codec_header_message(
     if (!read_data(parser, (uint8_t *)codecString, sizeof(codecString)-1)) return PARSER_RESTART_CONNECTION;
 
     codecString[sizeof(codecString)-1] = 0; // null terminate
-    ESP_LOGE(TAG, "Codec : %s... not supported", codecString);
+    ESP_LOGE(TAG, "Codec : %s... not supported (length: %lu)", codecString, codecStringLen);
     ESP_LOGI(TAG,
              "Change encoder codec to "
              "opus, flac or pcm in "

components/network_interface/CMakeLists.txt

@@ -1,3 +1,18 @@
-idf_component_register(SRCS "network_interface.c" "eth_interface.c" "wifi_interface.c"
+set(SRCS "network_interface.c" "wifi_interface.c")
+
+# Only include Ethernet interface if Ethernet is enabled
+if(CONFIG_SNAPCLIENT_USE_INTERNAL_ETHERNET OR CONFIG_SNAPCLIENT_USE_SPI_ETHERNET)
+    list(APPEND SRCS "eth_interface.c")
+endif()
+
+set(PRIV_DEPS driver esp_wifi esp_eth esp_netif esp_timer nvs_flash improv_wifi settings_manager lwip)
+
+# ping is only needed when Ethernet is enabled (used by eth_interface.c)
+if(CONFIG_SNAPCLIENT_USE_INTERNAL_ETHERNET OR CONFIG_SNAPCLIENT_USE_SPI_ETHERNET)
+    list(APPEND PRIV_DEPS ping)
+endif()
+
+idf_component_register(SRCS ${SRCS}
                        INCLUDE_DIRS "include"
-                       PRIV_REQUIRES driver esp_wifi esp_eth esp_netif esp_timer nvs_flash improv_wifi)
+                       PRIV_INCLUDE_DIRS "priv_include"
+                       PRIV_REQUIRES ${PRIV_DEPS})