Improve platform checks

Author: pschatzmann

src/BluetoothA2DPCommon.cpp

@@ -15,6 +15,9 @@
 
 #include "BluetoothA2DPCommon.h"
 
+#if IS_VALID_PLATFORM
+
+
 BluetoothA2DPCommon* actual_bluetooth_a2dp_common = nullptr;
 
 extern "C" void ccall_bt_app_task_handler(void* arg) {
@@ -596,3 +599,5 @@ unsigned long BluetoothA2DPCommon::get_millis() {
   return (unsigned long)(esp_timer_get_time() / 1000ULL);
 #endif
 }
+
+#endif // platform

src/BluetoothA2DPCommon.h

@@ -22,23 +22,27 @@
  */
 
 #pragma once
+
+// Compile only for ESP32
+#ifdef ARDUINO_ARCH_ESP32
+#  include "sdkconfig.h"
+#endif
+#if defined(CONFIG_IDF_TARGET_ESP32) 
+#  define IS_VALID_PLATFORM true
+#else
+#  error "Please read the Wiki about the supported Platforms!"
+#  define IS_VALID_PLATFORM false
+#endif
+
+// Continue only for ESP32
+#if IS_VALID_PLATFORM
+
 #include "config.h"
 // If you use #include "I2S.h" the i2s functionality is hidden in a namespace
 // this hack prevents any error messages
 #ifdef _I2S_H_INCLUDED
 using namespace esp_i2s;
 #endif
-// Compile only for ESP32
-#if defined(CONFIG_IDF_TARGET_ESP32C2) || \
-    defined(CONFIG_IDF_TARGET_ESP32C3) || \
-    defined(CONFIG_IDF_TARGET_ESP32C5) || \
-    defined(CONFIG_IDF_TARGET_ESP32C6) || \
-    defined(CONFIG_IDF_TARGET_ESP32S2) || \
-    defined(CONFIG_IDF_TARGET_ESP32S3) || \
-    defined(CONFIG_IDF_TARGET_ESP32H2) || \
-    defined(CONFIG_IDF_TARGET_ESP32P4) 
-#error "ESP32C3, ESP32S2, ESP32S3... do not support A2DP"
-#endif
 
 #include <math.h>
 #include <stdbool.h>
@@ -438,4 +442,6 @@ class BluetoothA2DPCommon {
 
 };
 
-extern BluetoothA2DPCommon *actual_bluetooth_a2dp_common;
+extern BluetoothA2DPCommon *actual_bluetooth_a2dp_common;
+
+#endif // platform

src/BluetoothA2DPOutput.cpp

@@ -1,5 +1,7 @@
 #include "BluetoothA2DPOutput.h"
 
+#if IS_VALID_PLATFORM
+
 BluetoothA2DPOutputLegacy::BluetoothA2DPOutputLegacy() {
 #if A2DP_LEGACY_I2S_SUPPORT
   // default i2s port is 0
@@ -259,3 +261,4 @@ void BluetoothA2DPOutputAudioTools::set_output_active(bool active) {
 #endif
 }
 
+#endif

src/BluetoothA2DPSink.cpp

@@ -15,6 +15,8 @@
 
 #include "BluetoothA2DPSink.h"
 
+#if IS_VALID_PLATFORM
+
 // to support static callback functions
 BluetoothA2DPSink *actual_bluetooth_a2dp_sink;
 
@@ -1422,3 +1424,5 @@ void BluetoothA2DPSink::av_hdl_avrc_tg_evt(uint16_t event, void *p_param) {
 }
 
 #endif
+
+#endif // platform

src/BluetoothA2DPSink.h

@@ -15,6 +15,8 @@
 
 #pragma once
 #include "BluetoothA2DPCommon.h"
+#if IS_VALID_PLATFORM
+
 #include "BluetoothA2DPOutput.h"
 #include "freertos/ringbuf.h"
 
@@ -536,3 +538,5 @@ class BluetoothA2DPSink : public BluetoothA2DPCommon {
   virtual bool isSource() { return false; }
 
 };
+
+#endif // platform

src/BluetoothA2DPSinkQueued.cpp

@@ -1,6 +1,8 @@
 
 #include "BluetoothA2DPSinkQueued.h"
 
+#if IS_VALID_PLATFORM
+
 void BluetoothA2DPSinkQueued::bt_i2s_task_start_up(void) {
     ESP_LOGI(BT_APP_TAG, "ringbuffer data empty! mode changed: RINGBUFFER_MODE_PREFETCHING");
     ringbuffer_mode = RINGBUFFER_MODE_PREFETCHING;
@@ -135,3 +137,5 @@ size_t BluetoothA2DPSinkQueued::write_audio(const uint8_t *data, size_t size)
 
     return done ? size : 0;
 }
+
+#endif // platform

src/BluetoothA2DPSinkQueued.h

@@ -2,6 +2,8 @@
 
 #include "BluetoothA2DPSink.h"
 
+#if IS_VALID_PLATFORM
+
 #define RINGBUF_HIGHEST_WATER_LEVEL (32 * 1024)
 #define RINGBUF_PREFETCH_PERCENT 65
 
@@ -98,3 +100,5 @@ class BluetoothA2DPSinkQueued : public BluetoothA2DPSink {
     return (bytes / 4 * 4);
   }
 };
+
+#endif // platform

src/BluetoothA2DPSource.cpp

@@ -16,6 +16,8 @@
 
 #include "BluetoothA2DPSource.h"
 
+#if IS_VALID_PLATFORM
+
 #define APP_RC_CT_TL_RN_VOLUME_CHANGE (1)
 #define BT_APP_HEART_BEAT_EVT (0xff00)
 
@@ -1101,3 +1103,5 @@ void BluetoothA2DPSource::av_hdl_avrc_tg_evt(uint16_t event, void *p_param) {
 }
 
 #endif
+
+#endif // platform

src/BluetoothA2DPSource.h

@@ -23,6 +23,8 @@
 
 #include "BluetoothA2DPCommon.h"
 
+#if IS_VALID_PLATFORM
+
 #ifdef ARDUINO
 #include "Stream.h"
 #endif
@@ -333,3 +335,6 @@ class BluetoothA2DPSource : public BluetoothA2DPCommon {
 
 #endif
 };
+
+
+#endif // platform