diff --git a/Config.h b/Config.h
index c7fb2b8..b201991 100644
--- a/Config.h
+++ b/Config.h
@@ -27,7 +27,7 @@
// Frequencies such as 10.0 Mhz (48000 * 208.333) or 20 Mhz (48000 * 416.666) are not suitable.
//
// For 12 MHz
-// #define EXTERNAL_OSC 12000000
+#define EXTERNAL_OSC 12000000
// For 12.288 MHz
// #define EXTERNAL_OSC 12288000
// For 14.4 MHz
diff --git a/DMRDMORX.cpp b/DMRDMORX.cpp
index 2c3bdef..d11957e 100644
--- a/DMRDMORX.cpp
+++ b/DMRDMORX.cpp
@@ -88,7 +88,7 @@ bool CDMRDMORX::processSample(q15_t sample, uint16_t rssi)
m_rssi[m_dataPtr] = rssi;
m_bitBuffer[m_bitPtr] <<= 1;
- if (sample < 0)
+ if (sample < m_dcLevel)
m_bitBuffer[m_bitPtr] |= 0x01U;
if (m_state == DMORXS_NONE) {
diff --git a/DMRSlotRX.cpp b/DMRSlotRX.cpp
index 0d34560..377aa41 100644
--- a/DMRSlotRX.cpp
+++ b/DMRSlotRX.cpp
@@ -104,7 +104,7 @@ bool CDMRSlotRX::processSample(q15_t sample, uint16_t rssi)
m_rssi[m_dataPtr] = rssi;
m_bitBuffer[m_bitPtr] <<= 1;
- if (sample < 0)
+ if (sample < m_dcLevel)
m_bitBuffer[m_bitPtr] |= 0x01U;
if (m_state == DMRRXS_NONE) {
diff --git a/DStarRX.cpp b/DStarRX.cpp
index 6c4f342..4159c0d 100644
--- a/DStarRX.cpp
+++ b/DStarRX.cpp
@@ -34,10 +34,6 @@ const unsigned int SYNC_POS = 21U * DSTAR_DATA_LENGTH_BITS;
const unsigned int SYNC_SCAN_START = SYNC_POS - 3U;
const unsigned int SYNC_SCAN_END = SYNC_POS + 3U;
-// Generated using [b, a] = butter(1, 0.001) in MATLAB
-static q31_t DC_FILTER[] = {3367972, 0, 3367972, 0, 2140747704, 0}; // {b0, 0, b1, b2, -a1, -a2}
-const uint32_t DC_FILTER_STAGES = 1U; // One Biquad stage
-
// D-Star bit order version of 0x55 0x55 0x6E 0x0A
const uint32_t FRAME_SYNC_DATA = 0x00557650U;
const uint32_t FRAME_SYNC_MASK = 0x00FFFFFFU;
@@ -261,16 +257,8 @@ m_pathMemory2(),
m_pathMemory3(),
m_fecOutput(),
m_rssiAccum(0U),
-m_rssiCount(0U),
-m_dcFilter(),
-m_dcState()
+m_rssiCount(0U)
{
- ::memset(m_dcState, 0x00U, 4U * sizeof(q31_t));
-
- m_dcFilter.numStages = DC_FILTER_STAGES;
- m_dcFilter.pState = m_dcState;
- m_dcFilter.pCoeffs = DC_FILTER;
- m_dcFilter.postShift = 0;
}
void CDStarRX::reset()
@@ -286,24 +274,12 @@ void CDStarRX::reset()
}
void CDStarRX::samples(const q15_t* samples, const uint16_t* rssi, uint8_t length)
-{
- q31_t dc_level = 0;
- q31_t dcVals[20];
- q31_t intSamp[20];
-
- ::arm_q15_to_q31((q15_t*)samples, intSamp, length);
- ::arm_biquad_cascade_df1_q31(&m_dcFilter, intSamp, dcVals, length);
-
- for (uint8_t i = 0U; i < length; i++)
- dc_level += dcVals[i];
-
- dc_level /= length;
-
+{
for (uint16_t i = 0U; i < length; i++) {
m_rssiAccum += rssi[i];
m_rssiCount++;
- bool bit = samples[i] < (q15_t) (dc_level >> 16);
+ bool bit = samples[i] < m_dcLevel;
if (bit != m_prev) {
if (m_pll < (PLLMAX / 2U))
diff --git a/DStarRX.h b/DStarRX.h
index 157fb41..d636cde 100644
--- a/DStarRX.h
+++ b/DStarRX.h
@@ -53,8 +53,6 @@ private:
uint8_t m_fecOutput[42U];
uint32_t m_rssiAccum;
uint16_t m_rssiCount;
- arm_biquad_casd_df1_inst_q31 m_dcFilter;
- q31_t m_dcState[4];
void processNone(bool bit);
void processHeader(bool bit);
diff --git a/Globals.h b/Globals.h
index 5edd5e3..e99022f 100644
--- a/Globals.h
+++ b/Globals.h
@@ -1,5 +1,5 @@
/*
- * Copyright (C) 2015,2016 by Jonathan Naylor G4KLX
+ * Copyright (C) 2015,2016,2017 by Jonathan Naylor G4KLX
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
@@ -93,6 +93,8 @@ extern bool m_duplex;
extern bool m_tx;
extern bool m_dcd;
+extern q15_t m_dcLevel;
+
extern CSerialPort serial;
extern CIO io;
diff --git a/IO.cpp b/IO.cpp
index 7072980..1562e38 100644
--- a/IO.cpp
+++ b/IO.cpp
@@ -26,6 +26,10 @@
static q15_t BOXCAR_FILTER[] = {12000, 12000, 12000, 12000, 12000, 0};
const uint16_t BOXCAR_FILTER_LEN = 6U;
+// Generated using [b, a] = butter(1, 0.001) in MATLAB
+static q31_t DC_FILTER[] = {3367972, 0, 3367972, 0, 2140747704, 0}; // {b0, 0, b1, b2, -a1, -a2}
+const uint32_t DC_FILTER_STAGES = 1U; // One Biquad stage
+
const uint16_t DC_OFFSET = 2048U;
CIO::CIO() :
@@ -48,13 +52,21 @@ m_detect(false),
m_adcOverflow(0U),
m_dacOverflow(0U),
m_watchdog(0U),
-m_lockout(false)
+m_lockout(false),
+m_dcFilter(),
+m_dcState()
{
::memset(m_boxcarState, 0x00U, 30U * sizeof(q15_t));
+ ::memset(m_dcState, 0x00U, 4U * sizeof(q31_t));
m_boxcarFilter.numTaps = BOXCAR_FILTER_LEN;
m_boxcarFilter.pState = m_boxcarState;
m_boxcarFilter.pCoeffs = BOXCAR_FILTER;
+
+ m_dcFilter.numStages = DC_FILTER_STAGES;
+ m_dcFilter.pState = m_dcState;
+ m_dcFilter.pCoeffs = DC_FILTER;
+ m_dcFilter.postShift = 0;
initInt();
}
@@ -132,6 +144,20 @@ void CIO::process()
if (m_lockout)
return;
+
+ q31_t dcLevel = 0;
+ q31_t dcVals[20];
+ q31_t intSamp[20];
+
+ ::arm_q15_to_q31((q15_t*)samples, intSamp, RX_BLOCK_SIZE);
+ ::arm_biquad_cascade_df1_q31(&m_dcFilter, intSamp, dcVals, RX_BLOCK_SIZE);
+
+ for (uint8_t i = 0U; i < RX_BLOCK_SIZE; i++)
+ dcLevel += dcVals[i];
+
+ dcLevel /= RX_BLOCK_SIZE;
+
+ m_dcLevel = q15_t(dcLevel >> 16);
q15_t vals[RX_BLOCK_SIZE];
::arm_fir_fast_q15(&m_boxcarFilter, samples, vals, RX_BLOCK_SIZE);
diff --git a/IO.h b/IO.h
index 8498b75..b303a6f 100644
--- a/IO.h
+++ b/IO.h
@@ -62,6 +62,8 @@ private:
arm_fir_instance_q15 m_boxcarFilter;
q15_t m_boxcarState[30U]; // NoTaps + BlockSize - 1, 6 + 20 - 1 plus some spare
+ arm_biquad_casd_df1_inst_q31 m_dcFilter;
+ q31_t m_dcState[4];
bool m_pttInvert;
q15_t m_rxLevel;
diff --git a/MMDVM.cpp b/MMDVM.cpp
index 6e2be23..4fda95c 100644
--- a/MMDVM.cpp
+++ b/MMDVM.cpp
@@ -36,6 +36,8 @@ bool m_duplex = true;
bool m_tx = false;
bool m_dcd = false;
+q15_t m_dcLevel = 0;
+
CDStarRX dstarRX;
CDStarTX dstarTX;
diff --git a/MMDVM.ino b/MMDVM.ino
index 0aaedb4..a7d694e 100644
--- a/MMDVM.ino
+++ b/MMDVM.ino
@@ -33,6 +33,8 @@ bool m_duplex = true;
bool m_tx = false;
bool m_dcd = false;
+q15_t m_dcLevel = 0;
+
CDStarRX dstarRX;
CDStarTX dstarTX;
diff --git a/MMDVM_STM32F4xx.coproj b/MMDVM_STM32F4xx.coproj
index ccb0fbd..6bfeefe 100644
--- a/MMDVM_STM32F4xx.coproj
+++ b/MMDVM_STM32F4xx.coproj
@@ -98,8 +98,6 @@
-
-
diff --git a/Makefile b/Makefile
index 260ae5d..2358e67 100644
--- a/Makefile
+++ b/Makefile
@@ -87,7 +87,7 @@ MCFLAGS=-mcpu=cortex-m4 -mthumb -mlittle-endian \
# STM32F4 Discovery board:
DEFS_DIS=-DUSE_STDPERIPH_DRIVER -DSTM32F4XX -DSTM32F40_41xxx -DSTM32F4_DISCOVERY -DHSE_VALUE=$(OSC) -DMADEBYMAKEFILE
# Pi board:
-DEFS_PI=-DUSE_STDPERIPH_DRIVER -DSTM32F4XX -DSTM32F446xx -DSTM32F4_PI -DARDUINO_MODE_PINS -DSEND_RSSI_DATA -DSERIAL_REPEATER -DHSE_VALUE=$(OSC) -DMADEBYMAKEFILE
+DEFS_PI=-DUSE_STDPERIPH_DRIVER -DSTM32F4XX -DSTM32F446xx -DSTM32F4_PI -DARDUINO_MODE_PINS -DHSE_VALUE=$(OSC) -DMADEBYMAKEFILE
# STM32F4 Nucleo 446 board:
DEFS_NUCLEO=-DUSE_STDPERIPH_DRIVER -DSTM32F4XX -DSTM32F446xx -DSTM32F4_NUCLEO -DHSE_VALUE=$(OSC) -DMADEBYMAKEFILE
@@ -178,11 +178,13 @@ endif
deploy-pi:
ifneq ($(wildcard /usr/local/bin/stm32flash),)
-/usr/local/bin/stm32flash -i 20,-21,21:-20,21 /dev/ttyAMA0
+ -/usr/local/bin/stm32ld /dev/ttyAMA0 57600 bin/outp.bin
/usr/local/bin/stm32flash -v -w bin/outp.bin -g 0x0 -R -c /dev/ttyAMA0
endif
ifneq ($(wildcard /usr/bin/stm32flash),)
-/usr/bin/stm32flash -i 20,-21,21:-20,21 /dev/ttyAMA0
+ -/usr/bin/stm32ld /dev/ttyAMA0 57600 bin/outp.bin
/usr/bin/stm32flash -v -w bin/outp.bin -g 0x0 -R -c /dev/ttyAMA0
endif
diff --git a/P25RX.cpp b/P25RX.cpp
index 9228c5b..f77bb7d 100644
--- a/P25RX.cpp
+++ b/P25RX.cpp
@@ -95,7 +95,7 @@ void CP25RX::samples(const q15_t* samples, uint16_t* rssi, uint8_t length)
m_rssiCount++;
m_bitBuffer[m_bitPtr] <<= 1;
- if (sample < 0)
+ if (sample < m_dcLevel)
m_bitBuffer[m_bitPtr] |= 0x01U;
m_buffer[m_dataPtr] = sample;
diff --git a/SerialSTM.cpp b/SerialSTM.cpp
index 7f6ba47..e2b128e 100644
--- a/SerialSTM.cpp
+++ b/SerialSTM.cpp
@@ -52,7 +52,6 @@ extern "C" {
/* ************* USART1 ***************** */
#if defined(STM32F4_PI) || (defined(STM32F4_NUCLEO) && defined(STM32F4_NUCLEO_ARDUINO_HEADER))
-volatile uint32_t intcount1;
volatile uint8_t TXSerialfifo1[TX_SERIAL_FIFO_SIZE];
volatile uint8_t RXSerialfifo1[RX_SERIAL_FIFO_SIZE];
volatile uint16_t TXSerialfifohead1, TXSerialfifotail1;
@@ -139,7 +138,6 @@ void USART1_IRQHandler()
}
USART_ClearITPendingBit(USART1, USART_IT_RXNE);
- intcount1++;
}
if (USART_GetITStatus(USART1, USART_IT_TXE)) {
@@ -245,7 +243,6 @@ void WriteUSART1(const uint8_t* data, uint16_t length)
/* ************* USART2 ***************** */
#if defined(STM32F4_NUCLEO)
-volatile uint32_t intcount2;
volatile uint8_t TXSerialfifo2[TX_SERIAL_FIFO_SIZE];
volatile uint8_t RXSerialfifo2[RX_SERIAL_FIFO_SIZE];
volatile uint16_t TXSerialfifohead2, TXSerialfifotail2;
@@ -332,7 +329,6 @@ void USART2_IRQHandler()
}
USART_ClearITPendingBit(USART2, USART_IT_RXNE);
- intcount2++;
}
if (USART_GetITStatus(USART2, USART_IT_TXE)) {
@@ -438,7 +434,6 @@ void WriteUSART2(const uint8_t* data, uint16_t length)
/* ************* USART3 ***************** */
#if defined(STM32F4_DISCOVERY) || defined(STM32F4_PI)
-volatile uint32_t intcount3;
volatile uint8_t TXSerialfifo3[TX_SERIAL_FIFO_SIZE];
volatile uint8_t RXSerialfifo3[RX_SERIAL_FIFO_SIZE];
volatile uint16_t TXSerialfifohead3, TXSerialfifotail3;
@@ -525,7 +520,6 @@ void USART3_IRQHandler()
}
USART_ClearITPendingBit(USART3, USART_IT_RXNE);
- intcount3++;
}
if (USART_GetITStatus(USART3, USART_IT_TXE)) {
@@ -631,7 +625,6 @@ void WriteUSART3(const uint8_t* data, uint16_t length)
/* ************* UART5 ***************** */
#if !(defined(STM32F4_NUCLEO) && defined(STM32F4_NUCLEO_ARDUINO_HEADER))
-volatile uint32_t intcount5;
volatile uint8_t TXSerialfifo5[TX_SERIAL_FIFO_SIZE];
volatile uint8_t RXSerialfifo5[RX_SERIAL_FIFO_SIZE];
volatile uint16_t TXSerialfifohead5, TXSerialfifotail5;
@@ -718,7 +711,6 @@ void UART5_IRQHandler()
}
USART_ClearITPendingBit(UART5, USART_IT_RXNE);
- intcount5++;
}
if (USART_GetITStatus(UART5, USART_IT_TXE)) {
diff --git a/YSFRX.cpp b/YSFRX.cpp
index 88c6f83..e405634 100644
--- a/YSFRX.cpp
+++ b/YSFRX.cpp
@@ -91,7 +91,7 @@ void CYSFRX::samples(const q15_t* samples, uint16_t* rssi, uint8_t length)
m_rssiCount++;
m_bitBuffer[m_bitPtr] <<= 1;
- if (sample < 0)
+ if (sample < m_dcLevel)
m_bitBuffer[m_bitPtr] |= 0x01U;
m_buffer[m_dataPtr] = sample;