refactor: bootloader & cmake cleanup (EdgeTX#5232)

wimalopaan · Jul 7, 2024 · d575242 · d575242
1 parent 848d794
commit d575242
Show file tree

Hide file tree

Showing 35 changed files with 470 additions and 534 deletions.
diff --git a/radio/src/CMakeLists.txt b/radio/src/CMakeLists.txt
@@ -254,6 +254,11 @@ if(SDCARD)
   set(FIRMWARE_SRC ${FIRMWARE_SRC})
 endif()
 
+if(BLUETOOTH)
+  add_definitions(-DBLUETOOTH)
+  set(SRC ${SRC} bluetooth.cpp)
+endif()
+
 if(SHUTDOWN_CONFIRMATION)
   add_definitions(-DSHUTDOWN_CONFIRMATION)
 endif()
@@ -431,11 +436,11 @@ endif()
 
 if(USBJ_EX)
   add_definitions(-DUSBJ_EX)
-  set(SRC
-    ${SRC}
+  set(SRC ${SRC}
     usb_joystick.cpp
-    )
+  )
 endif()
+
 foreach(FILE ${TARGET_SRC})
   set(SRC targets/${TARGET_DIR}/${FILE} ${SRC})
 endforeach()
@@ -463,7 +468,6 @@ else()
 endif()
 
 if(NATIVE_BUILD)
-
   set(RADIOLIB_NATIVE_SRC ${SRC})
   if(SIMU_DISKIO)
     set(RADIOLIB_NATIVE_SRC ${RADIOLIB_NATIVE_SRC}
@@ -472,7 +476,8 @@ if(NATIVE_BUILD)
 
   # Pack all radio sources into an object lib
   add_library(radiolib_native OBJECT EXCLUDE_FROM_ALL
-    ${RADIOLIB_NATIVE_SRC})
+    ${RADIOLIB_NATIVE_SRC}
+  )
 
   # Add -DSIMU here PUBLIC, so it can be imported by other targets
   # via INTERFACE_COMPILE_OPTIONS

diff --git a/radio/src/boards/generic_stm32/CMakeLists.txt b/radio/src/boards/generic_stm32/CMakeLists.txt
@@ -13,6 +13,7 @@ set(MINIMAL_BOARD_LIB_SRC
 
   boards/generic_stm32/inputs.cpp
   boards/generic_stm32/i2c_bus.cpp
+  boards/generic_stm32/bl_keys.cpp
 )
 
 # Dependencies only used in firmware

diff --git a/radio/src/boards/generic_stm32/bl_keys.cpp b/radio/src/boards/generic_stm32/bl_keys.cpp
@@ -0,0 +1,24 @@
+#include "board.h"
+#include "hal/key_driver.h"
+
+#if defined(BOOTLOADER_KEYS)
+
+bool boardBLStartCondition()
+{
+  // Trims combo activated
+  bool result = (readTrims() == BOOTLOADER_KEYS);
+#if defined(SECONDARY_BOOTLOADER_KEYS)
+  result |= (readTrims() == SECONDARY_BOOTLOADER_KEYS);
+#endif
+  return result;
+}
+
+#elif (defined(RADIO_T8) || defined(RADIO_COMMANDO8)) && !defined(RADIOMASTER_RELEASE)
+
+bool boardBLStartCondition()
+{
+  // Bind button pressed
+  return (KEYS_GPIO_REG_BIND->IDR & KEYS_GPIO_PIN_BIND) == 0;
+}
+
+#endif
diff --git a/radio/src/fonts/CMakeLists.txt b/radio/src/fonts/CMakeLists.txt
@@ -2,10 +2,13 @@
 # LVGL fonts
 #
 
-set(LVGL_FONT_DIR fonts/lvgl)
-
 file(GLOB LVGL_FONTS
   RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}/..
   lvgl/lv_font_*.c)
 
 set(LVGL_FONT_SOURCES ${LVGL_FONTS} PARENT_SCOPE)
+
+set(LVGL_FONT_SOURCES_MINIMAL
+  fonts/lvgl/lv_font_roboto_bl_16.c
+  PARENT_SCOPE
+)
diff --git a/radio/src/gui/128x64/CMakeLists.txt b/radio/src/gui/128x64/CMakeLists.txt
@@ -19,20 +19,30 @@ set(GUI_SRC
   radio_calibration.cpp
   view_main.cpp
   view_statistics.cpp
-  )
+)
 
 if(FLIGHT_MODES)
