summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorjacqueline <me@jacqueline.id.au>2023-08-08 20:25:42 +1000
committerjacqueline <me@jacqueline.id.au>2023-08-08 20:25:42 +1000
commite1181fbe59a835ea9c93d6e067e9757e8c522d3c (patch)
tree2fd61bb93713de8c2205b7b6d0a8c84c49832e93 /src
parentc3f40a8cc37114365ef3ec6f2888df64e5206b39 (diff)
parent592f231627843bc44ebaaa4506aec26da1f56499 (diff)
downloadtangara-fw-e1181fbe59a835ea9c93d6e067e9757e8c522d3c.tar.gz
Merge branch 'main' into opus
Diffstat (limited to 'src')
-rw-r--r--src/app_console/app_console.cpp116
-rw-r--r--src/audio/CMakeLists.txt2
-rw-r--r--src/audio/audio_task.cpp91
-rw-r--r--src/audio/fatfs_audio_input.cpp9
-rw-r--r--src/audio/i2s_audio_output.cpp38
-rw-r--r--src/audio/include/audio_sink.hpp5
-rw-r--r--src/audio/include/audio_task.hpp7
-rw-r--r--src/audio/include/fir.h131
-rw-r--r--src/audio/include/i2s_audio_output.hpp4
-rw-r--r--src/audio/include/resample.hpp44
-rw-r--r--src/audio/include/sink_mixer.hpp71
-rw-r--r--src/audio/include/stream_info.hpp37
-rw-r--r--src/audio/resample.cpp205
-rw-r--r--src/audio/sink_mixer.cpp224
-rw-r--r--src/audio/stream_info.cpp7
-rw-r--r--src/codecs/CMakeLists.txt4
-rw-r--r--src/codecs/codec.cpp3
-rw-r--r--src/codecs/foxenflac.cpp6
-rw-r--r--src/codecs/include/codec.hpp6
-rw-r--r--src/codecs/include/foxenflac.hpp3
-rw-r--r--src/codecs/include/mad.hpp3
-rw-r--r--src/codecs/include/sample.hpp65
-rw-r--r--src/codecs/mad.cpp40
-rw-r--r--src/drivers/CMakeLists.txt3
-rw-r--r--src/drivers/bluetooth.cpp253
-rw-r--r--src/drivers/i2s_dac.cpp1
-rw-r--r--src/drivers/include/bluetooth.hpp108
-rw-r--r--src/drivers/include/i2s_dac.hpp6
-rw-r--r--src/drivers/include/nvs.hpp27
-rw-r--r--src/drivers/nvs.cpp73
-rw-r--r--src/drivers/storage.cpp18
-rw-r--r--src/system_fsm/booting.cpp4
-rw-r--r--src/system_fsm/include/system_fsm.hpp2
-rw-r--r--src/system_fsm/system_fsm.cpp1
-rw-r--r--src/tasks/tasks.cpp16
-rw-r--r--src/tasks/tasks.hpp14
-rw-r--r--src/ui/lvgl_task.cpp19
37 files changed, 1556 insertions, 110 deletions
diff --git a/src/app_console/app_console.cpp b/src/app_console/app_console.cpp
index 6faa27d0..8686ac58 100644
--- a/src/app_console/app_console.cpp
+++ b/src/app_console/app_console.cpp
@@ -7,16 +7,19 @@
#include "app_console.hpp"
#include <dirent.h>
+#include <stdint.h>
#include <algorithm>
#include <cstdint>
#include <cstdio>
#include <cstdlib>
+#include <iomanip>
#include <iostream>
#include <ostream>
#include <sstream>
#include <string>
+#include "FreeRTOSConfig.h"
#include "audio_events.hpp"
#include "audio_fsm.hpp"
#include "database.hpp"
@@ -25,6 +28,8 @@
#include "esp_log.h"
#include "event_queue.hpp"
#include "ff.h"
+#include "freertos/FreeRTOSConfig_arch.h"
+#include "freertos/projdefs.h"
#include "index.hpp"
#include "track.hpp"
@@ -324,6 +329,116 @@ void RegisterDbDump() {
esp_console_cmd_register(&cmd);
}
+int CmdTaskStats(int argc, char** argv) {
+ if (!configUSE_TRACE_FACILITY) {
+ std::cout << "configUSE_TRACE_FACILITY must be enabled" << std::endl;
+ std::cout << "also consider configTASKLIST_USE_COREID" << std::endl;
+ return 1;
+ }
+
+ static const std::string usage = "usage: task_stats";
+ if (argc != 1) {
+ std::cout << usage << std::endl;
+ return 1;
+ }
+
+ // Pad the number of tasks so that uxTaskGetSystemState still returns info if
+ // new tasks are started during measurement.
+ size_t num_tasks = uxTaskGetNumberOfTasks() + 4;
+ TaskStatus_t* start_status = new TaskStatus_t[num_tasks];
+ TaskStatus_t* end_status = new TaskStatus_t[num_tasks];
+ uint32_t start_elapsed_ticks = 0;
+ uint32_t end_elapsed_ticks = 0;
+
+ size_t start_num_tasks =
+ uxTaskGetSystemState(start_status, num_tasks, &start_elapsed_ticks);
+
+ vTaskDelay(pdMS_TO_TICKS(2500));
+
+ size_t end_num_tasks =
+ uxTaskGetSystemState(end_status, num_tasks, &end_elapsed_ticks);
+
+ std::vector<std::pair<uint32_t, std::string>> info_strings;
+ for (int i = 0; i < start_num_tasks; i++) {
+ int k = -1;
+ for (int j = 0; j < end_num_tasks; j++) {
+ if (start_status[i].xHandle == end_status[j].xHandle) {
+ k = j;
+ break;
+ }
+ }
+
+ if (k >= 0) {
+ uint32_t run_time =
+ end_status[k].ulRunTimeCounter - start_status[i].ulRunTimeCounter;
+
+ float time_percent =
+ static_cast<float>(run_time) /
+ static_cast<float>(end_elapsed_ticks - start_elapsed_ticks);
+
+ auto depth = uxTaskGetStackHighWaterMark2(start_status[i].xHandle);
+ float depth_kib = static_cast<float>(depth) / 1024.0f;
+
+ std::ostringstream str;
+ str << start_status[i].pcTaskName;
+ if (str.str().size() < 8) {
+ str << "\t\t";
+ } else {
+ str << "\t";
+ }
+
+ if (configTASKLIST_INCLUDE_COREID) {
+ if (start_status[i].xCoreID == tskNO_AFFINITY) {
+ str << "any\t";
+ } else {
+ str << start_status[i].xCoreID << "\t";
+ }
+ }
+
+ str << std::fixed << std::setprecision(1) << depth_kib;
+ str << " KiB";
+ if (depth_kib >= 10) {
+ str << "\t";
+ } else {
+ str << "\t\t";
+ }
+
+ str << std::fixed << std::setprecision(1) << time_percent * 100;
+ str << "%";
+
+ info_strings.push_back({run_time, str.str()});
+ }
+ }
+
+ std::sort(info_strings.begin(), info_strings.end(),
+ [](const auto& first, const auto& second) {
+ return first.first >= second.first;
+ });
+
+ std::cout << "name\t\t";
+ if (configTASKLIST_INCLUDE_COREID) {
+ std::cout << "core\t";
+ }
+ std::cout << "free stack\trun time" << std::endl;
+ for (const auto& i : info_strings) {
+ std::cout << i.second << std::endl;
+ }
+
+ delete[] start_status;
+ delete[] end_status;
+
+ return 0;
+}
+
+void RegisterTaskStates() {
+ esp_console_cmd_t cmd{.command = "task_stats",
+ .help = "prints performance info for all tasks",
+ .hint = NULL,
+ .func = &CmdTaskStats,
+ .argtable = NULL};
+ esp_console_cmd_register(&cmd);
+}
+
auto AppConsole::RegisterExtraComponents() -> void {
RegisterListDir();
RegisterPlayFile();
@@ -336,6 +451,7 @@ auto AppConsole::RegisterExtraComponents() -> void {
RegisterDbTracks();
RegisterDbIndex();
RegisterDbDump();
+ RegisterTaskStates();
}
} // namespace console
diff --git a/src/audio/CMakeLists.txt b/src/audio/CMakeLists.txt
index 428ea691..bd4ba32d 100644
--- a/src/audio/CMakeLists.txt
+++ b/src/audio/CMakeLists.txt
@@ -5,7 +5,7 @@
idf_component_register(
SRCS "audio_task.cpp" "chunk.cpp" "fatfs_audio_input.cpp"
"stream_message.cpp" "i2s_audio_output.cpp" "stream_buffer.cpp" "track_queue.cpp"
- "stream_event.cpp" "stream_info.cpp" "audio_fsm.cpp"
+ "stream_event.cpp" "stream_info.cpp" "audio_fsm.cpp" "sink_mixer.cpp" "resample.cpp"
INCLUDE_DIRS "include"
REQUIRES "codecs" "drivers" "cbor" "result" "tasks" "span" "memory" "tinyfsm" "database" "system_fsm" "playlist")
diff --git a/src/audio/audio_task.cpp b/src/audio/audio_task.cpp
index 7e0fb207..75b44594 100644
--- a/src/audio/audio_task.cpp
+++ b/src/audio/audio_task.cpp
@@ -34,6 +34,8 @@
#include "freertos/queue.h"
#include "freertos/ringbuf.h"
#include "pipeline.hpp"
+#include "sample.hpp"
+#include "sink_mixer.hpp"
#include "span.hpp"
#include "arena.hpp"
@@ -107,7 +109,10 @@ auto Timer::bytes_to_samples(uint32_t bytes) -> uint32_t {
auto AudioTask::Start(IAudioSource* source, IAudioSink* sink) -> AudioTask* {
AudioTask* task = new AudioTask(source, sink);
- tasks::StartPersistent<tasks::Type::kAudio>([=]() { task->Main(); });
+ // Pin to CORE1 because codecs should be fixed point anyway, and being on
+ // the opposite core to the mixer maximises throughput in the worst case
+ // (some heavy codec like opus + resampling for bluetooth).
+ tasks::StartPersistent<tasks::Type::kAudio>(1, [=]() { task->Main(); });
return task;
}
@@ -115,14 +120,12 @@ AudioTask::AudioTask(IAudioSource* source, IAudioSink* sink)
: source_(source),
sink_(sink),
codec_(),
+ mixer_(new SinkMixer(sink->stream())),
timer_(),
has_begun_decoding_(false),
current_input_format_(),
current_output_format_(),
- sample_buffer_(reinterpret_cast<std::byte*>(
- heap_caps_malloc(kSampleBufferSize,
- MALLOC_CAP_INTERNAL | MALLOC_CAP_8BIT))),
- sample_buffer_len_(kSampleBufferSize) {}
+ codec_buffer_(new RawStream(kSampleBufferSize)) {}
void AudioTask::Main() {
for (;;) {
@@ -226,7 +229,7 @@ auto AudioTask::BeginDecoding(InputStream& stream) -> bool {
codecs::ICodec::OutputFormat format = res.second.value();
StreamInfo::Pcm new_format{
.channels = format.num_channels,
- .bits_per_sample = format.bits_per_sample,
+ .bits_per_sample = 32,
.sample_rate = format.sample_rate_hz,
};
@@ -246,13 +249,18 @@ auto AudioTask::BeginDecoding(InputStream& stream) -> bool {
return false;
}
+ OutputStream writer{codec_buffer_.get()};
+ writer.prepare(new_format, {});
+
return true;
}
auto AudioTask::ContinueDecoding(InputStream& stream) -> bool {
while (!stream.data().empty()) {
- auto res = codec_->ContinueStream(stream.data(),
- {sample_buffer_, sample_buffer_len_});
+ OutputStream writer{codec_buffer_.get()};
+
+ auto res =
+ codec_->ContinueStream(stream.data(), writer.data_as<sample::Sample>());
stream.consume(res.first);
@@ -263,9 +271,10 @@ auto AudioTask::ContinueDecoding(InputStream& stream) -> bool {
return false;
}
} else {
- xStreamBufferSend(sink_->stream(), sample_buffer_,
- res.second->bytes_written, portMAX_DELAY);
- timer_->AddBytes(res.second->bytes_written);
+ writer.add(res.second->samples_written * sizeof(sample::Sample));
+
+ InputStream reader{codec_buffer_.get()};
+ SendToSink(reader);
}
}
return true;
@@ -284,21 +293,23 @@ auto AudioTask::FinishDecoding(InputStream& stream) -> void {
std::unique_ptr<RawStream> mad_buffer;
mad_buffer.reset(new RawStream(stream.data().size_bytes() + 8));
- OutputStream writer{mad_buffer.get()};
+ OutputStream mad_writer{mad_buffer.get()};
std::copy(stream.data().begin(), stream.data().end(),
- writer.data().begin());
- std::fill(writer.data().begin(), writer.data().end(), std::byte{0});
+ mad_writer.data().begin());
+ std::fill(mad_writer.data().begin(), mad_writer.data().end(), std::byte{0});
InputStream padded_stream{mad_buffer.get()};
- auto res = codec_->ContinueStream(stream.data(),
- {sample_buffer_, sample_buffer_len_});
+ OutputStream writer{codec_buffer_.get()};
+ auto res =
+ codec_->ContinueStream(stream.data(), writer.data_as<sample::Sample>());
if (res.second.has_error()) {
return;
}
- xStreamBufferSend(sink_->stream(), sample_buffer_,
- res.second->bytes_written, portMAX_DELAY);
- timer_->AddBytes(res.second->bytes_written);
+ writer.add(res.second->samples_written * sizeof(sample::Sample));
+
+ InputStream reader{codec_buffer_.get()};
+ SendToSink(reader);
}
}
@@ -319,24 +330,31 @@ auto AudioTask::ForwardPcmStream(StreamInfo::Pcm& format,
xStreamBufferSend(sink_->stream(), samples.data(), samples.size_bytes(),
portMAX_DELAY);
timer_->AddBytes(samples.size_bytes());
+ InputStream reader{codec_buffer_.get()};
+ SendToSink(reader);
+
return true;
}
auto AudioTask::ConfigureSink(const StreamInfo::Pcm& format,
const Duration& duration) -> bool {
if (format != current_output_format_) {
- // The new format is different to the old one. Wait for the sink to drain
- // before continuing.
- while (!xStreamBufferIsEmpty(sink_->stream())) {
- ESP_LOGI(kTag, "waiting for sink stream to drain...");
- // TODO(jacqueline): Get the sink drain ISR to notify us of this
- // via semaphore instead of busy-ish waiting.
- vTaskDelay(pdMS_TO_TICKS(100));
- }
+ current_output_format_ = format;
+ StreamInfo::Pcm new_sink_format = sink_->PrepareFormat(format);
+ if (new_sink_format != current_sink_format_) {
+ current_sink_format_ = new_sink_format;
+
+ // The new format is different to the old one. Wait for the sink to drain
+ // before continuing.
+ while (!xStreamBufferIsEmpty(sink_->stream())) {
+ ESP_LOGI(kTag, "waiting for sink stream to drain...");
+ // TODO(jacqueline): Get the sink drain ISR to notify us of this
+ // via semaphore instead of busy-ish waiting.
+ vTaskDelay(pdMS_TO_TICKS(10));
+ }
- ESP_LOGI(kTag, "configuring sink");
- if (!sink_->Configure(format)) {
- return false;
+ ESP_LOGI(kTag, "configuring sink");
+ sink_->Configure(new_sink_format);
}
}
@@ -345,4 +363,17 @@ auto AudioTask::ConfigureSink(const StreamInfo::Pcm& format,
return true;
}
+auto AudioTask::SendToSink(InputStream& stream) -> void {
+ std::size_t bytes_to_send = stream.data().size_bytes();
+ std::size_t bytes_sent;
+ if (stream.info().format_as<StreamInfo::Pcm>() == current_sink_format_) {
+ bytes_sent = xStreamBufferSend(sink_->stream(), stream.data().data(),
+ bytes_to_send, portMAX_DELAY);
+ stream.consume(bytes_sent);
+ } else {
+ bytes_sent = mixer_->MixAndSend(stream, current_sink_format_.value());
+ }
+ timer_->AddBytes(bytes_sent);
+}
+
} // namespace audio
diff --git a/src/audio/fatfs_audio_input.cpp b/src/audio/fatfs_audio_input.cpp
index 0c3ef20d..73586f09 100644
--- a/src/audio/fatfs_audio_input.cpp
+++ b/src/audio/fatfs_audio_input.cpp
@@ -30,6 +30,7 @@
#include "freertos/portmacro.h"
#include "freertos/projdefs.h"
#include "future_fetcher.hpp"
+#include "idf_additions.h"
#include "span.hpp"
#include "stream_info.hpp"
#include "tag_parser.hpp"
@@ -40,8 +41,8 @@ static const char* kTag = "SRC";
namespace audio {
-static constexpr UINT kFileBufferSize = 4096 * 2;
-static constexpr UINT kStreamerBufferSize = 4096;
+static constexpr UINT kFileBufferSize = 8 * 1024;
+static constexpr UINT kStreamerBufferSize = 64 * 1024;
static StreamBufferHandle_t sForwardDest = nullptr;
@@ -143,7 +144,9 @@ FatfsAudioInput::FatfsAudioInput(
: IAudioSource(),
tag_parser_(tag_parser),
has_data_(xSemaphoreCreateBinary()),
- streamer_buffer_(xStreamBufferCreate(kStreamerBufferSize, 1)),
+ streamer_buffer_(xStreamBufferCreateWithCaps(kStreamerBufferSize,
+ 1,
+ MALLOC_CAP_SPIRAM)),
streamer_(new FileStreamer(streamer_buffer_, has_data_)),
input_buffer_(new RawStream(kFileBufferSize)),
source_mutex_(),
diff --git a/src/audio/i2s_audio_output.cpp b/src/audio/i2s_audio_output.cpp
index bb413b38..d60ddfa4 100644
--- a/src/audio/i2s_audio_output.cpp
+++ b/src/audio/i2s_audio_output.cpp
@@ -115,10 +115,20 @@ auto I2SAudioOutput::AdjustVolumeDown() -> bool {
return true;
}
-auto I2SAudioOutput::Configure(const StreamInfo::Pcm& pcm) -> bool {
+auto I2SAudioOutput::PrepareFormat(const StreamInfo::Pcm& orig)
+ -> StreamInfo::Pcm {
+ return StreamInfo::Pcm{
+ .channels = std::min<uint8_t>(orig.channels, 2),
+ .bits_per_sample = std::clamp<uint8_t>(orig.bits_per_sample, 16, 32),
+ .sample_rate = 44100,
+ //.sample_rate = std::clamp<uint32_t>(orig.sample_rate, 8000, 96000),
+ };
+}
+
+auto I2SAudioOutput::Configure(const StreamInfo::Pcm& pcm) -> void {
if (current_config_ && pcm == *current_config_) {
ESP_LOGI(kTag, "ignoring unchanged format");
- return true;
+ return;
}
ESP_LOGI(kTag, "incoming audio stream: %u ch %u bpp @ %lu Hz", pcm.channels,
@@ -134,7 +144,7 @@ auto I2SAudioOutput::Configure(const StreamInfo::Pcm& pcm) -> bool {
break;
default:
ESP_LOGE(kTag, "dropping stream with out of bounds channels");
- return false;
+ return;
}
drivers::I2SDac::BitsPerSample bps;
@@ -150,30 +160,36 @@ auto I2SAudioOutput::Configure(const StreamInfo::Pcm& pcm) -> bool {
break;
default:
ESP_LOGE(kTag, "dropping stream with unknown bps");
- return false;
+ return;
}
drivers::I2SDac::SampleRate sample_rate;
switch (pcm.sample_rate) {
+ case 8000:
+ sample_rate = drivers::I2SDac::SAMPLE_RATE_8;
+ break;
+ case 32000:
+ sample_rate = drivers::I2SDac::SAMPLE_RATE_32;
+ break;
case 44100:
sample_rate = drivers::I2SDac::SAMPLE_RATE_44_1;
break;
case 48000:
sample_rate = drivers::I2SDac::SAMPLE_RATE_48;
break;
+ case 88200:
+ sample_rate = drivers::I2SDac::SAMPLE_RATE_88_2;
+ break;
+ case 96000:
+ sample_rate = drivers::I2SDac::SAMPLE_RATE_96;
+ break;
default:
ESP_LOGE(kTag, "dropping stream with unknown rate");
- return false;
+ return;
}
dac_->Reconfigure(ch, bps, sample_rate);
current_config_ = pcm;
-
- return true;
-}
-
-auto I2SAudioOutput::Send(const cpp::span<std::byte>& data) -> void {
- dac_->WriteData(data);
}
} // namespace audio
diff --git a/src/audio/include/audio_sink.hpp b/src/audio/include/audio_sink.hpp
index 261f7c79..28acdc31 100644
--- a/src/audio/include/audio_sink.hpp
+++ b/src/audio/include/audio_sink.hpp
@@ -17,7 +17,6 @@ namespace audio {
class IAudioSink {
private:
- // TODO: tune. at least about 12KiB seems right for mp3
static const std::size_t kDrainBufferSize = 24 * 1024;
StreamBufferHandle_t stream_;
@@ -38,8 +37,8 @@ class IAudioSink {
virtual auto AdjustVolumeUp() -> bool = 0;
virtual auto AdjustVolumeDown() -> bool = 0;
- virtual auto Configure(const StreamInfo::Pcm& format) -> bool = 0;
- virtual auto Send(const cpp::span<std::byte>& data) -> void = 0;
+ virtual auto PrepareFormat(const StreamInfo::Pcm&) -> StreamInfo::Pcm = 0;
+ virtual auto Configure(const StreamInfo::Pcm& format) -> void = 0;
auto stream() -> StreamBufferHandle_t { return stream_; }
};
diff --git a/src/audio/include/audio_task.hpp b/src/audio/include/audio_task.hpp
index f6e9789b..b27aa039 100644
--- a/src/audio/include/audio_task.hpp
+++ b/src/audio/include/audio_task.hpp
@@ -14,6 +14,7 @@
#include "audio_source.hpp"
#include "codec.hpp"
#include "pipeline.hpp"
+#include "sink_mixer.hpp"
#include "stream_info.hpp"
namespace audio {
@@ -63,18 +64,20 @@ class AudioTask {
auto ForwardPcmStream(StreamInfo::Pcm&, cpp::span<const std::byte>) -> bool;
auto ConfigureSink(const StreamInfo::Pcm&, const Duration&) -> bool;
+ auto SendToSink(InputStream&) -> void;
IAudioSource* source_;
IAudioSink* sink_;
std::unique_ptr<codecs::ICodec> codec_;
+ std::unique_ptr<SinkMixer> mixer_;
std::unique_ptr<Timer> timer_;
bool has_begun_decoding_;
std::optional<StreamInfo::Format> current_input_format_;
std::optional<StreamInfo::Pcm> current_output_format_;
+ std::optional<StreamInfo::Pcm> current_sink_format_;
- std::byte* sample_buffer_;
- std::size_t sample_buffer_len_;
+ std::unique_ptr<RawStream> codec_buffer_;
};
} // namespace audio
diff --git a/src/audio/include/fir.h b/src/audio/include/fir.h
new file mode 100644
index 00000000..e50c3eff
--- /dev/null
+++ b/src/audio/include/fir.h
@@ -0,0 +1,131 @@
+/*
+ * FIR filter coefficients from resample-1.x smallfilter.h
+ * see Digital Audio Resampling Home Page located at
+ * http://ccrma.stanford.edu/~jos/resample/
+ */
+32767, 32766, 32764, 32760, 32755, 32749, 32741, 32731, 32721, 32708,
+32695, 32679, 32663, 32645, 32625, 32604, 32582, 32558, 32533, 32506,
+32478, 32448, 32417, 32385, 32351, 32316, 32279, 32241, 32202, 32161,
+32119, 32075, 32030, 31984, 31936, 31887, 31836, 31784, 31731, 31676,
+31620, 31563, 31504, 31444, 31383, 31320, 31256, 31191, 31124, 31056,
+30987, 30916, 30845, 30771, 30697, 30621, 30544, 30466, 30387, 30306,
+30224, 30141, 30057, 29971, 29884, 29796, 29707, 29617, 29525, 29433,
+29339, 29244, 29148, 29050, 28952, 28852, 28752, 28650, 28547, 28443,
+28338, 28232, 28125, 28017, 27908, 27797, 27686, 27574, 27461, 27346,
+27231, 27115, 26998, 26879, 26760, 26640, 26519, 26398, 26275, 26151,
+26027, 25901, 25775, 25648, 25520, 25391, 25262, 25131, 25000, 24868,
+24735, 24602, 24467, 24332, 24197, 24060, 23923, 23785, 23647, 23507,
+23368, 23227, 23086, 22944, 22802, 22659, 22515, 22371, 22226, 22081,
+21935, 21789, 21642, 21494, 21346, 21198, 21049, 20900, 20750, 20600,
+20449, 20298, 20146, 19995, 19842, 19690, 19537, 19383, 19230, 19076,
+18922, 18767, 18612, 18457, 18302, 18146, 17990, 17834, 17678, 17521,
+17365, 17208, 17051, 16894, 16737, 16579, 16422, 16264, 16106, 15949,
+15791, 15633, 15475, 15317, 15159, 15001, 14843, 14685, 14527, 14369,
+14212, 14054, 13896, 13739, 13581, 13424, 13266, 13109, 12952, 12795,
+12639, 12482, 12326, 12170, 12014, 11858, 11703, 11548, 11393, 11238,
+11084, 10929, 10776, 10622, 10469, 10316, 10164, 10011, 9860, 9708,
+9557, 9407, 9256, 9106, 8957, 8808, 8659, 8511, 8364, 8216, 8070,
+7924, 7778, 7633, 7488, 7344, 7200, 7057, 6914, 6773, 6631, 6490,
+6350, 6210, 6071, 5933, 5795, 5658, 5521, 5385, 5250, 5115, 4981,
+4848, 4716, 4584, 4452, 4322, 4192, 4063, 3935, 3807, 3680, 3554,
+3429, 3304, 3180, 3057, 2935, 2813, 2692, 2572, 2453, 2335, 2217,
+2101, 1985, 1870, 1755, 1642, 1529, 1418, 1307, 1197, 1088, 979, 872,
+765, 660, 555, 451, 348, 246, 145, 44, -54, -153, -250, -347, -443,
+-537, -631, -724, -816, -908, -998, -1087, -1175, -1263, -1349, -1435,
+-1519, -1603, -1685, -1767, -1848, -1928, -2006, -2084, -2161, -2237,
+-2312, -2386, -2459, -2531, -2603, -2673, -2742, -2810, -2878, -2944,
+-3009, -3074, -3137, -3200, -3261, -3322, -3381, -3440, -3498, -3554,
+-3610, -3665, -3719, -3772, -3824, -3875, -3925, -3974, -4022, -4069,
+-4116, -4161, -4205, -4249, -4291, -4333, -4374, -4413, -4452, -4490,
+-4527, -4563, -4599, -4633, -4666, -4699, -4730, -4761, -4791, -4820,
+-4848, -4875, -4901, -4926, -4951, -4974, -4997, -5019, -5040, -5060,
+-5080, -5098, -5116, -5133, -5149, -5164, -5178, -5192, -5205, -5217,
+-5228, -5238, -5248, -5257, -5265, -5272, -5278, -5284, -5289, -5293,
+-5297, -5299, -5301, -5303, -5303, -5303, -5302, -5300, -5298, -5295,
+-5291, -5287, -5282, -5276, -5270, -5263, -5255, -5246, -5237, -5228,
+-5217, -5206, -5195, -5183, -5170, -5157, -5143, -5128, -5113, -5097,
+-5081, -5064, -5047, -5029, -5010, -4991, -4972, -4952, -4931, -4910,
+-4889, -4867, -4844, -4821, -4797, -4774, -4749, -4724, -4699, -4673,
+-4647, -4620, -4593, -4566, -4538, -4510, -4481, -4452, -4422, -4393,
+-4363, -4332, -4301, -4270, -4238, -4206, -4174, -4142, -4109, -4076,
+-4042, -4009, -3975, -3940, -3906, -3871, -3836, -3801, -3765, -3729,
+-3693, -3657, -3620, -3584, -3547, -3510, -3472, -3435, -3397, -3360,
+-3322, -3283, -3245, -3207, -3168, -3129, -3091, -3052, -3013, -2973,
+-2934, -2895, -2855, -2816, -2776, -2736, -2697, -2657, -2617, -2577,
+-2537, -2497, -2457, -2417, -2377, -2337, -2297, -2256, -2216, -2176,
+-2136, -2096, -2056, -2016, -1976, -1936, -1896, -1856, -1817, -1777,
+-1737, -1698, -1658, -1619, -1579, -1540, -1501, -1462, -1423, -1384,
+-1345, -1306, -1268, -1230, -1191, -1153, -1115, -1077, -1040, -1002,
+-965, -927, -890, -854, -817, -780, -744, -708, -672, -636, -600,
+-565, -530, -494, -460, -425, -391, -356, -322, -289, -255, -222,
+-189, -156, -123, -91, -59, -27, 4, 35, 66, 97, 127, 158, 188, 218,
+247, 277, 306, 334, 363, 391, 419, 447, 474, 501, 528, 554, 581, 606,
+632, 657, 683, 707, 732, 756, 780, 803, 827, 850, 872, 895, 917, 939,
+960, 981, 1002, 1023, 1043, 1063, 1082, 1102, 1121, 1139, 1158, 1176,
+1194, 1211, 1228, 1245, 1262, 1278, 1294, 1309, 1325, 1340, 1354,
+1369, 1383, 1397, 1410, 1423, 1436, 1448, 1461, 1473, 1484, 1496,
+1507, 1517, 1528, 1538, 1548, 1557, 1566, 1575, 1584, 1592, 1600,
+1608, 1616, 1623, 1630, 1636, 1643, 1649, 1654, 1660, 1665, 1670,
+1675, 1679, 1683, 1687, 1690, 1694, 1697, 1700, 1702, 1704, 1706,
+1708, 1709, 1711, 1712, 1712, 1713, 1713, 1713, 1713, 1712, 1711,
+1710, 1709, 1708, 1706, 1704, 1702, 1700, 1697, 1694, 1691, 1688,
+1685, 1681, 1677, 1673, 1669, 1664, 1660, 1655, 1650, 1644, 1639,
+1633, 1627, 1621, 1615, 1609, 1602, 1596, 1589, 1582, 1575, 1567,
+1560, 1552, 1544, 1536, 1528, 1520, 1511, 1503, 1494, 1485, 1476,
+1467, 1458, 1448, 1439, 1429, 1419, 1409, 1399, 1389, 1379, 1368,
+1358, 1347, 1337, 1326, 1315, 1304, 1293, 1282, 1271, 1260, 1248,
+1237, 1225, 1213, 1202, 1190, 1178, 1166, 1154, 1142, 1130, 1118,
+1106, 1094, 1081, 1069, 1057, 1044, 1032, 1019, 1007, 994, 981, 969,
+956, 943, 931, 918, 905, 892, 879, 867, 854, 841, 828, 815, 802, 790,
+777, 764, 751, 738, 725, 713, 700, 687, 674, 662, 649, 636, 623, 611,
+598, 585, 573, 560, 548, 535, 523, 510, 498, 486, 473, 461, 449, 437,
+425, 413, 401, 389, 377, 365, 353, 341, 330, 318, 307, 295, 284, 272,
+261, 250, 239, 228, 217, 206, 195, 184, 173, 163, 152, 141, 131, 121,
+110, 100, 90, 80, 70, 60, 51, 41, 31, 22, 12, 3, -5, -14, -23, -32,
+-41, -50, -59, -67, -76, -84, -93, -101, -109, -117, -125, -133, -140,
+-148, -156, -163, -170, -178, -185, -192, -199, -206, -212, -219,
+-226, -232, -239, -245, -251, -257, -263, -269, -275, -280, -286,
+-291, -297, -302, -307, -312, -317, -322, -327, -332, -336, -341,
+-345, -349, -354, -358, -362, -366, -369, -373, -377, -380, -384,
+-387, -390, -394, -397, -400, -402, -405, -408, -411, -413, -416,
+-418, -420, -422, -424, -426, -428, -430, -432, -433, -435, -436,
+-438, -439, -440, -442, -443, -444, -445, -445, -446, -447, -447,
+-448, -448, -449, -449, -449, -449, -449, -449, -449, -449, -449,
+-449, -449, -448, -448, -447, -447, -446, -445, -444, -443, -443,
+-442, -441, -440, -438, -437, -436, -435, -433, -432, -430, -429,
+-427, -426, -424, -422, -420, -419, -417, -415, -413, -411, -409,
+-407, -405, -403, -400, -398, -396, -393, -391, -389, -386, -384,
+-381, -379, -376, -374, -371, -368, -366, -363, -360, -357, -355,
+-352, -349, -346, -343, -340, -337, -334, -331, -328, -325, -322,
+-319, -316, -313, -310, -307, -304, -301, -298, -294, -291, -288,
+-285, -282, -278, -275, -272, -269, -265, -262, -259, -256, -252,
+-249, -246, -243, -239, -236, -233, -230, -226, -223, -220, -217,
+-213, -210, -207, -204, -200, -197, -194, -191, -187, -184, -181,
+-178, -175, -172, -168, -165, -162, -159, -156, -153, -150, -147,
+-143, -140, -137, -134, -131, -128, -125, -122, -120, -117, -114,
+-111, -108, -105, -102, -99, -97, -94, -91, -88, -86, -83, -80, -78,
+-75, -72, -70, -67, -65, -62, -59, -57, -55, -52, -50, -47, -45, -43,
+-40, -38, -36, -33, -31, -29, -27, -25, -22, -20, -18, -16, -14, -12,
+-10, -8, -6, -4, -2, 0, 0, 2, 4, 6, 8, 9, 11, 13, 14, 16, 17, 19, 21,
+22, 24, 25, 27, 28, 29, 31, 32, 33, 35, 36, 37, 38, 40, 41, 42, 43,
+44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54, 55, 56, 56, 57, 58, 59,
+59, 60, 61, 62, 62, 63, 63, 64, 64, 65, 66, 66, 66, 67, 67, 68, 68,
+69, 69, 69, 70, 70, 70, 70, 71, 71, 71, 71, 71, 72, 72, 72, 72, 72,
+72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72, 72,
+72, 71, 71, 71, 71, 71, 70, 70, 70, 70, 69, 69, 69, 69, 68, 68, 68,
+67, 67, 67, 66, 66, 66, 65, 65, 64, 64, 64, 63, 63, 62, 62, 62, 61,
+61, 60, 60, 59, 59, 58, 58, 58, 57, 57, 56, 56, 55, 55, 54, 54, 53,
+53, 52, 52, 51, 51, 50, 50, 49, 48, 48, 47, 47, 46, 46, 45, 45, 44,
+44, 43, 43, 42, 42, 41, 41, 40, 39, 39, 38, 38, 37, 37, 36, 36, 35,
+35, 34, 34, 33, 33, 32, 32, 31, 31, 30, 30, 29, 29, 28, 28, 27, 27,
+26, 26, 25, 25, 24, 24, 23, 23, 23, 22, 22, 21, 21, 20, 20, 20, 19,
+19, 18, 18, 17, 17, 17, 16, 16, 15, 15, 15, 14, 14, 14, 13, 13, 12,
+12, 12, 11, 11, 11, 10, 10, 10, 9, 9, 9, 9, 8, 8, 8, 7, 7, 7, 7, 6, 6,
+6, 6, 5, 5, 5, 5, 4, 4, 4, 4, 3, 3, 3, 3, 3, 2, 2, 2, 2, 2, 1, 1, 1,
+1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -1, -1,
+-1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -2, -2, -2, -2, -2, -2, -2,
+-2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2,
+-2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2,
+-2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2, -2,
+-2, -2, -2, -2, -2, -2, -2, -2, -1, -1, -1, -1, -1, -1, -1, -1, -1,
+-1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1, -1,
diff --git a/src/audio/include/i2s_audio_output.hpp b/src/audio/include/i2s_audio_output.hpp
index 43155711..e0f791c5 100644
--- a/src/audio/include/i2s_audio_output.hpp
+++ b/src/audio/include/i2s_audio_output.hpp
@@ -35,8 +35,8 @@ class I2SAudioOutput : public IAudioSink {
auto AdjustVolumeUp() -> bool override;
auto AdjustVolumeDown() -> bool override;
- auto Configure(const StreamInfo::Pcm& format) -> bool override;
- auto Send(const cpp::span<std::byte>& data) -> void override;
+ auto PrepareFormat(const StreamInfo::Pcm&) -> StreamInfo::Pcm override;
+ auto Configure(const StreamInfo::Pcm& format) -> void override;
I2SAudioOutput(const I2SAudioOutput&) = delete;
I2SAudioOutput& operator=(const I2SAudioOutput&) = delete;
diff --git a/src/audio/include/resample.hpp b/src/audio/include/resample.hpp
new file mode 100644
index 00000000..3855415a
--- /dev/null
+++ b/src/audio/include/resample.hpp
@@ -0,0 +1,44 @@
+#pragma once
+
+#include <sys/_stdint.h>
+#include <vector>
+
+#include "span.hpp"
+
+#include "sample.hpp"
+
+namespace audio {
+
+class Resampler {
+ public:
+ Resampler(uint32_t source_sample_rate,
+ uint32_t target_sample_rate,
+ uint8_t num_channels);
+
+ ~Resampler();
+
+ auto source_sample_rate() -> uint32_t { return source_sample_rate_; }
+ auto target_sample_rate() -> uint32_t { return target_sample_rate_; }
+ auto channels() -> uint_fast8_t { return num_channels_; }
+
+ auto Process(cpp::span<const sample::Sample> input,
+ cpp::span<sample::Sample> output,
+ bool end_of_data) -> std::pair<size_t, size_t>;
+
+ private:
+ auto Subsample(int channel) -> float;
+ auto ApplyFilter(cpp::span<float> filter, cpp::span<float> input) -> float;
+
+ uint32_t source_sample_rate_;
+ uint32_t target_sample_rate_;
+ float factor_;
+ uint8_t num_channels_;
+
+ std::vector<float*> channel_buffers_;
+ size_t channel_buffer_size_;
+
+ float output_offset_;
+ int32_t input_index_;
+};
+
+} // namespace audio \ No newline at end of file
diff --git a/src/audio/include/sink_mixer.hpp b/src/audio/include/sink_mixer.hpp
new file mode 100644
index 00000000..d1e9aa8a
--- /dev/null
+++ b/src/audio/include/sink_mixer.hpp
@@ -0,0 +1,71 @@
+/*
+ * Copyright 2023 jacqueline <me@jacqueline.id.au>
+ *
+ * SPDX-License-Identifier: GPL-3.0-only
+ */
+
+#pragma once
+
+#include <sys/_stdint.h>
+#include <cstdint>
+#include <memory>
+
+#include "resample.hpp"
+#include "sample.hpp"
+
+#include "audio_decoder.hpp"
+#include "audio_sink.hpp"
+#include "audio_source.hpp"
+#include "codec.hpp"
+#include "pipeline.hpp"
+#include "stream_info.hpp"
+
+namespace audio {
+
+/*
+ * Handles the final downmix + resample + quantisation stage of audio,
+ * generation sending the result directly to an IAudioSink.
+ */
+class SinkMixer {
+ public:
+ SinkMixer(StreamBufferHandle_t dest);
+ ~SinkMixer();
+
+ auto MixAndSend(InputStream&, const StreamInfo::Pcm&) -> std::size_t;
+
+ private:
+ auto Main() -> void;
+
+ auto SetTargetFormat(const StreamInfo::Pcm& format) -> void;
+ auto HandleBytes() -> void;
+
+ auto Resample(InputStream&, OutputStream&) -> bool;
+ auto ApplyDither(cpp::span<sample::Sample> samples, uint_fast8_t bits)
+ -> void;
+ auto Downscale(cpp::span<sample::Sample>, cpp::span<int16_t>) -> void;
+
+ enum class Command {
+ kReadBytes,
+ kSetSourceFormat,
+ kSetTargetFormat,
+ };
+
+ struct Args {
+ Command cmd;
+ StreamInfo::Pcm format;
+ };
+
+ QueueHandle_t commands_;
+ SemaphoreHandle_t is_idle_;
+
+ std::unique_ptr<Resampler> resampler_;
+
+ std::unique_ptr<RawStream> input_stream_;
+ std::unique_ptr<RawStream> resampled_stream_;
+
+ StreamInfo::Pcm target_format_;
+ StreamBufferHandle_t source_;
+ StreamBufferHandle_t sink_;
+};
+
+} // namespace audio
diff --git a/src/audio/include/stream_info.hpp b/src/audio/include/stream_info.hpp
index d48c39a8..01dd282a 100644
--- a/src/audio/include/stream_info.hpp
+++ b/src/audio/include/stream_info.hpp
@@ -16,6 +16,7 @@
#include <utility>
#include <variant>
+#include "esp_heap_caps.h"
#include "freertos/FreeRTOS.h"
#include "freertos/ringbuf.h"
#include "freertos/stream_buffer.h"
@@ -56,6 +57,18 @@ class StreamInfo {
bool operator==(const Encoded&) const = default;
};
+ /*
+ * Two-channel, interleaved, 32-bit floating point pcm samples.
+ */
+ struct FloatingPointPcm {
+ // Number of channels in this stream.
+ uint8_t channels;
+ // The sample rate.
+ uint32_t sample_rate;
+
+ bool operator==(const FloatingPointPcm&) const = default;
+ };
+
struct Pcm {
// Number of channels in this stream.
uint8_t channels;
@@ -64,10 +77,14 @@ class StreamInfo {
// The sample rate.
uint32_t sample_rate;
+ auto bytes_per_sample() const -> uint8_t {
+ return bits_per_sample == 16 ? 2 : 4;
+ }
+
bool operator==(const Pcm&) const = default;
};
- typedef std::variant<std::monostate, Encoded, Pcm> Format;
+ typedef std::variant<std::monostate, Encoded, FloatingPointPcm, Pcm> Format;
auto format() const -> const Format& { return format_; }
auto set_format(Format f) -> void { format_ = f; }
@@ -94,10 +111,17 @@ class OutputStream;
class RawStream {
public:
explicit RawStream(std::size_t size);
+ RawStream(std::size_t size, uint32_t);
~RawStream();
auto info() -> StreamInfo& { return info_; }
auto data() -> cpp::span<std::byte>;
+ template <typename T>
+ auto data_as() -> cpp::span<T> {
+ auto orig = data();
+ return {reinterpret_cast<T*>(orig.data()), orig.size_bytes() / sizeof(T)};
+ }
+ auto empty() const -> bool { return info_.bytes_in_stream() == 0; }
private:
StreamInfo info_;
@@ -114,6 +138,12 @@ class InputStream {
const StreamInfo& info() const;
cpp::span<const std::byte> data() const;
+ template <typename T>
+ auto data_as() const -> cpp::span<const T> {
+ auto orig = data();
+ return {reinterpret_cast<const T*>(orig.data()),
+ orig.size_bytes() / sizeof(T)};
+ }
private:
RawStream* raw_;
@@ -131,6 +161,11 @@ class OutputStream {
const StreamInfo& info() const;
cpp::span<std::byte> data() const;
+ template <typename T>
+ auto data_as() const -> cpp::span<T> {
+ auto orig = data();
+ return {reinterpret_cast<T*>(orig.data()), orig.size_bytes() / sizeof(T)};
+ }
private:
RawStream* raw_;
diff --git a/src/audio/resample.cpp b/src/audio/resample.cpp
new file mode 100644
index 00000000..430a6a26
--- /dev/null
+++ b/src/audio/resample.cpp
@@ -0,0 +1,205 @@
+/*
+ * Copyright 2023 jacqueline <me@jacqueline.id.au>
+ *
+ * SPDX-License-Identifier: GPL-3.0-only
+ */
+#include "resample.hpp"
+/*
+ * This file contains the implementation for a 32-bit floating point resampler.
+ * It is largely based on David Bryant's ART resampler, which is BSD-licensed,
+ * and available at https://github.com/dbry/audio-resampler/.
+ *
+ * This resampler uses windowed sinc interpolation filters, with an additional
+ * lowpass filter to reduce aliasing.
+ */
+
+#include <stdint.h>
+#include <stdlib.h>
+#include <string.h>
+#include <algorithm>
+#include <cmath>
+#include <numeric>
+
+#include "esp_log.h"
+
+#include "sample.hpp"
+#include "stream_info.hpp"
+
+namespace audio {
+
+static constexpr double kLowPassRatio = 0.5;
+static constexpr size_t kNumFilters = 64;
+static constexpr size_t kFilterSize = 16;
+
+typedef std::array<float, kFilterSize> Filter;
+static std::array<Filter, kNumFilters + 1> sFilters{};
+static bool sFiltersInitialised = false;
+
+auto InitFilter(int index) -> void;
+
+Resampler::Resampler(uint32_t source_sample_rate,
+ uint32_t target_sample_rate,
+ uint8_t num_channels)
+ : source_sample_rate_(source_sample_rate),
+ target_sample_rate_(target_sample_rate),
+ factor_(static_cast<double>(target_sample_rate) /
+ static_cast<double>(source_sample_rate)),
+ num_channels_(num_channels) {
+ channel_buffers_.resize(num_channels);
+ channel_buffer_size_ = kFilterSize * 16;
+
+ for (int i = 0; i < num_channels; i++) {
+ channel_buffers_[i] =
+ static_cast<float*>(calloc(sizeof(float), channel_buffer_size_));
+ }
+
+ output_offset_ = kFilterSize / 2.0f;
+ input_index_ = kFilterSize;
+
+ if (!sFiltersInitialised) {
+ sFiltersInitialised = true;
+ for (int i = 0; i < kNumFilters + 1; i++) {
+ InitFilter(i);
+ }
+ }
+}
+
+Resampler::~Resampler() {}
+
+auto Resampler::Process(cpp::span<const sample::Sample> input,
+ cpp::span<sample::Sample> output,
+ bool end_of_data) -> std::pair<size_t, size_t> {
+ size_t samples_used = 0;
+ size_t samples_produced = 0;
+
+ size_t input_frames = input.size() / num_channels_;
+ size_t output_frames = output.size() / num_channels_;
+
+ int half_taps = kFilterSize / 2;
+ while (output_frames > 0) {
+ if (output_offset_ >= input_index_ - half_taps) {
+ if (input_frames > 0) {
+ // Check whether the channel buffers will overflow with the addition of
+ // this sample. If so, we need to move the remaining contents back to
+ // the beginning of the buffer.
+ if (input_index_ == channel_buffer_size_) {
+ for (int i = 0; i < num_channels_; ++i) {
+ memmove(channel_buffers_[i],
+ channel_buffers_[i] + channel_buffer_size_ - kFilterSize,
+ kFilterSize * sizeof(float));
+ }
+
+ output_offset_ -= channel_buffer_size_ - kFilterSize;
+ input_index_ -= channel_buffer_size_ - kFilterSize;
+ }
+
+ for (int i = 0; i < num_channels_; ++i) {
+ channel_buffers_[i][input_index_] =
+ sample::ToFloat(input[samples_used++]);
+ }
+
+ input_index_++;
+ input_frames--;
+ } else {
+ break;
+ }
+ } else {
+ for (int i = 0; i < num_channels_; i++) {
+ output[samples_produced++] = sample::FromFloat(Subsample(i));
+ }
+
+ // NOTE: floating point division here is potentially slow due to FPU
+ // limitations. Consider explicitly bunding the xtensa libgcc divsion via
+ // reciprocal implementation if we care about portability between
+ // compilers.
+ output_offset_ += 1.0f / factor_;
+ output_frames--;
+ }
+ }
+
+ return {samples_used, samples_produced};
+}
+
+/*
+ * Constructs the filter in-place for the given index of sFilters. This only
+ * needs to be done once, per-filter. 64-bit math is okay here, because filters
+ * will not be initialised within a performance critical path.
+ */
+auto InitFilter(int index) -> void {
+ Filter& filter = sFilters[index];
+ std::array<double, kFilterSize> working_buffer{};
+
+ double fraction = index / static_cast<double>(kNumFilters);
+ double filter_sum = 0.0;
+
+ for (int i = 0; i < kFilterSize; ++i) {
+ // "dist" is the absolute distance from the sinc maximum to the filter tap
+ // to be calculated, in radians.
+ double dist = fabs((kFilterSize / 2.0 - 1.0) + fraction - i) * M_PI;
+ // "ratio" is that distance divided by half the tap count such that it
+ // reaches π at the window extremes
+ double ratio = dist / (kFilterSize / 2.0);
+
+ double value;
+ if (dist != 0.0) {
+ value = sin(dist * kLowPassRatio) / (dist * kLowPassRatio);
+
+ // Hann window. We could alternatively use a Blackman Harris window,
+ // however our unusually small filter size makes the Hann window's
+ // steeper cutoff more important.
+ value *= 0.5 * (1.0 + cos(ratio));
+ } else {
+ value = 1.0;
+ }
+
+ working_buffer[i] = value;
+ filter_sum += value;
+ }
+
+ // Filter should have unity DC gain
+ double scaler = 1.0 / filter_sum;
+ double error = 0.0;
+
+ for (int i = kFilterSize / 2; i < kFilterSize;
+ i = kFilterSize - i - (i >= kFilterSize / 2)) {
+ working_buffer[i] *= scaler;
+ filter[i] = working_buffer[i] - error;
+ error += static_cast<double>(filter[i]) - working_buffer[i];
+ }
+}
+
+/*
+ * Performs sub-sampling with interpolation for the given channel. Assumes that
+ * the channel buffer has already been filled with samples.
+ */
+auto Resampler::Subsample(int channel) -> float {
+ cpp::span<float> source{channel_buffers_[channel], channel_buffer_size_};
+
+ int offset_integral = std::floor(output_offset_);
+ source = source.subspan(offset_integral);
+ float offset_fractional = output_offset_ - offset_integral;
+
+ offset_fractional *= kNumFilters;
+ int filter_index = std::floor(offset_fractional);
+
+ float sum1 = ApplyFilter(sFilters[filter_index],
+ {source.data() - kFilterSize / 2 + 1, kFilterSize});
+
+ offset_fractional -= filter_index;
+
+ float sum2 = ApplyFilter(sFilters[filter_index + 1],
+ {source.data() - kFilterSize / 2 + 1, kFilterSize});
+
+ return (sum2 * offset_fractional) + (sum1 * (1.0f - offset_fractional));
+}
+
+auto Resampler::ApplyFilter(cpp::span<float> filter, cpp::span<float> input)
+ -> float {
+ float sum = 0.0;
+ for (int i = 0; i < kFilterSize; i++) {
+ sum += filter[i] * input[i];
+ }
+ return sum;
+}
+
+} // namespace audio
diff --git a/src/audio/sink_mixer.cpp b/src/audio/sink_mixer.cpp
new file mode 100644
index 00000000..6c72c8b0
--- /dev/null
+++ b/src/audio/sink_mixer.cpp
@@ -0,0 +1,224 @@
+/*
+ * Copyright 2023 jacqueline <me@jacqueline.id.au>
+ *
+ * SPDX-License-Identifier: GPL-3.0-only
+ */
+
+#include "sink_mixer.hpp"
+
+#include <stdint.h>
+#include <cmath>
+
+#include "esp_heap_caps.h"
+#include "esp_log.h"
+#include "freertos/portmacro.h"
+#include "freertos/projdefs.h"
+#include "idf_additions.h"
+#include "resample.hpp"
+#include "sample.hpp"
+
+#include "stream_info.hpp"
+#include "tasks.hpp"
+
+static constexpr char kTag[] = "mixer";
+
+static constexpr std::size_t kSourceBufferLength = 8 * 1024;
+static constexpr std::size_t kSampleBufferLength = 240 * 2 * sizeof(int32_t);
+
+namespace audio {
+
+SinkMixer::SinkMixer(StreamBufferHandle_t dest)
+ : commands_(xQueueCreate(1, sizeof(Args))),
+ is_idle_(xSemaphoreCreateBinary()),
+ resampler_(nullptr),
+ source_(xStreamBufferCreateWithCaps(kSourceBufferLength,
+ 1,
+ MALLOC_CAP_SPIRAM)),
+ sink_(dest) {
+ input_stream_.reset(new RawStream(kSampleBufferLength));
+ resampled_stream_.reset(new RawStream(kSampleBufferLength));
+
+ // Pin to CORE0 because we need the FPU.
+ // FIXME: A fixed point implementation could run freely on either core,
+ // which should lead to a big performance increase.
+ tasks::StartPersistent<tasks::Type::kMixer>(0, [&]() { Main(); });
+}
+
+SinkMixer::~SinkMixer() {
+ vQueueDelete(commands_);
+ vSemaphoreDelete(is_idle_);
+ vStreamBufferDelete(source_);
+}
+
+auto SinkMixer::MixAndSend(InputStream& input, const StreamInfo::Pcm& target)
+ -> std::size_t {
+ if (input.info().format_as<StreamInfo::Pcm>() !=
+ input_stream_->info().format_as<StreamInfo::Pcm>()) {
+ xSemaphoreTake(is_idle_, portMAX_DELAY);
+ Args args{
+ .cmd = Command::kSetSourceFormat,
+ .format = input.info().format_as<StreamInfo::Pcm>().value(),
+ };
+ xQueueSend(commands_, &args, portMAX_DELAY);
+ xSemaphoreGive(is_idle_);
+ }
+ if (target_format_ != target) {
+ xSemaphoreTake(is_idle_, portMAX_DELAY);
+ Args args{
+ .cmd = Command::kSetTargetFormat,
+ .format = target,
+ };
+ xQueueSend(commands_, &args, portMAX_DELAY);
+ xSemaphoreGive(is_idle_);
+ }
+
+ Args args{
+ .cmd = Command::kReadBytes,
+ .format = {},
+ };
+ xQueueSend(commands_, &args, portMAX_DELAY);
+
+ auto buf = input.data();
+ std::size_t bytes_sent =
+ xStreamBufferSend(source_, buf.data(), buf.size_bytes(), portMAX_DELAY);
+ input.consume(bytes_sent);
+ return bytes_sent;
+}
+
+auto SinkMixer::Main() -> void {
+ OutputStream input_receiver{input_stream_.get()};
+ xSemaphoreGive(is_idle_);
+
+ for (;;) {
+ Args args;
+ while (!xQueueReceive(commands_, &args, portMAX_DELAY)) {
+ }
+ switch (args.cmd) {
+ case Command::kSetSourceFormat:
+ ESP_LOGI(kTag, "setting source format");
+ input_receiver.prepare(args.format, {});
+ resampler_.reset();
+ break;
+ case Command::kSetTargetFormat:
+ ESP_LOGI(kTag, "setting target format");
+ target_format_ = args.format;
+ resampler_.reset();
+ break;
+ case Command::kReadBytes:
+ xSemaphoreTake(is_idle_, 0);
+ while (!xStreamBufferIsEmpty(source_)) {
+ auto buf = input_receiver.data();
+ std::size_t bytes_received = xStreamBufferReceive(
+ source_, buf.data(), buf.size_bytes(), portMAX_DELAY);
+ input_receiver.add(bytes_received);
+ HandleBytes();
+ }
+ xSemaphoreGive(is_idle_);
+ break;
+ }
+ }
+}
+
+auto SinkMixer::HandleBytes() -> void {
+ InputStream input{input_stream_.get()};
+ auto pcm = input.info().format_as<StreamInfo::Pcm>();
+ if (!pcm) {
+ ESP_LOGE(kTag, "mixer got unsupported data");
+ return;
+ }
+
+ if (*pcm == target_format_) {
+ // The happiest possible case: the input format matches the output
+ // format already. Streams like this should probably have bypassed the
+ // mixer.
+ // TODO(jacqueline): Make this an error; it's slow to use the mixer in this
+ // case, compared to just writing directly to the sink.
+ auto buf = input.data();
+ std::size_t bytes_sent =
+ xStreamBufferSend(sink_, buf.data(), buf.size_bytes(), portMAX_DELAY);
+ input.consume(bytes_sent);
+ return;
+ }
+
+ while (input_stream_->info().bytes_in_stream() >= sizeof(sample::Sample)) {
+ RawStream* output_source;
+ if (pcm->sample_rate != target_format_.sample_rate) {
+ OutputStream resampled_writer{resampled_stream_.get()};
+ if (Resample(input, resampled_writer)) {
+ // Zero samples used or written. We need more input.
+ break;
+ }
+ output_source = resampled_stream_.get();
+ } else {
+ output_source = input_stream_.get();
+ }
+
+ size_t bytes_consumed = output_source->info().bytes_in_stream();
+ size_t bytes_to_send = output_source->info().bytes_in_stream();
+
+ if (target_format_.bits_per_sample == 16) {
+ // This is slightly scary; we're basically reaching into the internals of
+ // the stream buffer to do in-place conversion of samples. Saving an
+ // extra buffer + copy into that buffer is certainly worth it however.
+ cpp::span<sample::Sample> src =
+ output_source->data_as<sample::Sample>().first(
+ output_source->info().bytes_in_stream() / sizeof(sample::Sample));
+ cpp::span<int16_t> dest{reinterpret_cast<int16_t*>(src.data()),
+ src.size()};
+
+ ApplyDither(src, 16);
+ Downscale(src, dest);
+
+ bytes_consumed = src.size_bytes();
+ bytes_to_send = src.size_bytes() / 2;
+ }
+
+ InputStream output{output_source};
+ cpp::span<const std::byte> buf = output.data();
+
+ size_t bytes_sent = 0;
+ while (bytes_sent < bytes_to_send) {
+ auto cropped = buf.subspan(bytes_sent, bytes_to_send - bytes_sent);
+ bytes_sent += xStreamBufferSend(sink_, cropped.data(),
+ cropped.size_bytes(), portMAX_DELAY);
+ }
+ output.consume(bytes_consumed);
+ }
+}
+
+auto SinkMixer::Resample(InputStream& in, OutputStream& out) -> bool {
+ if (resampler_ == nullptr) {
+ ESP_LOGI(kTag, "creating new resampler");
+ auto format = in.info().format_as<StreamInfo::Pcm>();
+ resampler_.reset(new Resampler(
+ format->sample_rate, target_format_.sample_rate, format->channels));
+ }
+
+ auto res = resampler_->Process(in.data_as<sample::Sample>(),
+ out.data_as<sample::Sample>(), false);
+
+ in.consume(res.first * sizeof(sample::Sample));
+ out.add(res.second * sizeof(sample::Sample));
+
+ return res.first == 0 && res.second == 0;
+}
+
+auto SinkMixer::Downscale(cpp::span<sample::Sample> samples,
+ cpp::span<int16_t> output) -> void {
+ for (size_t i = 0; i < samples.size(); i++) {
+ output[i] = sample::ToSigned16Bit(samples[i]);
+ }
+}
+
+auto SinkMixer::ApplyDither(cpp::span<sample::Sample> samples,
+ uint_fast8_t bits) -> void {
+ static uint32_t prnd;
+ for (auto& s : samples) {
+ prnd = (prnd * 0x19660dL + 0x3c6ef35fL) & 0xffffffffL;
+ s = sample::Clip(
+ static_cast<int64_t>(s) +
+ (static_cast<int>(prnd) >> (sizeof(sample::Sample) - bits)));
+ }
+}
+
+} // namespace audio
diff --git a/src/audio/stream_info.cpp b/src/audio/stream_info.cpp
index 6efe297e..73dbf91b 100644
--- a/src/audio/stream_info.cpp
+++ b/src/audio/stream_info.cpp
@@ -30,6 +30,13 @@ RawStream::RawStream(std::size_t size)
assert(buffer_ != NULL);
}
+RawStream::RawStream(std::size_t size, uint32_t caps)
+ : info_(),
+ buffer_size_(size),
+ buffer_(reinterpret_cast<std::byte*>(heap_caps_malloc(size, caps))) {
+ assert(buffer_ != NULL);
+}
+
RawStream::~RawStream() {
heap_caps_free(buffer_);
}
diff --git a/src/codecs/CMakeLists.txt b/src/codecs/CMakeLists.txt
index 8502268c..866da8c8 100644
--- a/src/codecs/CMakeLists.txt
+++ b/src/codecs/CMakeLists.txt
@@ -3,9 +3,9 @@
# SPDX-License-Identifier: GPL-3.0-only
idf_component_register(
- SRCS "codec.cpp" "mad.cpp" "foxenflac.cpp" "stbvorbis.cpp"
+ SRCS "codec.cpp" "mad.cpp" "foxenflac.cpp"
INCLUDE_DIRS "include"
- REQUIRES "result" "span" "libmad" "libfoxenflac" "stb_vorbis")
+ REQUIRES "result" "span" "libmad" "libfoxenflac")
target_compile_options("${COMPONENT_LIB}" PRIVATE ${EXTRA_WARNINGS})
diff --git a/src/codecs/codec.cpp b/src/codecs/codec.cpp
index e23b8702..404ea214 100644
--- a/src/codecs/codec.cpp
+++ b/src/codecs/codec.cpp
@@ -11,7 +11,6 @@
#include "foxenflac.hpp"
#include "mad.hpp"
-#include "stbvorbis.hpp"
#include "types.hpp"
namespace codecs {
@@ -22,8 +21,6 @@ auto CreateCodecForType(StreamType type) -> std::optional<ICodec*> {
return new MadMp3Decoder();
case StreamType::kFlac:
return new FoxenFlacDecoder();
- case StreamType::kVorbis:
- return new StbVorbisDecoder();
default:
return {};
}
diff --git a/src/codecs/foxenflac.cpp b/src/codecs/foxenflac.cpp
index 3a727ce2..b676f82a 100644
--- a/src/codecs/foxenflac.cpp
+++ b/src/codecs/foxenflac.cpp
@@ -12,6 +12,7 @@
#include "esp_log.h"
#include "foxen/flac.h"
+#include "sample.hpp"
namespace codecs {
@@ -47,7 +48,6 @@ auto FoxenFlacDecoder::BeginStream(const cpp::span<const std::byte> input)
OutputFormat format{
.num_channels = static_cast<uint8_t>(channels),
- .bits_per_sample = 32, // libfoxenflac output is fixed-size.
.sample_rate_hz = static_cast<uint32_t>(fs),
.duration_seconds = {},
.bits_per_second = {},
@@ -62,7 +62,7 @@ auto FoxenFlacDecoder::BeginStream(const cpp::span<const std::byte> input)
}
auto FoxenFlacDecoder::ContinueStream(cpp::span<const std::byte> input,
- cpp::span<std::byte> output)
+ cpp::span<sample::Sample> output)
-> Result<OutputInfo> {
cpp::span<int32_t> output_as_samples{
reinterpret_cast<int32_t*>(output.data()), output.size_bytes() / 4};
@@ -78,7 +78,7 @@ auto FoxenFlacDecoder::ContinueStream(cpp::span<const std::byte> input,
if (samples_written > 0) {
return {bytes_read,
- OutputInfo{.bytes_written = samples_written * 4,
+ OutputInfo{.samples_written = samples_written,
.is_finished_writing = state == FLAC_END_OF_FRAME}};
}
diff --git a/src/codecs/include/codec.hpp b/src/codecs/include/codec.hpp
index e8be8f0a..32ebef69 100644
--- a/src/codecs/include/codec.hpp
+++ b/src/codecs/include/codec.hpp
@@ -17,6 +17,7 @@
#include <utility>
#include "result.hpp"
+#include "sample.hpp"
#include "span.hpp"
#include "types.hpp"
@@ -61,7 +62,6 @@ class ICodec {
struct OutputFormat {
uint8_t num_channels;
- uint8_t bits_per_sample;
uint32_t sample_rate_hz;
std::optional<uint32_t> duration_seconds;
@@ -76,7 +76,7 @@ class ICodec {
-> Result<OutputFormat> = 0;
struct OutputInfo {
- std::size_t bytes_written;
+ std::size_t samples_written;
bool is_finished_writing;
};
@@ -84,7 +84,7 @@ class ICodec {
* Writes PCM samples to the given output buffer.
*/
virtual auto ContinueStream(cpp::span<const std::byte> input,
- cpp::span<std::byte> output)
+ cpp::span<sample::Sample> output)
-> Result<OutputInfo> = 0;
virtual auto SeekStream(cpp::span<const std::byte> input,
diff --git a/src/codecs/include/foxenflac.hpp b/src/codecs/include/foxenflac.hpp
index cce1b762..abfa6d80 100644
--- a/src/codecs/include/foxenflac.hpp
+++ b/src/codecs/include/foxenflac.hpp
@@ -14,6 +14,7 @@
#include <utility>
#include "foxen/flac.h"
+#include "sample.hpp"
#include "span.hpp"
#include "codec.hpp"
@@ -26,7 +27,7 @@ class FoxenFlacDecoder : public ICodec {
~FoxenFlacDecoder();
auto BeginStream(cpp::span<const std::byte>) -> Result<OutputFormat> override;
- auto ContinueStream(cpp::span<const std::byte>, cpp::span<std::byte>)
+ auto ContinueStream(cpp::span<const std::byte>, cpp::span<sample::Sample>)
-> Result<OutputInfo> override;
auto SeekStream(cpp::span<const std::byte> input, std::size_t target_sample)
-> Result<void> override;
diff --git a/src/codecs/include/mad.hpp b/src/codecs/include/mad.hpp
index fbae560c..b81e4acb 100644
--- a/src/codecs/include/mad.hpp
+++ b/src/codecs/include/mad.hpp
@@ -13,6 +13,7 @@
#include <utility>
#include "mad.h"
+#include "sample.hpp"
#include "span.hpp"
#include "codec.hpp"
@@ -35,7 +36,7 @@ class MadMp3Decoder : public ICodec {
* Writes samples for the current frame.
*/
auto ContinueStream(cpp::span<const std::byte> input,
- cpp::span<std::byte> output)
+ cpp::span<sample::Sample> output)
-> Result<OutputInfo> override;
auto SeekStream(cpp::span<const std::byte> input, std::size_t target_sample)
diff --git a/src/codecs/include/sample.hpp b/src/codecs/include/sample.hpp
new file mode 100644
index 00000000..f8e08cdc
--- /dev/null
+++ b/src/codecs/include/sample.hpp
@@ -0,0 +1,65 @@
+#pragma once
+
+#include <stdint.h>
+
+#include <algorithm>
+
+#include <mad.h>
+
+namespace sample {
+
+// A signed, 32-bit PCM sample.
+typedef int32_t Sample;
+
+constexpr auto Clip(int64_t v) -> Sample {
+ if (v > INT32_MAX)
+ return INT32_MAX;
+ if (v < INT32_MIN)
+ return INT32_MIN;
+ return v;
+}
+
+constexpr auto FromSigned(int32_t src, uint_fast8_t bits) -> Sample {
+ // Left-align samples, effectively scaling them up to 32 bits.
+ return src << (sizeof(Sample) * 8 - bits);
+}
+
+constexpr auto FromUnsigned(uint32_t src, uint_fast8_t bits) -> Sample {
+ // Left-align, then substract the max value / 2 to make the sample centred
+ // around zero.
+ return (src << (sizeof(uint32_t) * 8 - bits)) - (~0UL >> 1);
+}
+
+constexpr auto FromFloat(float src) -> Sample {
+ return std::clamp<float>(src, -1.0f, 1.0f) * static_cast<float>(INT32_MAX);
+}
+
+constexpr auto FromDouble(double src) -> Sample {
+ return std::clamp<double>(src, -1.0, 1.0) * static_cast<double>(INT32_MAX);
+}
+
+constexpr auto FromMad(mad_fixed_t src) -> Sample {
+ // Round the bottom bits.
+ src += (1L << (MAD_F_FRACBITS - 24));
+
+ // Clip the leftover bits to within range.
+ if (src >= MAD_F_ONE)
+ src = MAD_F_ONE - 1;
+ else if (src < -MAD_F_ONE)
+ src = -MAD_F_ONE;
+
+ // Quantize.
+ return FromSigned(src >> (MAD_F_FRACBITS + 1 - 24), 24);
+}
+
+constexpr auto ToSigned16Bit(Sample src) -> int16_t {
+ return src >> 16;
+}
+
+static constexpr float kFactor = 1.0f / static_cast<float>(INT32_MAX);
+
+constexpr auto ToFloat(Sample src) -> float {
+ return src * kFactor;
+}
+
+} // namespace sample
diff --git a/src/codecs/mad.cpp b/src/codecs/mad.cpp
index 29e34a0f..a2739bcd 100644
--- a/src/codecs/mad.cpp
+++ b/src/codecs/mad.cpp
@@ -17,24 +17,11 @@
#include "codec.hpp"
#include "esp_log.h"
#include "result.hpp"
+#include "sample.hpp"
#include "types.hpp"
namespace codecs {
-static uint32_t mad_fixed_to_pcm(mad_fixed_t sample, uint8_t bits) {
- // Round the bottom bits.
- sample += (1L << (MAD_F_FRACBITS - bits));
-
- // Clip the leftover bits to within range.
- if (sample >= MAD_F_ONE)
- sample = MAD_F_ONE - 1;
- else if (sample < -MAD_F_ONE)
- sample = -MAD_F_ONE;
-
- // Quantize.
- return sample >> (MAD_F_FRACBITS + 1 - bits);
-}
-
MadMp3Decoder::MadMp3Decoder() {
mad_stream_init(&stream_);
mad_frame_init(&frame_);
@@ -83,7 +70,6 @@ auto MadMp3Decoder::BeginStream(const cpp::span<const std::byte> input)
uint8_t channels = MAD_NCHANNELS(&header);
OutputFormat output{
.num_channels = channels,
- .bits_per_sample = 24, // We always scale to 24 bits
.sample_rate_hz = header.samplerate,
.duration_seconds = {},
.bits_per_second = {},
@@ -100,7 +86,7 @@ auto MadMp3Decoder::BeginStream(const cpp::span<const std::byte> input)
}
auto MadMp3Decoder::ContinueStream(cpp::span<const std::byte> input,
- cpp::span<std::byte> output)
+ cpp::span<sample::Sample> output)
-> Result<OutputInfo> {
std::size_t bytes_read = 0;
if (current_sample_ < 0) {
@@ -133,32 +119,24 @@ auto MadMp3Decoder::ContinueStream(cpp::span<const std::byte> input,
bytes_read = GetBytesUsed(input.size_bytes());
}
- size_t output_byte = 0;
+ size_t output_sample = 0;
while (current_sample_ < synth_.pcm.length) {
- if (output_byte + (4 * synth_.pcm.channels) >= output.size()) {
- // We can't fit the next sample into the buffer. Stop now, and also avoid
- // writing the sample for only half the channels.
- return {bytes_read, OutputInfo{.bytes_written = output_byte,
+ if (output_sample + synth_.pcm.channels >= output.size()) {
+ // We can't fit the next full frame into the buffer.
+ return {bytes_read, OutputInfo{.samples_written = output_sample,
.is_finished_writing = false}};
}
for (int channel = 0; channel < synth_.pcm.channels; channel++) {
- uint32_t sample_24 =
- mad_fixed_to_pcm(synth_.pcm.samples[channel][current_sample_], 24);
-
- // 24 bit samples must still be aligned to 32 bits. The LSB is ignored.
- output[output_byte++] = static_cast<std::byte>(0);
-
- output[output_byte++] = static_cast<std::byte>((sample_24)&0xFF);
- output[output_byte++] = static_cast<std::byte>((sample_24 >> 8) & 0xFF);
- output[output_byte++] = static_cast<std::byte>((sample_24 >> 16) & 0xFF);
+ output[output_sample++] =
+ sample::FromMad(synth_.pcm.samples[channel][current_sample_]);
}
current_sample_++;
}
// We wrote everything! Reset, ready for the next frame.
current_sample_ = -1;
- return {bytes_read, OutputInfo{.bytes_written = output_byte,
+ return {bytes_read, OutputInfo{.samples_written = output_sample,
.is_finished_writing = true}};
}
diff --git a/src/drivers/CMakeLists.txt b/src/drivers/CMakeLists.txt
index 40cd0c4f..a64495f0 100644
--- a/src/drivers/CMakeLists.txt
+++ b/src/drivers/CMakeLists.txt
@@ -5,6 +5,7 @@
idf_component_register(
SRCS "touchwheel.cpp" "i2s_dac.cpp" "gpios.cpp" "battery.cpp" "storage.cpp" "i2c.cpp"
"spi.cpp" "display.cpp" "display_init.cpp" "samd.cpp" "relative_wheel.cpp" "wm8523.cpp"
+ "nvs.cpp" "bluetooth.cpp"
INCLUDE_DIRS "include"
- REQUIRES "esp_adc" "fatfs" "result" "lvgl" "span" "tasks")
+ REQUIRES "esp_adc" "fatfs" "result" "lvgl" "span" "tasks" "nvs_flash" "bt" "tinyfsm")
target_compile_options(${COMPONENT_LIB} PRIVATE ${EXTRA_WARNINGS})
diff --git a/src/drivers/bluetooth.cpp b/src/drivers/bluetooth.cpp
new file mode 100644
index 00000000..f9ab4e95
--- /dev/null
+++ b/src/drivers/bluetooth.cpp
@@ -0,0 +1,253 @@
+#include "bluetooth.hpp"
+
+#include <stdint.h>
+
+#include <atomic>
+#include <ostream>
+#include <sstream>
+
+#include "esp_a2dp_api.h"
+#include "esp_avrc_api.h"
+#include "esp_bt.h"
+#include "esp_bt_device.h"
+#include "esp_bt_main.h"
+#include "esp_gap_bt_api.h"
+#include "esp_log.h"
+#include "esp_mac.h"
+#include "tinyfsm/include/tinyfsm.hpp"
+
+namespace drivers {
+
+static constexpr char kTag[] = "bluetooth";
+
+static std::atomic<StreamBufferHandle_t> sStream;
+
+auto gap_cb(esp_bt_gap_cb_event_t event, esp_bt_gap_cb_param_t* param) -> void {
+ tinyfsm::FsmList<bluetooth::BluetoothState>::dispatch(
+ bluetooth::events::internal::Gap{.type = event, .param = param});
+}
+
+auto avrcp_cb(esp_avrc_ct_cb_event_t event, esp_avrc_ct_cb_param_t* param)
+ -> void {
+ tinyfsm::FsmList<bluetooth::BluetoothState>::dispatch(
+ bluetooth::events::internal::Avrc{.type = event, .param = param});
+}
+
+auto a2dp_cb(esp_a2d_cb_event_t event, esp_a2d_cb_param_t* param) -> void {
+ tinyfsm::FsmList<bluetooth::BluetoothState>::dispatch(
+ bluetooth::events::internal::A2dp{.type = event, .param = param});
+}
+
+auto a2dp_data_cb(uint8_t* buf, int32_t buf_size) -> int32_t {
+ if (buf == nullptr || buf_size <= 0) {
+ return 0;
+ }
+ StreamBufferHandle_t stream = sStream.load();
+ if (stream == nullptr) {
+ return 0;
+ }
+ return xStreamBufferReceive(stream, buf, buf_size, 0);
+}
+
+Bluetooth::Bluetooth() {
+ tinyfsm::FsmList<bluetooth::BluetoothState>::start();
+}
+
+auto Bluetooth::Enable() -> bool {
+ tinyfsm::FsmList<bluetooth::BluetoothState>::dispatch(
+ bluetooth::events::Enable{});
+
+ return !bluetooth::BluetoothState::is_in_state<bluetooth::Disabled>();
+}
+
+auto Bluetooth::Disable() -> void {
+ tinyfsm::FsmList<bluetooth::BluetoothState>::dispatch(
+ bluetooth::events::Disable{});
+}
+
+auto DeviceName() -> std::string {
+ uint8_t mac[8]{0};
+ esp_efuse_mac_get_default(mac);
+ std::ostringstream name;
+ name << "TANGARA " << std::hex << mac[0] << mac[1];
+ return name.str();
+}
+
+namespace bluetooth {
+
+static bool sIsFirstEntry = true;
+
+void Disabled::entry() {
+ if (sIsFirstEntry) {
+ // We only use BT Classic, to claw back ~60KiB from the BLE firmware.
+ esp_bt_controller_mem_release(ESP_BT_MODE_BLE);
+ sIsFirstEntry = false;
+ return;
+ }
+
+ esp_bluedroid_disable();
+ esp_bluedroid_deinit();
+ esp_bt_controller_disable();
+}
+
+void Disabled::react(const events::Enable&) {
+ esp_bt_controller_config_t config = BT_CONTROLLER_INIT_CONFIG_DEFAULT();
+ if (esp_bt_controller_init(&config) != ESP_OK) {
+ ESP_LOGE(kTag, "initialize controller failed");
+ return;
+ }
+
+ if (esp_bt_controller_enable(ESP_BT_MODE_CLASSIC_BT) != ESP_OK) {
+ ESP_LOGE(kTag, "enable controller failed");
+ return;
+ }
+
+ if (esp_bluedroid_init() != ESP_OK) {
+ ESP_LOGE(kTag, "initialize bluedroid failed");
+ return;
+ }
+
+ if (esp_bluedroid_enable() != ESP_OK) {
+ ESP_LOGE(kTag, "enable bluedroid failed");
+ return;
+ }
+
+ // Enable Secure Simple Pairing
+ esp_bt_sp_param_t param_type = ESP_BT_SP_IOCAP_MODE;
+ esp_bt_io_cap_t iocap = ESP_BT_IO_CAP_IO;
+ esp_bt_gap_set_security_param(param_type, &iocap, sizeof(uint8_t));
+
+ // Set a reasonable name for the device.
+ std::string name = DeviceName();
+ esp_bt_dev_set_device_name(name.c_str());
+
+ // Initialise GAP. This controls advertising our device, and scanning for
+ // other devices.
+ esp_bt_gap_register_callback(gap_cb);
+
+ // Initialise AVRCP. This handles playback controls; play/pause/volume/etc.
+ // esp_avrc_ct_init();
+ // esp_avrc_ct_register_callback(avrcp_cb);
+
+ // Initialise A2DP. This handles streaming audio. Currently ESP-IDF's SBC
+ // encoder only supports 2 channels of interleaved 16 bit samples, at
+ // 44.1kHz, so there is no additional configuration to be done for the
+ // stream itself.
+ esp_a2d_source_init();
+ esp_a2d_register_callback(a2dp_cb);
+ esp_a2d_source_register_data_callback(a2dp_data_cb);
+
+ // Don't let anyone interact with us before we're ready.
+ esp_bt_gap_set_scan_mode(ESP_BT_NON_CONNECTABLE, ESP_BT_NON_DISCOVERABLE);
+
+ transit<Scanning>();
+}
+
+static constexpr uint8_t kDiscoveryTimeSeconds = 10;
+static constexpr uint8_t kDiscoveryMaxResults = 0;
+
+void Scanning::entry() {
+ ESP_LOGI(kTag, "scanning for devices");
+ esp_bt_gap_start_discovery(ESP_BT_INQ_MODE_GENERAL_INQUIRY,
+ kDiscoveryTimeSeconds, kDiscoveryMaxResults);
+}
+
+void Scanning::exit() {
+ esp_bt_gap_cancel_discovery();
+}
+
+auto OnDeviceDiscovered(esp_bt_gap_cb_param_t* param) -> void {
+ ESP_LOGI(kTag, "device discovered");
+}
+
+void Scanning::react(const events::internal::Gap& ev) {
+ switch (ev.type) {
+ case ESP_BT_GAP_DISC_RES_EVT:
+ OnDeviceDiscovered(ev.param);
+ break;
+ case ESP_BT_GAP_DISC_STATE_CHANGED_EVT:
+ if (ev.param->disc_st_chg.state == ESP_BT_GAP_DISCOVERY_STOPPED) {
+ ESP_LOGI(kTag, "still scanning");
+ esp_bt_gap_start_discovery(ESP_BT_INQ_MODE_GENERAL_INQUIRY,
+ kDiscoveryTimeSeconds, kDiscoveryMaxResults);
+ }
+ break;
+ case ESP_BT_GAP_MODE_CHG_EVT:
+ // todo: mode change. is this important?
+ ESP_LOGI(kTag, "GAP mode changed");
+ break;
+ default:
+ ESP_LOGW(kTag, "unhandled GAP event: %u", ev.type);
+ }
+}
+
+void Connecting::entry() {
+ ESP_LOGI(kTag, "connecting to device");
+ esp_a2d_source_connect(nullptr);
+}
+
+void Connecting::exit() {}
+
+void Connecting::react(const events::internal::Gap& ev) {
+ switch (ev.type) {
+ case ESP_BT_GAP_AUTH_CMPL_EVT:
+ // todo: auth completed. check if we succeeded.
+ break;
+ case ESP_BT_GAP_PIN_REQ_EVT:
+ // todo: device needs a pin to connect.
+ break;
+ case ESP_BT_GAP_CFM_REQ_EVT:
+ // todo: device needs user to click okay.
+ break;
+ case ESP_BT_GAP_KEY_NOTIF_EVT:
+ // todo: device is telling us a password?
+ break;
+ case ESP_BT_GAP_KEY_REQ_EVT:
+ // todo: device needs a password
+ break;
+ case ESP_BT_GAP_MODE_CHG_EVT:
+ // todo: mode change. is this important?
+ break;
+ default:
+ ESP_LOGW(kTag, "unhandled GAP event: %u", ev.type);
+ }
+}
+
+void Connecting::react(const events::internal::A2dp& ev) {
+ switch (ev.type) {
+ case ESP_A2D_CONNECTION_STATE_EVT:
+ // todo: connection state changed. we might be connected!
+ break;
+ default:
+ ESP_LOGW(kTag, "unhandled A2DP event: %u", ev.type);
+ }
+}
+
+void Connected::react(const events::internal::A2dp& ev) {
+ switch (ev.type) {
+ case ESP_A2D_CONNECTION_STATE_EVT:
+ // todo: connection state changed. we might have dropped
+ break;
+ case ESP_A2D_AUDIO_STATE_EVT:
+ // todo: audio state changed. who knows, dude.
+ break;
+ default:
+ ESP_LOGW(kTag, "unhandled A2DP event: %u", ev.type);
+ }
+}
+
+void Connected::react(const events::internal::Avrc& ev) {
+ switch (ev.type) {
+ case ESP_AVRC_CT_CONNECTION_STATE_EVT:
+ // todo: avrc connected. send our capabilities.
+ default:
+ ESP_LOGW(kTag, "unhandled AVRC event: %u", ev.type);
+ }
+}
+
+} // namespace bluetooth
+
+} // namespace drivers
+
+FSM_INITIAL_STATE(drivers::bluetooth::BluetoothState,
+ drivers::bluetooth::Disabled)
diff --git a/src/drivers/i2s_dac.cpp b/src/drivers/i2s_dac.cpp
index c835fb1f..885321d1 100644
--- a/src/drivers/i2s_dac.cpp
+++ b/src/drivers/i2s_dac.cpp
@@ -161,7 +161,6 @@ auto I2SDac::Reconfigure(Channels ch, BitsPerSample bps, SampleRate rate)
word_length = 0b10;
break;
case BPS_32:
- // TODO(jacqueline): Error on this? It's not supported anymore.
slot_config_.data_bit_width = I2S_DATA_BIT_WIDTH_32BIT;
slot_config_.ws_width = 32;
word_length = 0b11;
diff --git a/src/drivers/include/bluetooth.hpp b/src/drivers/include/bluetooth.hpp
new file mode 100644
index 00000000..2b5e6a8d
--- /dev/null
+++ b/src/drivers/include/bluetooth.hpp
@@ -0,0 +1,108 @@
+
+#pragma once
+
+#include <string>
+#include <vector>
+
+#include <freertos/FreeRTOS.h>
+#include <freertos/stream_buffer.h>
+#include "esp_a2dp_api.h"
+#include "esp_avrc_api.h"
+#include "esp_gap_bt_api.h"
+#include "tinyfsm.hpp"
+#include "tinyfsm/include/tinyfsm.hpp"
+
+namespace drivers {
+
+/*
+ * A handle used to interact with the bluetooth state machine.
+ */
+class Bluetooth {
+ public:
+ Bluetooth();
+
+ auto Enable() -> bool;
+ auto Disable() -> void;
+
+ auto SetSource(StreamBufferHandle_t) -> void;
+};
+
+namespace bluetooth {
+
+namespace events {
+struct Enable : public tinyfsm::Event {};
+struct Disable : public tinyfsm::Event {};
+
+namespace internal {
+struct Gap : public tinyfsm::Event {
+ esp_bt_gap_cb_event_t type;
+ esp_bt_gap_cb_param_t* param;
+};
+struct A2dp : public tinyfsm::Event {
+ esp_a2d_cb_event_t type;
+ esp_a2d_cb_param_t* param;
+};
+struct Avrc : public tinyfsm::Event {
+ esp_avrc_ct_cb_event_t type;
+ esp_avrc_ct_cb_param_t* param;
+};
+} // namespace internal
+} // namespace events
+
+class BluetoothState : public tinyfsm::Fsm<BluetoothState> {
+ public:
+ virtual ~BluetoothState(){};
+
+ virtual void entry() {}
+ virtual void exit() {}
+
+ virtual void react(const events::Enable& ev){};
+ virtual void react(const events::Disable& ev) = 0;
+
+ virtual void react(const events::internal::Gap& ev) = 0;
+ virtual void react(const events::internal::A2dp& ev) = 0;
+ virtual void react(const events::internal::Avrc& ev){};
+};
+
+class Disabled : public BluetoothState {
+ void entry() override;
+
+ void react(const events::Enable& ev) override;
+ void react(const events::Disable& ev) override{};
+
+ void react(const events::internal::Gap& ev) override {}
+ void react(const events::internal::A2dp& ev) override {}
+};
+
+class Scanning : public BluetoothState {
+ void entry() override;
+ void exit() override;
+
+ void react(const events::Disable& ev) override;
+
+ void react(const events::internal::Gap& ev) override;
+ void react(const events::internal::A2dp& ev) override;
+};
+
+class Connecting : public BluetoothState {
+ void entry() override;
+ void exit() override;
+
+ void react(const events::Disable& ev) override;
+ void react(const events::internal::Gap& ev) override;
+ void react(const events::internal::A2dp& ev) override;
+};
+
+class Connected : public BluetoothState {
+ void entry() override;
+ void exit() override;
+
+ void react(const events::Disable& ev) override;
+ void react(const events::internal::Gap& ev) override;
+ void react(const events::internal::A2dp& ev) override;
+ void react(const events::internal::Avrc& ev) override;
+};
+
+} // namespace bluetooth
+
+} // namespace drivers
diff --git a/src/drivers/include/i2s_dac.hpp b/src/drivers/include/i2s_dac.hpp
index 06c0dc16..889ba68c 100644
--- a/src/drivers/include/i2s_dac.hpp
+++ b/src/drivers/include/i2s_dac.hpp
@@ -51,14 +51,12 @@ class I2SDac {
BPS_32 = I2S_DATA_BIT_WIDTH_32BIT,
};
enum SampleRate {
- SAMPLE_RATE_11_025 = 11025,
- SAMPLE_RATE_16 = 16000,
- SAMPLE_RATE_22_05 = 22050,
+ SAMPLE_RATE_8 = 8000,
SAMPLE_RATE_32 = 32000,
SAMPLE_RATE_44_1 = 44100,
SAMPLE_RATE_48 = 48000,
+ SAMPLE_RATE_88_2 = 88200,
SAMPLE_RATE_96 = 96000,
- SAMPLE_RATE_192 = 192000,
};
auto Reconfigure(Channels ch, BitsPerSample bps, SampleRate rate) -> void;
diff --git a/src/drivers/include/nvs.hpp b/src/drivers/include/nvs.hpp
new file mode 100644
index 00000000..be783583
--- /dev/null
+++ b/src/drivers/include/nvs.hpp
@@ -0,0 +1,27 @@
+/*
+ * Copyright 2023 jacqueline <me@jacqueline.id.au>
+ *
+ * SPDX-License-Identifier: GPL-3.0-only
+ */
+
+#pragma once
+
+#include "esp_err.h"
+#include "nvs.h"
+
+namespace drivers {
+
+class NvsStorage {
+ public:
+ static auto Open() -> NvsStorage*;
+
+ auto SchemaVersion() -> uint8_t;
+
+ explicit NvsStorage(nvs_handle_t);
+ ~NvsStorage();
+
+ private:
+ nvs_handle_t handle_;
+};
+
+} // namespace drivers \ No newline at end of file
diff --git a/src/drivers/nvs.cpp b/src/drivers/nvs.cpp
new file mode 100644
index 00000000..a2de9518
--- /dev/null
+++ b/src/drivers/nvs.cpp
@@ -0,0 +1,73 @@
+/*
+ * Copyright 2023 jacqueline <me@jacqueline.id.au>
+ *
+ * SPDX-License-Identifier: GPL-3.0-only
+ */
+
+#include "nvs.hpp"
+#include <stdint.h>
+
+#include <cstdint>
+#include <memory>
+
+#include "esp_log.h"
+#include "nvs.h"
+#include "nvs_flash.h"
+
+namespace drivers {
+
+static constexpr char kTag[] = "nvm";
+static constexpr uint8_t kSchemaVersion = 1;
+
+static constexpr char kKeyVersion[] = "ver";
+
+auto NvsStorage::Open() -> NvsStorage* {
+ esp_err_t err = nvs_flash_init();
+ if (err == ESP_ERR_NVS_NO_FREE_PAGES) {
+ ESP_LOGW(kTag, "partition needs initialisation");
+ nvs_flash_erase();
+ err = nvs_flash_init();
+ }
+ if (err != ESP_OK) {
+ ESP_LOGE(kTag, "failed to init nvm");
+ return nullptr;
+ }
+
+ nvs_handle_t handle;
+ if ((err = nvs_open("tangara", NVS_READWRITE, &handle)) != ESP_OK) {
+ ESP_LOGE(kTag, "failed to open nvs namespace");
+ return nullptr;
+ }
+
+ std::unique_ptr<NvsStorage> instance = std::make_unique<NvsStorage>(handle);
+ if (instance->SchemaVersion() < kSchemaVersion) {
+ ESP_LOGW(kTag, "namespace needs downgrading");
+ nvs_erase_all(handle);
+ nvs_set_u8(handle, kKeyVersion, kSchemaVersion);
+ err = nvs_commit(handle);
+ if (err != ESP_OK) {
+ ESP_LOGW(kTag, "failed to init namespace");
+ return nullptr;
+ }
+ }
+
+ ESP_LOGI(kTag, "nvm storage initialised okay");
+ return instance.release();
+}
+
+NvsStorage::NvsStorage(nvs_handle_t handle) : handle_(handle) {}
+
+NvsStorage::~NvsStorage() {
+ nvs_close(handle_);
+ nvs_flash_deinit();
+}
+
+auto NvsStorage::SchemaVersion() -> uint8_t {
+ uint8_t ret;
+ if (nvs_get_u8(handle_, kKeyVersion, &ret) != ESP_OK) {
+ return UINT8_MAX;
+ }
+ return ret;
+}
+
+} // namespace drivers
diff --git a/src/drivers/storage.cpp b/src/drivers/storage.cpp
index db257dee..f253a79a 100644
--- a/src/drivers/storage.cpp
+++ b/src/drivers/storage.cpp
@@ -63,7 +63,7 @@ auto SdStorage::Create(IGpios* gpio) -> cpp::result<SdStorage*, Error> {
// Will return ESP_ERR_INVALID_RESPONSE if there is no card
esp_err_t err = sdmmc_card_init(host.get(), card.get());
if (err != ESP_OK) {
- ESP_LOGW(kTag, "Failed to read, err: %d", err);
+ ESP_LOGW(kTag, "Failed to read, err: %s", esp_err_to_name(err));
return cpp::fail(Error::FAILED_TO_READ);
}
@@ -74,7 +74,21 @@ auto SdStorage::Create(IGpios* gpio) -> cpp::result<SdStorage*, Error> {
// Mount right now, not on first operation.
FRESULT ferr = f_mount(fs, "", 1);
if (ferr != FR_OK) {
- ESP_LOGW(kTag, "Failed to mount, err: %d", ferr);
+ std::string err_str;
+ switch (ferr) {
+ case FR_DISK_ERR:
+ err_str = "FR_DISK_ERR";
+ break;
+ case FR_NOT_READY:
+ err_str = "FR_NOT_READY";
+ break;
+ case FR_NO_FILESYSTEM:
+ err_str = "FR_NO_FILESYSTEM";
+ break;
+ default:
+ err_str = std::to_string(ferr);
+ }
+ ESP_LOGW(kTag, "Failed to mount, err: %s", err_str.c_str());
return cpp::fail(Error::FAILED_TO_MOUNT);
}
diff --git a/src/system_fsm/booting.cpp b/src/system_fsm/booting.cpp
index 4686748e..3d6c6a46 100644
--- a/src/system_fsm/booting.cpp
+++ b/src/system_fsm/booting.cpp
@@ -13,6 +13,7 @@
#include "event_queue.hpp"
#include "gpios.hpp"
#include "lvgl/lvgl.h"
+#include "nvs.hpp"
#include "relative_wheel.hpp"
#include "spi.hpp"
#include "system_events.hpp"
@@ -50,9 +51,10 @@ auto Booting::entry() -> void {
ESP_LOGI(kTag, "installing remaining drivers");
sSamd.reset(drivers::Samd::Create());
sBattery.reset(drivers::Battery::Create());
+ sNvs.reset(drivers::NvsStorage::Open());
sTagParser.reset(new database::TagParserImpl());
- if (!sSamd || !sBattery) {
+ if (!sSamd || !sBattery || !sNvs) {
events::System().Dispatch(FatalError{});
events::Ui().Dispatch(FatalError{});
return;
diff --git a/src/system_fsm/include/system_fsm.hpp b/src/system_fsm/include/system_fsm.hpp
index 6f0eb563..d30a712c 100644
--- a/src/system_fsm/include/system_fsm.hpp
+++ b/src/system_fsm/include/system_fsm.hpp
@@ -13,6 +13,7 @@
#include "database.hpp"
#include "display.hpp"
#include "gpios.hpp"
+#include "nvs.hpp"
#include "relative_wheel.hpp"
#include "samd.hpp"
#include "storage.hpp"
@@ -54,6 +55,7 @@ class SystemState : public tinyfsm::Fsm<SystemState> {
protected:
static std::shared_ptr<drivers::Gpios> sGpios;
static std::shared_ptr<drivers::Samd> sSamd;
+ static std::shared_ptr<drivers::NvsStorage> sNvs;
static std::shared_ptr<drivers::TouchWheel> sTouch;
static std::shared_ptr<drivers::RelativeWheel> sRelativeTouch;
diff --git a/src/system_fsm/system_fsm.cpp b/src/system_fsm/system_fsm.cpp
index 5f85d43c..527a8770 100644
--- a/src/system_fsm/system_fsm.cpp
+++ b/src/system_fsm/system_fsm.cpp
@@ -17,6 +17,7 @@ namespace system_fsm {
std::shared_ptr<drivers::Gpios> SystemState::sGpios;
std::shared_ptr<drivers::Samd> SystemState::sSamd;
+std::shared_ptr<drivers::NvsStorage> SystemState::sNvs;
std::shared_ptr<drivers::TouchWheel> SystemState::sTouch;
std::shared_ptr<drivers::RelativeWheel> SystemState::sRelativeTouch;
diff --git a/src/tasks/tasks.cpp b/src/tasks/tasks.cpp
index 861c7bf0..34c690f3 100644
--- a/src/tasks/tasks.cpp
+++ b/src/tasks/tasks.cpp
@@ -34,6 +34,10 @@ auto Name<Type::kAudio>() -> std::string {
return "AUDIO";
}
template <>
+auto Name<Type::kMixer>() -> std::string {
+ return "MIXER";
+}
+template <>
auto Name<Type::kDatabase>() -> std::string {
return "DB";
}
@@ -77,6 +81,14 @@ auto AllocateStack<Type::kFileStreamer>() -> cpp::span<StackType_t> {
size};
}
+template <>
+auto AllocateStack<Type::kMixer>() -> cpp::span<StackType_t> {
+ std::size_t size = 4 * 1024;
+ return {static_cast<StackType_t*>(
+ heap_caps_malloc(size, MALLOC_CAP_INTERNAL | MALLOC_CAP_8BIT)),
+ size};
+}
+
// Leveldb is designed for non-embedded use cases, where stack space isn't so
// much of a concern. It therefore uses an eye-wateringly large amount of stack.
template <>
@@ -105,6 +117,10 @@ auto Priority() -> UBaseType_t;
// Realtime audio is the entire point of this device, so give this task the
// highest priority.
template <>
+auto Priority<Type::kMixer>() -> UBaseType_t {
+ return 12;
+}
+template <>
auto Priority<Type::kAudio>() -> UBaseType_t {
return 11;
}
diff --git a/src/tasks/tasks.hpp b/src/tasks/tasks.hpp
index 742bb3cc..a0c201d5 100644
--- a/src/tasks/tasks.hpp
+++ b/src/tasks/tasks.hpp
@@ -36,6 +36,8 @@ enum class Type {
kFileStreamer,
// The main audio pipeline task.
kAudio,
+ // TODO
+ kMixer,
// Task for running database queries.
kDatabase,
// Task for internal database operations
@@ -57,9 +59,19 @@ template <Type t>
auto StartPersistent(const std::function<void(void)>& fn) -> void {
StaticTask_t* task_buffer = new StaticTask_t;
cpp::span<StackType_t> stack = AllocateStack<t>();
+ xTaskCreateStatic(&PersistentMain, Name<t>().c_str(), stack.size(),
+ new std::function<void(void)>(fn), Priority<t>(),
+ stack.data(), task_buffer);
+}
+
+template <Type t>
+auto StartPersistent(BaseType_t core, const std::function<void(void)>& fn)
+ -> void {
+ StaticTask_t* task_buffer = new StaticTask_t;
+ cpp::span<StackType_t> stack = AllocateStack<t>();
xTaskCreateStaticPinnedToCore(&PersistentMain, Name<t>().c_str(),
stack.size(), new std::function<void(void)>(fn),
- Priority<t>(), stack.data(), task_buffer, 0);
+ Priority<t>(), stack.data(), task_buffer, core);
}
class Worker {
diff --git a/src/ui/lvgl_task.cpp b/src/ui/lvgl_task.cpp
index 06a6b28b..340282ee 100644
--- a/src/ui/lvgl_task.cpp
+++ b/src/ui/lvgl_task.cpp
@@ -48,12 +48,25 @@
namespace ui {
static const char* kTag = "lv_task";
+static const TickType_t kMaxFrameRate = pdMS_TO_TICKS(33);
+
+static int sTimerId;
+static SemaphoreHandle_t sFrameSemaphore;
+
+auto next_frame(TimerHandle_t) {
+ xSemaphoreGive(sFrameSemaphore);
+}
void LvglMain(std::weak_ptr<drivers::RelativeWheel> weak_touch_wheel,
std::weak_ptr<drivers::Display> weak_display) {
ESP_LOGI(kTag, "init lvgl");
lv_init();
+ sFrameSemaphore = xSemaphoreCreateBinary();
+ auto timer =
+ xTimerCreate("lvgl_frame", kMaxFrameRate, pdTRUE, &sTimerId, next_frame);
+ xTimerStart(timer, portMAX_DELAY);
+
lv_theme_t* base_theme = lv_theme_basic_init(NULL);
lv_disp_set_theme(NULL, base_theme);
static themes::Theme sTheme{};
@@ -80,9 +93,9 @@ void LvglMain(std::weak_ptr<drivers::RelativeWheel> weak_touch_wheel,
}
lv_task_handler();
- // 30 FPS
- // TODO(jacqueline): make this dynamic
- vTaskDelay(pdMS_TO_TICKS(33));
+
+ // Wait for the signal to loop again.
+ xSemaphoreTake(sFrameSemaphore, portMAX_DELAY);
}
}