From 9a43a6ebb772e35ffc0eb320d285439048d7b68f Mon Sep 17 00:00:00 2001
From: Bertrand Songis <bsongis@gmail.com>
Date: Sat, 3 Dec 2016 11:15:40 +0100
Subject: [PATCH] [Sky9x] Remarks from @projectkk2glider taken into account

---
 radio/src/audio_arm.h                  | 16 +++++++++-------
 radio/src/targets/sky9x/CMakeLists.txt |  2 +-
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/radio/src/audio_arm.h b/radio/src/audio_arm.h
index 88b8cfad3..950f6637a 100644
--- a/radio/src/audio_arm.h
+++ b/radio/src/audio_arm.h
@@ -86,7 +86,7 @@ template <unsigned int NUM_BITS> class BitField {
 #define BEEP_KEY_UP_FREQ               (BEEP_DEFAULT_FREQ+150)
 #define BEEP_KEY_DOWN_FREQ             (BEEP_DEFAULT_FREQ-150)
 
-#if defined(PCBSKY9X)
+#if defined(AUDIO_DUAL_BUFFER)
 enum AudioBufferState
 {
   AUDIO_BUFFER_FREE,
@@ -118,7 +118,9 @@ enum AudioBufferState
 struct AudioBuffer {
   audio_data_t data[AUDIO_BUFFER_SIZE];
   uint16_t size;
-  uint8_t  state;
+#if defined(AUDIO_DUAL_BUFFER)
+  uint8_t state;
+#endif
 };
 
 extern AudioBuffer audioBuffers[AUDIO_BUFFER_COUNT];
@@ -317,7 +319,7 @@ class AudioBufferFifo {
     // returns an empty buffer to be filled wit data and put back into FIFO with audioPushBuffer()
     AudioBuffer * getEmptyBuffer() const
     {
-#if defined(PCBSKY9X)
+#if defined(AUDIO_DUAL_BUFFER)
       AudioBuffer * buffer = &audioBuffers[writeIdx];
       return buffer->state == AUDIO_BUFFER_FREE ? buffer : NULL;
 #else
@@ -329,7 +331,7 @@ class AudioBufferFifo {
     void audioPushBuffer()
     {
       audioDisableIrq();
-#if defined(PCBSKY9X)
+#if defined(AUDIO_DUAL_BUFFER)
       AudioBuffer * buffer = &audioBuffers[writeIdx];
       buffer->state = AUDIO_BUFFER_FILLED;
 #endif
@@ -341,7 +343,7 @@ class AudioBufferFifo {
     // returns a pointer to the audio buffer to be played
     const AudioBuffer * getNextFilledBuffer()
     {
-#if defined(PCBSKY9X)
+#if defined(AUDIO_DUAL_BUFFER)
       uint8_t idx = readIdx;
       do {
         AudioBuffer * buffer = &audioBuffers[idx];
@@ -362,7 +364,7 @@ class AudioBufferFifo {
     void freeNextFilledBuffer()
     {
       audioDisableIrq();
-#if defined(PCBSKY9X)
+#if defined(AUDIO_DUAL_BUFFER)
       if (audioBuffers[readIdx].state == AUDIO_BUFFER_PLAYING) {
         audioBuffers[readIdx].state = AUDIO_BUFFER_FREE;
         readIdx = nextBufferIdx(readIdx);
@@ -377,7 +379,7 @@ class AudioBufferFifo {
 
     bool filledAtleast(int noBuffers) const
     {
-#if defined(PCBSKY9X)
+#if defined(AUDIO_DUAL_BUFFER)
       int count = 0;
       for(int n= 0; n<AUDIO_BUFFER_COUNT; ++n) {
         if (audioBuffers[n].state == AUDIO_BUFFER_FILLED) {
diff --git a/radio/src/targets/sky9x/CMakeLists.txt b/radio/src/targets/sky9x/CMakeLists.txt
index f8ce10168..a6152247c 100644
--- a/radio/src/targets/sky9x/CMakeLists.txt
+++ b/radio/src/targets/sky9x/CMakeLists.txt
@@ -39,7 +39,7 @@ else()
   endif()
 endif()
 
-add_definitions(-DPCBSKY9X -DPPM_PIN_SERIAL)
+add_definitions(-DPCBSKY9X -DPPM_PIN_SERIAL -DAUDIO_DUAL_BUFFER)
 add_definitions(-DAUDIO -DVOICE -DRTCLOCK -DVARIO)
 add_definitions(-DEEPROM_VARIANT=0)