Attempted to fixup CI build for Windows and Linux

Author: ValentiWorkLearning

.github/workflows/build.yml

@@ -49,7 +49,7 @@ jobs:
         cmakeListsOrSettingsJson: CMakeListsTxtAdvanced
         cmakeListsTxtPath: ${{ github.workspace }}/Firmware/CMakeLists.txt
         cmakeBuildType: Debug
-        cmakeAppendedArgs: '-DPACKAGE_TESTS=ON -DTARGET_PLATFORM:STRING="FIRMWARE_SIMULATOR"'
+        cmakeAppendedArgs: '-DPACKAGE_TESTS=ON -DTARGET_PLATFORM:STRING="FIRMWARE_SIMULATOR" -DCMAKE_BUILD_TYPE:STRING=Release'
         buildDirectory: '${{ runner.workspace }}/build/vsfirmwarebuild'
         buildWithCMake: true
     - name : Run firmware testing

Firmware/drivers/gpio/gpio_pin.cpp

@@ -52,4 +52,6 @@ template <std::uint8_t GpioPinNumber, Direction pinDirection>
 GpioPin<GpioPinNumber, pinDirection>::~GpioPin() = default;
 
 template class GpioPin<Gpio::Pins::Display_DataCommand, Direction::Output>;
+template class GpioPin<Gpio::Pins::Display_Reset, Direction::Output>;
+template class GpioPin<Gpio::Pins::LedPin, Direction::Output>;
 }; // namespace Gpio

Firmware/drivers/gpio/inc/gpio/gpio_pin.hpp

@@ -13,9 +13,9 @@ inline constexpr std::uint8_t Display_DataCommand = DISP_DC_PIN;
 inline constexpr std::uint8_t Display_Reset = DISP_RST;
 inline constexpr std::uint8_t LedPin = 13;
 #else
-inline constexpr std::uint8_t Display_DataCommand = 0;
-inline constexpr std::uint8_t Display_Reset = 0;
-inline constexpr std::uint8_t LedPin = 0;
+inline constexpr std::uint8_t Display_DataCommand = 26;
+inline constexpr std::uint8_t Display_Reset = 31;
+inline constexpr std::uint8_t LedPin = 13;
 #endif
 
 } // namespace Gpio::Pins

Firmware/graphics/platform/CMakeLists.txt

@@ -32,7 +32,7 @@ elseif( ${TARGET_PLATFORM} STREQUAL "FIRMWARE_SIMULATOR"  )
     target_compile_definitions(
         graphics_backend
         PRIVATE
-        USE_WINSDL_BACKEND
+        USE_SDL_BACKEND
     )
     target_link_libraries(
         graphics_backend

Firmware/graphics/platform/gs_platform_layer.cpp

@@ -77,7 +77,7 @@ void PlatformBackend::executeLvTaskHandler() noexcept
 } // namespace Graphics
 #endif
 
-#if defined USE_WINSDL_BACKEND
+#if defined USE_SDL_BACKEND
 
 #include <chrono>
 #include <thread>
@@ -92,7 +92,10 @@ void PlatformBackend::executeLvTaskHandler() noexcept
 namespace Graphics
 {
 
-PlatformBackend::PlatformBackend() noexcept = default;
+PlatformBackend::PlatformBackend() noexcept
+{
+    m_isTickThreadRunning.store(false);
+};
 
 void PlatformBackend::platformDependentInit(lv_disp_drv_t* _displayDriver) noexcept
 {
@@ -102,14 +105,6 @@ void PlatformBackend::platformDependentInit(lv_disp_drv_t* _displayDriver) noexc
 
 void PlatformBackend::initPlatformGfxTimer() noexcept
 {
-    m_tickThread = std::thread([] {
-        while (true)
-        {
-            lv_tick_inc(LvglNotificationTime);
-            std::this_thread::sleep_for(std::chrono::milliseconds(LvglNotificationTime));
-        }
-    });
-    m_tickThread.detach();
     indevPlatformInit();
     lv_indev_drv_init(&m_indevDriver);
 }
@@ -144,6 +139,18 @@ void PlatformBackend::memoryMonitor(lv_timer_t* _param) noexcept
 
 void PlatformBackend::executeLvTaskHandler() noexcept
 {
+    if(!m_isTickThreadRunning)
+    {
+        m_isTickThreadRunning = true;
+        m_tickThread = std::thread([this] {
+            while (m_isTickThreadRunning)
+            {
+                lv_tick_inc(LvglNotificationTime);
+                std::this_thread::sleep_for(std::chrono::milliseconds(LvglNotificationTime));
+            }
+        });
+        m_tickThread.detach();
+    }
     lv_task_handler();
     std::this_thread::sleep_for(std::chrono::milliseconds(LvglNotificationTime));
 }

Firmware/graphics/platform/gs_platform_layer.hpp

@@ -4,9 +4,10 @@
 
 #include <lvgl.h>
 
-#ifdef USE_WINSDL_BACKEND
+#ifdef USE_SDL_BACKEND
 #include <chrono>
 #include <thread>
+#include <atomic>
 #endif // USE_WINSDL_BACKEND
 
 namespace DisplayDriver
@@ -52,8 +53,9 @@ class PlatformBackend
     std::unique_ptr<TDisplayDriver> m_hardwareDisplayDriver;
 #endif
 
-#if defined USE_WINSDL_BACKEND
+#if defined USE_SDL_BACKEND
     std::thread m_tickThread;
+     std::atomic_bool m_isTickThreadRunning;
     lv_indev_drv_t m_indevDriver;
 #endif
 };