-  set(GUI_SRC
-    ${GUI_SRC}
+  set(GUI_SRC ${GUI_SRC}
     model_flightmodes.cpp
-    )
+  )
+endif()
+
+if(GVARS)
+  set(GUI_SRC ${GUI_SRC}
+    model_gvars.cpp
+  )
 endif()
 
 if(LUA_MIXER)
-  set(GUI_SRC
-    ${GUI_SRC}
+  set(GUI_SRC ${GUI_SRC}
     model_custom_scripts.cpp
-    )
+  )
 endif()
 
 include(gui/common/stdlcd/CMakeLists.txt)
+
+set(BOOTLOADER_SRC ${BOOTLOADER_SRC}
+  ${RADIO_SRC_DIR}/gui/128x64/lcd.cpp
+  ${RADIO_SRC_DIR}/gui/common/stdlcd/fonts.cpp
+  ${RADIO_SRC_DIR}/gui/common/stdlcd/utf8.cpp
+)
diff --git a/radio/src/gui/212x64/CMakeLists.txt b/radio/src/gui/212x64/CMakeLists.txt
@@ -28,6 +28,12 @@ if(FLIGHT_MODES)
     )
 endif()
 
+if(GVARS)
+  set(GUI_SRC ${GUI_SRC}
+    model_gvars.cpp
+  )
+endif()
+
 if(LUA_MIXER)
   set(GUI_SRC
     ${GUI_SRC}
@@ -36,3 +42,9 @@ if(LUA_MIXER)
 endif()
 
 include(gui/common/stdlcd/CMakeLists.txt)
+
+set(BOOTLOADER_SRC ${BOOTLOADER_SRC}
+  ${RADIO_SRC_DIR}/gui/212x64/lcd.cpp
+  ${RADIO_SRC_DIR}/gui/common/stdlcd/fonts.cpp
+  ${RADIO_SRC_DIR}/gui/common/stdlcd/utf8.cpp
+)
diff --git a/radio/src/gui/colorlcd/CMakeLists.txt b/radio/src/gui/colorlcd/CMakeLists.txt
@@ -113,13 +113,6 @@ set(GUI_SRC
   radio_version.cpp
   )
 
-if(FUNCTION_SWITCHES)
-  set(GUI_SRC ${GUI_SRC}
-  function_switches.cpp
-  radio_diagcustswitches.cpp
-  )
-endif()
-
 macro(add_gui_src src)
   set(GUI_SRC
     ${GUI_SRC}
@@ -141,6 +134,15 @@ if(LUA_MIXER)
   add_gui_src(model_mixer_scripts.cpp)
 endif()
 
+if(FUNCTION_SWITCHES)
+  add_gui_src(function_switches.cpp)
+  add_gui_src(radio_diagcustswitches.cpp)
+endif()
+
+if(GVARS)
+  add_gui_src(model_gvars.cpp)
+endif()
+
 if(HELI)
   add_gui_src(model_heli.cpp)
 endif()
@@ -187,6 +189,10 @@ if(BLUETOOTH)
   add_gui_src(hw_bluetooth.cpp)
 endif()
 
+if(USBJ_EX)
+  add_gui_src(model_usbjoystick.cpp)
+endif()
+
 set(SRC ${SRC} storage/modelslist.cpp)
 
 option(WINDOWS_INSPECT_BORDERS "Draw bounding boxes around windows" OFF)
@@ -206,10 +212,28 @@ set(LVGL_SRC_DIR ${LIBOPENUI_SRC_DIR}/thirdparty/lvgl/src)
 include(${LIBOPENUI_SRC_DIR}/src/CMakeLists.txt)
 include(${LIBOPENUI_SRC_DIR}/thirdparty/CMakeLists.txt)
 
-set(SRC ${SRC} ${LVGL_FONT_SOURCES})
+list(TRANSFORM LVGL_SOURCES_MINIMAL PREPEND ${RADIO_SRC_DIR}/)
+list(TRANSFORM LVGL_FONT_SOURCES_MINIMAL PREPEND ${RADIO_SRC_DIR}/)
+
+set(BOOTLOADER_SRC ${BOOTLOADER_SRC}
+  ${LVGL_SOURCES_MINIMAL}
+  ${LVGL_FONT_SOURCES_MINIMAL}
+  ${RADIO_SRC_DIR}/gui/colorlcd/lcd.cpp
+  ${RADIO_SRC_DIR}/gui/colorlcd/fonts.cpp
+  ${RADIO_SRC_DIR}/gui/colorlcd/colors.cpp
+  ${RADIO_SRC_DIR}/gui/colorlcd/bitmaps.cpp
+  ${RADIO_SRC_DIR}/${LIBOPENUI_SRC_DIR}/thirdparty/lz4/lz4.c
+  ${RADIO_SRC_DIR}/${LIBOPENUI_SRC_DIR}/src/bitmapbuffer.cpp
+)
 
-add_definitions(-DLIBOPENUI)
+set(SRC ${SRC}
+  ${LVGL_SOURCES}
+  ${LVGL_FONT_SOURCES}
+  ${LIBOPENUI_SOURCES}
+)
 
-if(USBJ_EX)
-  add_gui_src(model_usbjoystick.cpp)
-endif()
+foreach(LVGL_FILE ${LVGL_SOURCES})
+  SET_SOURCE_FILES_PROPERTIES(${LVGL_FILE} PROPERTIES COMPILE_FLAGS -O3)
+endforeach()
+
+add_definitions(-DLIBOPENUI)
diff --git a/radio/src/hal/CMakeLists.txt b/radio/src/hal/CMakeLists.txt
@@ -1,12 +1,15 @@
 
 AddHWGenTarget(${HW_DESC_JSON} hal_keys hal_keys.inc)
+# add_library(hal_keys OBJECT EXCLUDE_FROM_ALL
+#   ${CMAKE_CURRENT_BINARY_DIR}/hal_keys.inc
+#   hal/key_driver.cpp
+# )
 
 set(SRC ${SRC}
   ${CMAKE_CURRENT_BINARY_DIR}/hal_keys.inc
-
+  hal/key_driver.cpp
   hal/module_port.cpp
   hal/adc_driver.cpp
-  hal/key_driver.cpp
   hal/switch_driver.cpp
 )
 

diff --git a/radio/src/targets/common/arm/stm32/CMakeLists.txt b/radio/src/targets/common/arm/stm32/CMakeLists.txt
@@ -117,26 +117,8 @@ if(NOT NATIVE_BUILD)
   set(BOOTLOADER_SRC ${BOOTLOADER_SRC} $<TARGET_OBJECTS:stm32_drivers_w_dbg_bl>)
 endif()
 
-set(FIRMWARE_TARGET_SRC ${FIRMWARE_TARGET_SRC}
-  ../common/arm/stm32/rtc_driver.cpp
-  ../common/arm/stm32/delays_driver.cpp
-  ../common/arm/stm32/trainer_driver.cpp
-  ../common/arm/stm32/flash_driver.cpp
-  ../common/arm/stm32/stm32_softserial_driver.cpp
-  ../common/arm/stm32/stm32_pulse_driver.cpp
-  ../common/arm/stm32/module_timer_driver.cpp
-  ../common/arm/stm32/stm32_switch_driver.cpp
-  ../common/arm/stm32/mixer_scheduler_driver.cpp
-  ../common/arm/stm32/pwr_driver.cpp
-  ../common/arm/stm32/watchdog_driver.cpp
-  ../common/arm/stm32/abnormal_reboot.cpp
-  )
-
 if(IMU)
   add_definitions(-DIMU)
-  set(FIRMWARE_SRC ${FIRMWARE_SRC}
-    targets/common/arm/stm32/lsm6ds_driver.cpp
-    )
   set(SRC ${SRC} gyro.cpp)
 endif()
 
@@ -164,10 +146,6 @@ if(DEBUG OR CLI OR USB_SERIAL)
   message("-- Adding support for USB serial")
 endif()
 
-if(GVARS)
-  set(GUI_SRC ${GUI_SRC} ${GVAR_SCREEN})
-endif()
-
 set(FIRMWARE_SRC ${FIRMWARE_SRC} syscalls.c)
 
 set(TARGET_SRC ${TARGET_SRC} ../common/arm/stm32/cpu_id.cpp)