everything working again

.gitignore

@@ -24,7 +24,9 @@ venv
 
 # Build folders
 /build*
+/cmake-build*
 
+# Makefile artifacts
 _dep
 _obj
 _bin

.idea/runConfigurations/fsfw_example_stm32h7_freertos_always_display_size.xml

@@ -0,0 +1,7 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="false" name="fsfw-example-stm32h7-freertos_always_display_size" type="CMakeRunConfiguration" factoryName="Application" REDIRECT_INPUT="false" ELEVATE="false" USE_EXTERNAL_CONSOLE="false" PASS_PARENT_ENVS_2="true" PROJECT_NAME="fsfw-example-stm32h7-freertos" TARGET_NAME="fsfw-example-stm32h7-freertos_always_display_size" CONFIG_NAME="Debug">
+    <method v="2">
+      <option name="com.jetbrains.cidr.execution.CidrBuildBeforeRunTaskProvider$BuildBeforeRunTask" enabled="true" />
+    </method>
+  </configuration>
+</component>

CMakeLists.txt

@@ -1,38 +1,48 @@
-################################################################################
+# ##############################################################################
-# CMake support for the Flight Software Framework
+# CMake support for the Flight Software Framework Author: R. Mueller
-# Author: R. Mueller
+# ##############################################################################
-################################################################################
 
-################################################################################
+# ##############################################################################
 # Pre-Project preparation
-################################################################################
+# ##############################################################################
 cmake_minimum_required(VERSION 3.13)
 
 # set(CMAKE_VERBOSE TRUE)
 
 set(CMAKE_SCRIPT_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake")
+set(CMAKE_COMMON_SCRIPT_PATH "${CMAKE_CURRENT_SOURCE_DIR}/example_common/cmake")
+
+list(APPEND CMAKE_MODULE_PATH ${CMAKE_SCRIPT_PATH})
+list(APPEND CMAKE_MODULE_PATH ${CMAKE_COMMON_SCRIPT_PATH})
+
+message(STATUS ${CMAKE_MODULE_PATH})
+
 set(STM32_CMAKE_PATH "${CMAKE_SCRIPT_PATH}/stm32-cmake")
 set(STM32_CUBE_H7_PATH STM32CubeH7)
 set(STM32_TOOLCHAIN_PATH $ENV{STM32_TOOLCHAIN_PATH})
 set(TARGET_TRIPLET "arm-none-eabi")
-set(FSFW_OSAL freertos CACHE STRING "OS for the FSFW")
-set(STM32_ADD_NETWORKING_CODE ON)
-set(FSFW_HAL_ADD_STM32H7 ON)
 
 # This call has to come before the project call
 set(CMAKE_TOOLCHAIN_FILE ${STM32_CMAKE_PATH}/cmake/stm32_gcc.cmake)
 # Project Name
 project(fsfw-example-stm32h7-freertos ASM C CXX)
 
-################################################################################
+set(FSFW_OSAL
+    freertos
+    CACHE STRING "OS for the FSFW")
+set(STM32_ADD_NETWORKING_CODE ON)
+set(FSFW_HAL_ADD_STM32H7 ON)
+
+set(OBSW_MAX_SCHEDULED_TCS 200)
+
+# ##############################################################################
 # Pre-Sources preparation
-################################################################################
+# ##############################################################################
 
 # Specify the C++ standard
-set(CMAKE_CXX_STANDARD 11)
+set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED True)
 
-
 set(FSFW_OSAL "freertos")
 set(TGT_BSP "arm/stm32h743zi-nucleo")
 set(FREERTOS_NAMESPACE FreeRTOS::STM32::H7::M7)
@@ -52,29 +62,41 @@ set(BSP_NUCLEO_PATH "${BSP_PATH}/NUCLEO-H743ZI")
 set(BSP_NUCLEO_INC_PATH "${BSP_NUCLEO_PATH}/Inc")
 
 # See stm32-cmake documentation for more information
-find_package(CMSIS COMPONENTS STM32H743ZI STM32H7_M7 RTOS REQUIRED)
+find_package(
-find_package(FreeRTOS COMPONENTS STM32H7 ARM_CM7 REQUIRED)
+  CMSIS
-find_package(HAL COMPONENTS STM32H7M7 STM32H743ZI RCC GPIO UART TIM CORTEX ETH SPI DMA REQUIRED)
+  COMPONENTS STM32H743ZI STM32H7_M7 RTOS
+  REQUIRED)
+find_package(
+  FreeRTOS
+  COMPONENTS STM32H7 ARM_CM7
+  REQUIRED)
+find_package(
+  HAL
+  COMPONENTS STM32H7M7
+             STM32H743ZI
+             RCC
+             GPIO
+             UART
+             TIM
+             CORTEX
+             ETH
+             SPI
+             DMA
+  REQUIRED)
 find_package(LwIP REQUIRED)
 
-# These need to be set for the FSFW so the sources are compiled with the ABI flags
+# These need to be set for the FSFW so the sources are compiled with the ABI
-# and to compile FreeRTOS first
+# flags and to compile FreeRTOS first
 set(LIB_OS_NAME ${FREERTOS_NAMESPACE}::ARM_CM7)
 # This target is used because a custom linker script is used
 set(CMSIS_LINK_TARGET CMSIS::STM32::H743xx::M7)
 set(FSFW_ADDITIONAL_LINK_LIBS ${CMSIS_LINK_TARGET})
-set(FSFW_HAL_LINK_LIBS 
+set(FSFW_HAL_LINK_LIBS HAL::STM32::H7::M7::CORTEX ${CMSIS_LINK_TARGET}
-    HAL::STM32::H7::M7::CORTEX
+                       ${LIB_OS_NAME})
-    ${CMSIS_LINK_TARGET}
-    ${LIB_OS_NAME}
-)
 set(FSFW_HAL_ADDITIONAL_INC_PATHS ${BSP_NUCLEO_INC_PATH})
 set(FSFW_ADDITIONAL_INC_PATHS
-    "${COMMON_CONFIG_PATH}"
+    "${COMMON_CONFIG_PATH}" "${CMAKE_CURRENT_BINARY_DIR}"
-    "${CMAKE_CURRENT_BINARY_DIR}"
+    CACHE STRING "FSFW configuration paths")
-    CACHE STRING
-    "FSFW configuration paths"
-)
 
 configure_file(${COMMON_CONFIG_PATH}/commonConfig.h.in commonConfig.h)
 configure_file(${FSFW_CONFIG_PATH}/FSFWConfig.h.in FSFWConfig.h)
@@ -82,25 +104,9 @@ configure_file(${FSFW_CONFIG_PATH}/OBSWConfig.h.in OBSWConfig.h)
 configure_file(${BSP_NUCLEO_PATH}/Inc/FreeRTOSConfig.h.in FreeRTOSConfig.h)
 configure_file(${BSP_NUCLEO_PATH}/Inc/lwipopts.h.in lwipopts.h)
 
-if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
+# ##############################################################################
-    set(WARNING_FLAGS
-        -Wall
-        -Wextra
-        -Wimplicit-fallthrough=1
-        -Wno-unused-parameter
-        -Wno-psabi
-    )
-
-    set(FSFW_WARNING_FLAGS ${WARNING_FLAGS})
-elseif(CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
-    add_compile_options(/permissive- /d2SSAOptimizer-)
-    # To avoid nameclashes with min and max macro
-    add_compile_definitions(NOMINMAX)
-endif()
-
-################################################################################
 # Executable and Sources
-################################################################################
+# ##############################################################################
 
 # Add executable
 add_executable(${TARGET_NAME})
@@ -108,17 +114,44 @@ add_executable(${TARGET_NAME})
 # Add subdirectories
 add_subdirectory(${BSP_PATH})
 add_subdirectory(${FSFW_PATH})
-# add_subdirectory(${LIB_FSFW_HAL_PATH})
 add_subdirectory(${COMMON_PATH})
 
-################################################################################
+# ##############################################################################
 # Post-Sources preparation
-################################################################################
+# ##############################################################################
+
+# Check whether the user has already installed ETL first
+# find_package(etl ${FSFW_ETL_LIB_MAJOR_VERSION} CONFIG QUIET)
+  # Not installed, so use FetchContent to download and provide etl
+  # if(NOT etl_FOUND)
+# There are issues using find_package. For now, always use FetchContent
+# See this github issue: https://github.com/ETLCPP/etl/issues/548
+message(
+  STATUS
+    "No ETL installation was found with find_package. Installing and providing "
+    "etl with FindPackage")
+include(FetchContent)
+
+FetchContent_Declare(
+  etl
+  GIT_REPOSITORY https://github.com/ETLCPP/etl
+  GIT_TAG ${FSFW_ETL_LIB_VERSION})
+
+list(APPEND FSFW_FETCH_CONTENT_TARGETS etl)
+# endif()
+
+# The documentation for FetchContent recommends declaring all the dependencies
+# before making them available. We make all declared dependency available here
+# after their declaration
+if(FSFW_FETCH_CONTENT_TARGETS)
+  FetchContent_MakeAvailable(${FSFW_FETCH_CONTENT_TARGETS})
+endif()
 
 # Add libraries for all sources.
-target_link_libraries(${TARGET_NAME} PRIVATE
+target_link_libraries(
-    ${LIB_FSFW_NAME}
+  ${TARGET_NAME}
-    # ${LIB_FSFW_HAL_NAME}
+  PRIVATE ${LIB_FSFW_NAME}
+          etl::etl
           ${FREERTOS_NAMESPACE}::Heap::4
           ${FREERTOS_NAMESPACE}::ARM_CM7
           HAL::STM32::H7::M7::RCC
@@ -139,46 +172,18 @@ target_link_libraries(${TARGET_NAME} PRIVATE
           STM32::Nano::FloatScan
           LwIP
           LwIP::IPv4
-    LwIP::NETIF
+          LwIP::NETIF)
-)
+
+include(common)
+get_common_build_flags(${TARGET_NAME})
 
 # Add include paths for all sources.
-target_include_directories(${TARGET_NAME} PRIVATE
+target_include_directories(
-    ${CMAKE_CURRENT_SOURCE_DIR}
+  ${TARGET_NAME} PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}
-    ${CMAKE_CURRENT_BINARY_DIR}
+                         ${CMAKE_CURRENT_BINARY_DIR} ${FSFW_CONFIG_PATH})
-    ${FSFW_CONFIG_PATH}
-)    
 
-
+target_compile_options(${TARGET_NAME} PRIVATE ${COMMON_COMPILE_OPTS} ${COMMON_WARNING_FLAGS})
-if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
+target_link_options(${TARGET_NAME} PRIVATE ${COMMON_LINK_OPTS})
-    target_link_options(${TARGET_NAME} PRIVATE
-        "-Wl,-Map=${TARGET_NAME}.map"
-    )
-
-    # Remove unused sections.
-    target_compile_options(${TARGET_NAME} PRIVATE
-        "-ffunction-sections"
-        "-fdata-sections"
-    )
-
-    # Removed unused sections.
-    if(NOT FSFW_OSAL MATCHES rtems)
-        target_link_options(${TARGET_NAME} PRIVATE
-            "-Wl,--gc-sections"
-        )
-    endif()
-elseif(CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
-endif()
-
-if(CMAKE_VERBOSE)
-    message(STATUS "Warning flags: ${WARNING_FLAGS}")
-endif()
-         
-
-# Compile options for all sources.
-target_compile_options(${TARGET_NAME} PRIVATE
-    ${WARNING_FLAGS}
-)
 
 if(TGT_BSP)
   set(TARGET_STRING "Target BSP: ${TGT_BSP}")
@@ -186,16 +191,14 @@ else()
   set(TARGET_STRING "Target BSP: Hosted")
 endif()
 
-string(CONCAT POST_BUILD_COMMENT
+string(CONCAT POST_BUILD_COMMENT "Build directory: ${CMAKE_BINARY_DIR}\n"
-    "Build directory: ${CMAKE_BINARY_DIR}\n"
               "Target OSAL: ${FSFW_OSAL}\n"
-    "Target Build Type: ${CMAKE_BUILD_TYPE}\n"
+              "Target Build Type: ${CMAKE_BUILD_TYPE}\n" "${TARGET_STRING}")
-    "${TARGET_STRING}"
-)
 
-stm32_add_linker_script(${TARGET_NAME} "PRIVATE" "${BSP_NUCLEO_PATH}/STM32H743ZITx_FLASH.ld")
+stm32_add_linker_script(${TARGET_NAME} "PRIVATE"
+                        "${BSP_NUCLEO_PATH}/STM32H743ZITx_FLASH.ld")
 
-include (${CMAKE_SCRIPT_PATH}/BuildType.cmake)
+include(BuildType)
 set_build_type()
 
 stm32_print_size_of_target(${TARGET_NAME})

README.md

@@ -1,4 +1,4 @@
-<img align="center" src="https://egit.irs.uni-stuttgart.de/fsfw/fsfw/raw/branch/mueller/master/misc/logo/FSFW_Logo_V3_bw.png" width="50%">
+<img align="center" src="https://egit.irs.uni-stuttgart.de/fsfw/fsfw/raw/branch/development/misc/logo/FSFW_Logo_V3_bw.png" width="50%">
 
 # <a id="top"></a> <a name="linux"></a> FSFW Example Application
 
@@ -48,23 +48,28 @@ This demo can be run on a STM32H743ZI-Nucleo board with the FreeRTOS OSAL.
 
 ## General Information
 
-The board is flashed and debugged with OpenOCD and this README specifies on how
+The board is flashed and debugged with [OpenOCD](https://openocd.org/) and this README specifies on
-to make this work with the Eclipse IDE. Other IDEs or the command line can be used as well as long 
+how to make this work with the Eclipse IDE. Other IDEs or the command line can be used as well as
-as long as OpenOCD integration is given. The example demo uses newlib nano (glibc). 
+long as long as OpenOCD integration is given. The example demo uses newlib nano (glibc). 
-Some system calls were overriden so the C and C++ stdio functions work. IO is sent via the HUART3,
+Some system calls were overriden so the C and C++ `stdio` functions work. IO is sent via the HUART3,
 so debug output can be read directly from the USB connection to the board.
 
-## Prerequisite
+## Prerequisites
 
-1. [MinGW64](https://www.msys2.org/) or [Ninja Build](https://ninja-build.org/) installed on Windows.
+If you have not set up the prerequisites yet, go to the [prerequisites chapter](#prereq) for
-   Not required on Linux.
+detailed instructions on how to install these.
-2. [GNU ARM Toolchain](https://xpack.github.io/arm-none-eabi-gcc/install/) installed, recommended 
+
-   to add binaries to system path.
+Here is a brief overview of the required tools to develop software for the STM32H7 microcontroller:
-3. Recommended for application code development: 
+
+1. CMake build system generator installed
+2. Build system like [Ninja Build](https://ninja-build.org/) or [Make](https://www.msys2.org/)
+   installed.
+3. Bare-Metal ARM toolchain installed
+4. Recommended for application code development: 
    [Eclipse for C/C++](https://www.eclipse.org/downloads/packages/) installed with the Eclipse MCU 
    plugin
-4. [OpenOCD](https://xpack.github.io/openocd/) installed for Eclipse debugging
+5. [OpenOCD](https://xpack.github.io/openocd/) installed for Eclipse debugging
-5. STM32 USB drivers installed, separate steps for 
+6. STM32 USB drivers installed, separate steps for 
    [Windows](https://www.st.com/en/development-tools/stsw-link009.html) or 
    [Linux](https://fishpepper.de/2016/09/16/installing-using-st-link-v2-to-flash-stm32-on-linux/)
 
@@ -75,7 +80,7 @@ after installing MSYS2 or inside another Unix shell like `git bash`.
 
 1. Clone this repository
    ```sh
-   git clone https://egit.irs.uni-stuttgart.de/fsfw/fsfw_example.git
+   git clone https://egit.irs.uni-stuttgart.de/fsfw/fsfw-example-stm32h7-freertos
    ```
 
 2. Set up submodules
@@ -88,8 +93,8 @@ after installing MSYS2 or inside another Unix shell like `git bash`.
    Debug build folder.
 
    ```sh
-   mkdir build-Debug
+   mkdir cmake-build-debug
-   cd build-Debug
+   cd cmake-build-debug
    ```
 
 4. Ensure that the ARM compiler has been added to the path and can be called from 
@@ -131,79 +136,145 @@ after installing MSYS2 or inside another Unix shell like `git bash`.
    The debug output is also sent via the connected USB port and a blink pattern (1 second interval) 
    can be used to verify the software is running properly.
    
-## Setting up the prerequisites
+## <a id="prereq"></a> Setting up Prerequisites
 
-### Windows
+### CMake
 
-It is recommended to install [MSYS2](https://www.msys2.org/) first.
+**Linux**
-Open MinGW64 and run the following commands to update it and install make and cmake 
-(replace x86_64 if compiling on different architecture):
 
 ```sh
-pacman -Syu
+sudo apt-get install cmake
 ```
 
-```sh
+**Windows**
-pacman -S mingw-w64-x86_64-make mingw-w64-x86_64-cmake
-```
 
-Alternatively, you can install [Ninja Build](https://ninja-build.org/), but you need
+On Windows, you can use `pacman -S mingw-w64-x86_64-cmake`, but you can also install the Windows
-to add the folder containing the `ninja.exe` executable to the system path so you
+CMake via the [installer](https://cmake.org/download/). It is recommended to pick the install
-can run `ninja -v` from the command line. If you do this, you can also use
+option `Add CMake to system PATH for all users` to CMake can be used from the command line.
-`git bash` or the Windows command lines with the CMake Ninja generator
+Please note that you need to add the Windows CMake path to the MinGW64 path manually
-to build the software.
+if you want to use it in CMake.
 
-The code needs to be cross-compiled for the ARM target system and we will use the
+### Cross-Compiler
-[GNU ARM Toolchain](https://xpack.github.io/arm-none-eabi-gcc/install/).
 
-1. Install NodeJS LTS. Add nodejs folder (e.g. "C:\Program Files\nodejs\")
+The instuctions here specify how to install and use a specific version of the
-   to system variables. Test by running `npm --version` in command line
+[xPacks cross-compiler](https://xpack.github.io/arm-none-eabi-gcc/) but you can use any other
-2. Install [XPM](https://www.npmjs.com/package/xpm)
+ARM cross-compiler which can generate bare-metal code, usually denoted by the `arm-none-eabi`
-   ```sh
+cross-compiler triplet.
-   npm install --global xpm
-   ```
-
-3. Install gnu-arm Toolchain for Eclipse (version can be specified)
-   ```sh
-   xpm install --global @xpack-dev-tools/arm-none-eabi-gcc@latest
-   ```
-   
-   `xpm` will display where the package was installed. Search in that path for
-   the hidden `.content` folder, which will contain a `bin` folder and store
-   the full path to that `bin` folder for later.
-
-   Install OpenOCD for STM32 debugging
-   ```sh
-   xpm install --global @xpack-dev-tools/openocd@latest
-   ```
-   
-4. If you want to build from the command line, you need to add the `arm-none-eabi-gcc` 
-   binary location in the xPack folder to system variables. You can do this in a Unix
-   shell like `git bash` or `MinGW64` with the following command
-
-   ```sh
-   export PATH=$PATH:"<pathToToolchain>"
-   ```
-   
-   You can also add these lines to a shell script like `path_setter.sh` and then source
-   the script with `. path_setter.sh` to do this conveniently. You can test whether
-   the path was set up properly by running `arm-none-eabi-gcc -v`
-
-5. Install the [STM32 USB drivers](https://www.st.com/en/development-tools/stsw-link009.html)
 
 If you don't want to install nodejs you may go with the 
 [four-command manual installation](https://xpack.github.io/arm-none-eabi-gcc/install/#manual-install). 
 
-### Linux
+**Windows**
 
-Install the [GNU ARM toolchain](https://xpack.github.io/arm-none-eabi-gcc/install/)
+On Windows, it is recommended to perform the `xpm` and toolchain installation from the Windows
-like explained above.
+command line.The simple way required npm, which can be installed by installing
+[NodeJS](https://nodejs.org/en/). Make sure `npm` can be run from the command line by adding
+the folder containing `npm.exe` to the system path and running the following command 
 
-To install general buildtools for the linux binary, run:
 ```sh
-sudo apt-get install build-essential
+npm install --global xpm@latest
+xpm install --global @xpack-dev-tools/arm-none-eabi-gcc@11.2.1-1.1.1 --verbose
 ```
 
+On Windows, the toolchain binaries will be located in a folder like this
+
+```sh
+C:\Users\<User>\AppData\Roaming\xPacks\@xpack-dev-tools\arm-none-eabi-gcc\<version>\.content\bin
+```
+
+You can now run the following commands in the repository root:
+
+```sh
+xpm init
+xpm install @xpack-dev-tools/arm-none-eabi-gcc@11.2.1-1.1.1
+```
+
+to create symlinks to the toolchain in `./xpacks/.bin`.
+You can now set up the environment by using `. load_path.sh` or the following command
+
+```sh
+export PATH="$(pwd)/xpacks/.bin":$PATH
+```
+
+On Windows, you can use git bash to perform the step above.
+
+Alternatively, you can use the graphical system environmental variables editor to add the
+`.bin` path to the system variables permanently or use the appriate command for `CMD` or
+PowerShell to update the `PATH` variable.
+
+**Linux**
+
+Install `npm` and `nodejs` first. Example for Ubuntu according to
+[this guide](https://linuxize.com/post/how-to-install-node-js-on-ubuntu-20-04/).
+
+```sh
+curl -sL https://deb.nodesource.com/setup_14.x | sudo -E bash -
+sudo apt-get update && sudo apt-get install nodejs
+```
+
+Check that `npm` is installed with `npm --version`.
+Then `xpm` and the cross-compiler are installed.
+
+```sh
+sudo npm install --global xpm@latest
+xpm install --global @xpack-dev-tools/arm-none-eabi-gcc@11.2.1-1.1.1 --verbose
+```
+
+You can now run the following commands in the repository root:
+
+```sh
+xpm init
+xpm install @xpack-dev-tools/arm-none-eabi-gcc@11.2.1-1.1.1
+```
+
+to create symlinks to the toolchain in `./xpacks/.bin`.
+You can now set up the environment by using `. load_path.sh` or the following command
+
+```sh
+export PATH="$(pwd)/xpacks/.bin":$PATH
+```
+
+### Build System
+
+It is recommended to use `ninja` or `make` as the software build system.
+
+**Windows**
+
+It is recommended to use the [Ninja build system](https://ninja-build.org/). Download the ninja
+executable and place it somewhere.
+You need to add the folder containing the ninja executable to the system environmental variables
+so it can be used in the build process. You can test whether `ninja` works by running
+`ninja --version` in the command line.
+
+Alternatively or additionally, you can also install `mingw32-make`
+which comes bundled with [MinGW64](https://www.msys2.org/). Make sure to add the binaries path to
+the Windows path during installation. Otherwise, you need to add `msys64/mingw64/bin` to the
+Windows path so you can run installed binaries from the command line.
+Open the `MinGW64` shell and run the following commands.
+
+```sh
+pacman -S mingw-w64-x86_64-toolchain mingw-w64-x86_64-make mingw-w64-x86_64-cmake
+```
+
+You can test successfull installation with `mingw32-make -v` from the Windows Command Line.
+
+**Linux**
+
+On Linux, `make` is pre-installed and it is recommended to use it directly.
+You can install `ninja` with
+
+```sh
+sudo apt-get install ninja-build
+```
+
+### USB Drivers
+
+**Windows**
+
+Install the [STM32 USB drivers](https://www.st.com/en/development-tools/stsw-link009.html).
+
+**Linux**
+
 Install the [USB drivers](https://github.com/stlink-org/stlink) on Linux.
 On Ubuntu, you can run the following command to install it:
 

STM32CubeH7

@@ -1 +1 @@
-Subproject commit 5975bffae9358bc2b2890a35a203d940a395efef
+Subproject commit b340b13929e36a3427b8d94e8b1006022f82273f

bsp_stm32h7_freertos/CMakeLists.txt

@@ -1,16 +1,11 @@
-target_sources(${TARGET_NAME} PRIVATE
+target_sources(${TARGET_NAME} PRIVATE main.cpp)
-	main.cpp
-)
 
-target_compile_options(${TARGET_NAME} PRIVATE
+target_compile_options(${TARGET_NAME} PRIVATE)
-)
 
-target_link_options(${TARGET_NAME} PRIVATE
+target_link_options(${TARGET_NAME} PRIVATE)
-)
 
-target_compile_definitions(${TARGET_NAME} PRIVATE
+target_compile_definitions(${TARGET_NAME}
-	BOARD_NAME="stm32h743zi-nucleo"
+                           PRIVATE BOARD_NAME="stm32h743zi-nucleo")
-)
 
 add_subdirectory(core)
 add_subdirectory(utility)

bsp_stm32h7_freertos/NUCLEO-H743ZI/CMakeLists.txt

@@ -1,5 +1,3 @@
 add_subdirectory(Src)
 
-target_include_directories(${TARGET_NAME} PRIVATE
+target_include_directories(${TARGET_NAME} PRIVATE Inc)
-	Inc
-)

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/app_dhcp.h

@@ -8,12 +8,12 @@
 #include <lwip/netif.h>
 
 /* DHCP process states */
-#define DHCP_OFF                   (uint8_t) 0
+#define DHCP_OFF (uint8_t)0
-#define DHCP_START                 (uint8_t) 1
+#define DHCP_START (uint8_t)1
-#define DHCP_WAIT_ADDRESS          (uint8_t) 2
+#define DHCP_WAIT_ADDRESS (uint8_t)2
-#define DHCP_ADDRESS_ASSIGNED      (uint8_t) 3
+#define DHCP_ADDRESS_ASSIGNED (uint8_t)3
-#define DHCP_TIMEOUT               (uint8_t) 4
+#define DHCP_TIMEOUT (uint8_t)4
-#define DHCP_LINK_DOWN             (uint8_t) 5
+#define DHCP_LINK_DOWN (uint8_t)5
 
 uint8_t get_dhcp_state();
 void set_dhcp_state(uint8_t new_state);

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/app_ethernet.h

@@ -48,7 +48,7 @@
 #define __APP_ETHERNET_H
 
 #ifdef __cplusplus
- extern "C" {
+extern "C" {
 #endif
 
 /* Includes ------------------------------------------------------------------*/
@@ -58,13 +58,12 @@
 
 /* Exported constants --------------------------------------------------------*/
 
-   
 /* Exported macro ------------------------------------------------------------*/
 /* Exported functions ------------------------------------------------------- */
 void ethernet_link_status_updated(struct netif *netif);
 void ethernet_link_periodic_handle(struct netif *netif);
 
-void set_lwip_addresses(ip_addr_t* ipaddr, ip_addr_t* netmask, ip_addr_t* gw);
+void set_lwip_addresses(ip_addr_t *ipaddr, ip_addr_t *netmask, ip_addr_t *gw);
 
 #ifdef __cplusplus
 }
@@ -72,6 +71,4 @@ void set_lwip_addresses(ip_addr_t* ipaddr, ip_addr_t* netmask, ip_addr_t* gw);
 
 #endif /* __APP_ETHERNET_H */
 
-
 /************************ (C) COPYRIGHT STMicroelectronics *****END OF FILE****/
-

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/boardconfig.h

@@ -5,6 +5,4 @@
 #define DEBUG_UART_MS_TIMEOUT 10
 #define DEBUG_UART_BAUDRATE 115200
 
-
-
 #endif /* BSP_STM32_BOARDS_NUCLEO_H743ZI_INC_BOARDCONFIG_H_ */

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/hardware_init.h

@@ -9,8 +9,8 @@
 extern "C" {
 #endif
 
-#include <stdint.h>
 #include <stdbool.h>
+#include <stdint.h>
 
 #include <stm32h7xx_hal.h>
 #include <stm32h7xx_hal_gpio.h>

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/lan8742.h

@@ -23,7 +23,7 @@
 #define LAN8742_H
 
 #ifdef __cplusplus
- extern "C" {
+extern "C" {
 #endif
 
 /* Includes ------------------------------------------------------------------*/
@@ -340,13 +340,13 @@
 #define LAN8742_STATUS_ADDRESS_ERROR ((int32_t)-3)
 #define LAN8742_STATUS_RESET_TIMEOUT ((int32_t)-2)
 #define LAN8742_STATUS_ERROR ((int32_t)-1)
-#define  LAN8742_STATUS_OK                    ((int32_t) 0)
+#define LAN8742_STATUS_OK ((int32_t)0)
-#define  LAN8742_STATUS_LINK_DOWN             ((int32_t) 1)
+#define LAN8742_STATUS_LINK_DOWN ((int32_t)1)
-#define  LAN8742_STATUS_100MBITS_FULLDUPLEX   ((int32_t) 2)
+#define LAN8742_STATUS_100MBITS_FULLDUPLEX ((int32_t)2)
-#define  LAN8742_STATUS_100MBITS_HALFDUPLEX   ((int32_t) 3)
+#define LAN8742_STATUS_100MBITS_HALFDUPLEX ((int32_t)3)
-#define  LAN8742_STATUS_10MBITS_FULLDUPLEX    ((int32_t) 4)
+#define LAN8742_STATUS_10MBITS_FULLDUPLEX ((int32_t)4)
-#define  LAN8742_STATUS_10MBITS_HALFDUPLEX    ((int32_t) 5)
+#define LAN8742_STATUS_10MBITS_HALFDUPLEX ((int32_t)5)
-#define  LAN8742_STATUS_AUTONEGO_NOTDONE      ((int32_t) 6)
+#define LAN8742_STATUS_AUTONEGO_NOTDONE ((int32_t)6)
 /**
  * @}
  */
@@ -374,14 +374,13 @@
 /** @defgroup LAN8742_Exported_Types LAN8742 Exported Types
  * @{
  */
-typedef int32_t  (*lan8742_Init_Func) (void); 
+typedef int32_t (*lan8742_Init_Func)(void);
-typedef int32_t  (*lan8742_DeInit_Func) (void);
+typedef int32_t (*lan8742_DeInit_Func)(void);
-typedef int32_t  (*lan8742_ReadReg_Func)   (uint32_t, uint32_t, uint32_t *);
+typedef int32_t (*lan8742_ReadReg_Func)(uint32_t, uint32_t, uint32_t *);
-typedef int32_t  (*lan8742_WriteReg_Func)  (uint32_t, uint32_t, uint32_t);
+typedef int32_t (*lan8742_WriteReg_Func)(uint32_t, uint32_t, uint32_t);
-typedef int32_t  (*lan8742_GetTick_Func)  (void);
+typedef int32_t (*lan8742_GetTick_Func)(void);
 
-typedef struct 
+typedef struct {
-{                   
   lan8742_Init_Func Init;
   lan8742_DeInit_Func DeInit;
   lan8742_WriteReg_Func WriteReg;
@@ -389,14 +388,12 @@ typedef struct
   lan8742_GetTick_Func GetTick;
 } lan8742_IOCtx_t;
 
-  
+typedef struct {
-typedef struct 
-{
   uint32_t DevAddr;
   uint32_t Is_Initialized;
   lan8742_IOCtx_t IO;
   void *pData;
-}lan8742_Object_t;
+} lan8742_Object_t;
 /**
  * @}
  */
@@ -429,7 +426,6 @@ int32_t LAN8742_GetITStatus(lan8742_Object_t *pObj, uint32_t Interrupt);
 #endif
 #endif /* LAN8742_H */
 
-
 /**
  * @}
  */

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/lwipopts.h.in

@@ -98,7 +98,7 @@ a lot of data that needs to be copied, this should be set high. */
 #define PBUF_POOL_SIZE          4
 
 /* PBUF_POOL_BUFSIZE: the size of each pbuf in the pbuf pool. */
-#define PBUF_POOL_BUFSIZE       1528
+#define PBUF_POOL_BUFSIZE       1536
 
 /* LWIP_SUPPORT_CUSTOM_PBUF == 1: to pass directly MAC Rx buffers to the stack 
    no copy is needed */
@@ -232,12 +232,12 @@ The STM32H7xx allows computing and verifying the IP, UDP, TCP and ICMP checksums
 */
 
 #define TCPIP_THREAD_NAME              "TCP/IP"
-#define TCPIP_THREAD_STACKSIZE          1000
+#define TCPIP_THREAD_STACKSIZE          2048
 #define TCPIP_MBOX_SIZE                 6
 #define DEFAULT_UDP_RECVMBOX_SIZE       6
 #define DEFAULT_TCP_RECVMBOX_SIZE       6
 #define DEFAULT_ACCEPTMBOX_SIZE         6
-#define DEFAULT_THREAD_STACKSIZE        500
+#define DEFAULT_THREAD_STACKSIZE        1024
 #define TCPIP_THREAD_PRIO               osPriorityHigh
 
 /*

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/main.h

@@ -62,7 +62,8 @@ extern "C" {
 /* UDP remote connection port */
 #define UDP_CLIENT_PORT 2008
 
-/*Static DEST IP ADDRESS: DEST_IP_ADDR0.DEST_IP_ADDR1.DEST_IP_ADDR2.DEST_IP_ADDR3 */
+/*Static DEST IP ADDRESS:
+ * DEST_IP_ADDR0.DEST_IP_ADDR1.DEST_IP_ADDR2.DEST_IP_ADDR3 */
 #define DEST_IP_ADDR0 ((uint8_t)169U)
 #define DEST_IP_ADDR1 ((uint8_t)254U)
 #define DEST_IP_ADDR2 ((uint8_t)39U)
@@ -92,5 +93,4 @@ extern "C" {
 
 #endif /* __MAIN_H */
 
-
 /************************ (C) COPYRIGHT STMicroelectronics *****END OF FILE****/

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/stm32h7xx_hal_conf.h

@@ -24,7 +24,7 @@
 #define __STM32H7xx_HAL_CONF_H
 
 #ifdef __cplusplus
- extern "C" {
+extern "C" {
 #endif
 
 /* Exported types ------------------------------------------------------------*/
@@ -89,63 +89,73 @@
 #define HAL_USART_MODULE_ENABLED
 #define HAL_WWDG_MODULE_ENABLED
 
-/* ########################## Oscillator Values adaptation ####################*/
+/* ########################## Oscillator Values adaptation
+ * ####################*/
 /**
-  * @brief Adjust the value of External High Speed oscillator (HSE) used in your application.
+ * @brief Adjust the value of External High Speed oscillator (HSE) used in your
-  *        This value is used by the RCC HAL module to compute the system frequency
+ * application. This value is used by the RCC HAL module to compute the system
-  *        (when HSE is used as system clock source, directly or through the PLL).  
+ * frequency (when HSE is used as system clock source, directly or through the
+ * PLL).
  */
-#if !defined  (HSE_VALUE) 
+#if !defined(HSE_VALUE)
-#define HSE_VALUE    ((uint32_t)8000000) /*!< Value of the External oscillator in Hz */
+#define HSE_VALUE                                                              \
+  ((uint32_t)8000000) /*!< Value of the External oscillator in Hz */
 #endif                /* HSE_VALUE */
 
-#if !defined  (HSE_STARTUP_TIMEOUT)
+#if !defined(HSE_STARTUP_TIMEOUT)
-  #define HSE_STARTUP_TIMEOUT    ((uint32_t)5000)   /*!< Time out for HSE start up, in ms */
+#define HSE_STARTUP_TIMEOUT                                                    \
+  ((uint32_t)5000) /*!< Time out for HSE start up, in ms */
 #endif             /* HSE_STARTUP_TIMEOUT */
 
 /**
  * @brief Internal  oscillator (CSI) default value.
  *        This value is the default CSI value after Reset.
  */
-#if !defined  (CSI_VALUE)
+#if !defined(CSI_VALUE)
-  #define CSI_VALUE    ((uint32_t)4000000) /*!< Value of the Internal oscillator in Hz*/
+#define CSI_VALUE                                                              \
+  ((uint32_t)4000000) /*!< Value of the Internal oscillator in Hz*/
 #endif                /* CSI_VALUE */
 
 /**
  * @brief Internal High Speed oscillator (HSI) value.
-  *        This value is used by the RCC HAL module to compute the system frequency
+ *        This value is used by the RCC HAL module to compute the system
-  *        (when HSI is used as system clock source, directly or through the PLL). 
+ * frequency (when HSI is used as system clock source, directly or through the
+ * PLL).
  */
-#if !defined  (HSI_VALUE)
+#if !defined(HSI_VALUE)
-  #define HSI_VALUE    ((uint32_t)64000000) /*!< Value of the Internal oscillator in Hz*/
+#define HSI_VALUE                                                              \
+  ((uint32_t)64000000) /*!< Value of the Internal oscillator in Hz*/
 #endif                 /* HSI_VALUE */
 
 /**
  * @brief External Low Speed oscillator (LSE) value.
-  *        This value is used by the UART, RTC HAL module to compute the system frequency
+ *        This value is used by the UART, RTC HAL module to compute the system
+ * frequency
  */
-#if !defined  (LSE_VALUE)
+#if !defined(LSE_VALUE)
-  #define LSE_VALUE    ((uint32_t)32768) /*!< Value of the External oscillator in Hz*/
+#define LSE_VALUE                                                              \
+  ((uint32_t)32768) /*!< Value of the External oscillator in Hz*/
 #endif              /* LSE_VALUE */
 
-   
+#if !defined(LSE_STARTUP_TIMEOUT)
-#if !defined  (LSE_STARTUP_TIMEOUT)
+#define LSE_STARTUP_TIMEOUT                                                    \
-  #define LSE_STARTUP_TIMEOUT    ((uint32_t)5000)   /*!< Time out for LSE start up, in ms */
+  ((uint32_t)5000) /*!< Time out for LSE start up, in ms */
 #endif             /* LSE_STARTUP_TIMEOUT */
 
-#if !defined  (LSI_VALUE) 
+#if !defined(LSI_VALUE)
-  #define LSI_VALUE  ((uint32_t)32000)      /*!< LSI Typical Value in Hz*/
+#define LSI_VALUE ((uint32_t)32000) /*!< LSI Typical Value in Hz*/
-#endif /* LSI_VALUE */                      /*!< Value of the Internal Low Speed oscillator in Hz
+#endif /* LSI_VALUE */ /*!< Value of the Internal Low Speed oscillator in Hz   \
-                                              The real value may vary depending on the variations
+                         The real value may vary depending on the variations   \
                          in voltage and temperature.*/
 
 /**
  * @brief External clock source for I2S peripheral
-  *        This value is used by the I2S HAL module to compute the I2S clock source 
+ *        This value is used by the I2S HAL module to compute the I2S clock
-  *        frequency, this source is inserted directly through I2S_CKIN pad. 
+ * source frequency, this source is inserted directly through I2S_CKIN pad.
  */
-#if !defined  (EXTERNAL_CLOCK_VALUE)
+#if !defined(EXTERNAL_CLOCK_VALUE)
-  #define EXTERNAL_CLOCK_VALUE    12288000U /*!< Value of the External clock in Hz*/
+#define EXTERNAL_CLOCK_VALUE 12288000U /*!< Value of the External clock in     \
+                                          Hz*/
 #endif                                 /* EXTERNAL_CLOCK_VALUE */
 
 /* Tip: To avoid modifying this file each time you need to use different HSE,
@@ -161,7 +171,8 @@
 /* #define  USE_SD_TRANSCEIVER           1U   */ /*!< use uSD Transceiver */
 #define USE_SPI_CRC 1U                           /*!< use CRC in SPI */
 
-/* ########################### Ethernet Configuration ######################### */
+/* ########################### Ethernet Configuration #########################
+ */
 #define ETH_TX_DESC_CNT 4 /* number of Ethernet Tx DMA descriptors */
 #define ETH_RX_DESC_CNT 4 /* number of Ethernet Rx DMA descriptors */
 
@@ -179,131 +190,129 @@
  */
 /* #define USE_FULL_ASSERT    1 */
 
-
 /* Includes ------------------------------------------------------------------*/
 /**
  * @brief Include module's header file
  */
 
 #ifdef HAL_RCC_MODULE_ENABLED
-  #include "stm32h7xx_hal_rcc.h"
+#include "stm32h7xx_hal_rcc.h"
 #endif /* HAL_RCC_MODULE_ENABLED */
 
 #ifdef HAL_GPIO_MODULE_ENABLED
-  #include "stm32h7xx_hal_gpio.h"
+#include "stm32h7xx_hal_gpio.h"
 #endif /* HAL_GPIO_MODULE_ENABLED */
 
 #ifdef HAL_DMA_MODULE_ENABLED
-  #include "stm32h7xx_hal_dma.h"
+#include "stm32h7xx_hal_dma.h"
 #endif /* HAL_DMA_MODULE_ENABLED */
 
 #ifdef HAL_MDMA_MODULE_ENABLED
- #include "stm32h7xx_hal_mdma.h"
+#include "stm32h7xx_hal_mdma.h"
 #endif /* HAL_MDMA_MODULE_ENABLED */
 
 #ifdef HAL_HASH_MODULE_ENABLED
-  #include "stm32h7xx_hal_hash.h"
+#include "stm32h7xx_hal_hash.h"
 #endif /* HAL_HASH_MODULE_ENABLED */
 
 #ifdef HAL_DCMI_MODULE_ENABLED
-  #include "stm32h7xx_hal_dcmi.h"
+#include "stm32h7xx_hal_dcmi.h"
 #endif /* HAL_DCMI_MODULE_ENABLED */
 
 #ifdef HAL_DMA2D_MODULE_ENABLED
-  #include "stm32h7xx_hal_dma2d.h"
+#include "stm32h7xx_hal_dma2d.h"
 #endif /* HAL_DMA2D_MODULE_ENABLED */
 
 #ifdef HAL_DFSDM_MODULE_ENABLED
-  #include "stm32h7xx_hal_dfsdm.h"
+#include "stm32h7xx_hal_dfsdm.h"
 #endif /* HAL_DFSDM_MODULE_ENABLED */
 
 #ifdef HAL_ETH_MODULE_ENABLED
-  #include "stm32h7xx_hal_eth.h"
+#include "stm32h7xx_hal_eth.h"
 #endif /* HAL_ETH_MODULE_ENABLED */
 
 #ifdef HAL_EXTI_MODULE_ENABLED
-  #include "stm32h7xx_hal_exti.h"
+#include "stm32h7xx_hal_exti.h"
 #endif /* HAL_EXTI_MODULE_ENABLED */
 
 #ifdef HAL_CORTEX_MODULE_ENABLED
-  #include "stm32h7xx_hal_cortex.h"
+#include "stm32h7xx_hal_cortex.h"
 #endif /* HAL_CORTEX_MODULE_ENABLED */
 
 #ifdef HAL_ADC_MODULE_ENABLED
-  #include "stm32h7xx_hal_adc.h"
+#include "stm32h7xx_hal_adc.h"
 #endif /* HAL_ADC_MODULE_ENABLED */
 
 #ifdef HAL_FDCAN_MODULE_ENABLED
-  #include "stm32h7xx_hal_fdcan.h"
+#include "stm32h7xx_hal_fdcan.h"
 #endif /* HAL_FDCAN_MODULE_ENABLED */
 
 #ifdef HAL_CEC_MODULE_ENABLED
-  #include "stm32h7xx_hal_cec.h"
+#include "stm32h7xx_hal_cec.h"
 #endif /* HAL_CEC_MODULE_ENABLED */
 
 #ifdef HAL_COMP_MODULE_ENABLED
-  #include "stm32h7xx_hal_comp.h"
+#include "stm32h7xx_hal_comp.h"
 #endif /* HAL_COMP_MODULE_ENABLED */
 
 #ifdef HAL_CRC_MODULE_ENABLED
-  #include "stm32h7xx_hal_crc.h"
+#include "stm32h7xx_hal_crc.h"
 #endif /* HAL_CRC_MODULE_ENABLED */
 
 #ifdef HAL_CRYP_MODULE_ENABLED
-  #include "stm32h7xx_hal_cryp.h" 
+#include "stm32h7xx_hal_cryp.h"
 #endif /* HAL_CRYP_MODULE_ENABLED */
 
 #ifdef HAL_DAC_MODULE_ENABLED
-  #include "stm32h7xx_hal_dac.h"
+#include "stm32h7xx_hal_dac.h"
 #endif /* HAL_DAC_MODULE_ENABLED */
 
 #ifdef HAL_FLASH_MODULE_ENABLED
-  #include "stm32h7xx_hal_flash.h"
+#include "stm32h7xx_hal_flash.h"
 #endif /* HAL_FLASH_MODULE_ENABLED */
 
 #ifdef HAL_HRTIM_MODULE_ENABLED
-  #include "stm32h7xx_hal_hrtim.h"
+#include "stm32h7xx_hal_hrtim.h"
 #endif /* HAL_HRTIM_MODULE_ENABLED */
 
 #ifdef HAL_HSEM_MODULE_ENABLED
-  #include "stm32h7xx_hal_hsem.h"
+#include "stm32h7xx_hal_hsem.h"
 #endif /* HAL_HSEM_MODULE_ENABLED */
 
 #ifdef HAL_SRAM_MODULE_ENABLED
-  #include "stm32h7xx_hal_sram.h"
+#include "stm32h7xx_hal_sram.h"
 #endif /* HAL_SRAM_MODULE_ENABLED */
 
 #ifdef HAL_NOR_MODULE_ENABLED
-  #include "stm32h7xx_hal_nor.h"
+#include "stm32h7xx_hal_nor.h"
 #endif /* HAL_NOR_MODULE_ENABLED */
 
 #ifdef HAL_NAND_MODULE_ENABLED
-  #include "stm32h7xx_hal_nand.h"
+#include "stm32h7xx_hal_nand.h"
 #endif /* HAL_NAND_MODULE_ENABLED */
 
 #ifdef HAL_I2C_MODULE_ENABLED
- #include "stm32h7xx_hal_i2c.h"
+#include "stm32h7xx_hal_i2c.h"
 #endif /* HAL_I2C_MODULE_ENABLED */
 
 #ifdef HAL_I2S_MODULE_ENABLED
- #include "stm32h7xx_hal_i2s.h"
+#include "stm32h7xx_hal_i2s.h"
 #endif /* HAL_I2S_MODULE_ENABLED */
 
 #ifdef HAL_IWDG_MODULE_ENABLED
- #include "stm32h7xx_hal_iwdg.h"
+#include "stm32h7xx_hal_iwdg.h"
 #endif /* HAL_IWDG_MODULE_ENABLED */
 
 #ifdef HAL_JPEG_MODULE_ENABLED
- #include "stm32h7xx_hal_jpeg.h"
+#include "stm32h7xx_hal_jpeg.h"
 #endif /* HAL_JPEG_MODULE_ENABLED */
 
 #ifdef HAL_MDIOS_MODULE_ENABLED
- #include "stm32h7xx_hal_mdios.h"
+#include "stm32h7xx_hal_mdios.h"
 #endif /* HAL_MDIOS_MODULE_ENABLED */
 
-
 #ifdef HAL_MMC_MODULE_ENABLED
- #include "stm32h7xx_hal_mmc.h"
+#include "stm32h7xx_hal_mmc.h"
 #endif /* HAL_MMC_MODULE_ENABLED */
 
 #ifdef HAL_LPTIM_MODULE_ENABLED
@@ -319,83 +328,83 @@
 #endif /* HAL_OPAMP_MODULE_ENABLED */
 
 #ifdef HAL_PWR_MODULE_ENABLED
- #include "stm32h7xx_hal_pwr.h"
+#include "stm32h7xx_hal_pwr.h"
 #endif /* HAL_PWR_MODULE_ENABLED */
 
 #ifdef HAL_QSPI_MODULE_ENABLED
- #include "stm32h7xx_hal_qspi.h"
+#include "stm32h7xx_hal_qspi.h"
 #endif /* HAL_QSPI_MODULE_ENABLED */
 
 #ifdef HAL_RAMECC_MODULE_ENABLED
- #include "stm32h7xx_hal_ramecc.h"
+#include "stm32h7xx_hal_ramecc.h"
 #endif /* HAL_HCD_MODULE_ENABLED */
 
 #ifdef HAL_RNG_MODULE_ENABLED
- #include "stm32h7xx_hal_rng.h"
+#include "stm32h7xx_hal_rng.h"
 #endif /* HAL_RNG_MODULE_ENABLED */
 
 #ifdef HAL_RTC_MODULE_ENABLED
- #include "stm32h7xx_hal_rtc.h"
+#include "stm32h7xx_hal_rtc.h"
 #endif /* HAL_RTC_MODULE_ENABLED */
 
 #ifdef HAL_SAI_MODULE_ENABLED
- #include "stm32h7xx_hal_sai.h"
+#include "stm32h7xx_hal_sai.h"
 #endif /* HAL_SAI_MODULE_ENABLED */
 
 #ifdef HAL_SD_MODULE_ENABLED
- #include "stm32h7xx_hal_sd.h"
+#include "stm32h7xx_hal_sd.h"
 #endif /* HAL_SD_MODULE_ENABLED */
 
 #ifdef HAL_SDRAM_MODULE_ENABLED
- #include "stm32h7xx_hal_sdram.h"
+#include "stm32h7xx_hal_sdram.h"
 #endif /* HAL_SDRAM_MODULE_ENABLED */
 
 #ifdef HAL_SPI_MODULE_ENABLED
- #include "stm32h7xx_hal_spi.h"
+#include "stm32h7xx_hal_spi.h"
 #endif /* HAL_SPI_MODULE_ENABLED */
 
 #ifdef HAL_SPDIFRX_MODULE_ENABLED
- #include "stm32h7xx_hal_spdifrx.h"
+#include "stm32h7xx_hal_spdifrx.h"
 #endif /* HAL_SPDIFRX_MODULE_ENABLED */
 
 #ifdef HAL_SWPMI_MODULE_ENABLED
- #include "stm32h7xx_hal_swpmi.h"
+#include "stm32h7xx_hal_swpmi.h"
 #endif /* HAL_SWPMI_MODULE_ENABLED */
 
 #ifdef HAL_TIM_MODULE_ENABLED
- #include "stm32h7xx_hal_tim.h"
+#include "stm32h7xx_hal_tim.h"
 #endif /* HAL_TIM_MODULE_ENABLED */
 
 #ifdef HAL_UART_MODULE_ENABLED
- #include "stm32h7xx_hal_uart.h"
+#include "stm32h7xx_hal_uart.h"
 #endif /* HAL_UART_MODULE_ENABLED */
 
 #ifdef HAL_USART_MODULE_ENABLED
- #include "stm32h7xx_hal_usart.h"
+#include "stm32h7xx_hal_usart.h"
 #endif /* HAL_USART_MODULE_ENABLED */
 
 #ifdef HAL_IRDA_MODULE_ENABLED
- #include "stm32h7xx_hal_irda.h"
+#include "stm32h7xx_hal_irda.h"
 #endif /* HAL_IRDA_MODULE_ENABLED */
 
 #ifdef HAL_SMARTCARD_MODULE_ENABLED
- #include "stm32h7xx_hal_smartcard.h"
+#include "stm32h7xx_hal_smartcard.h"
 #endif /* HAL_SMARTCARD_MODULE_ENABLED */
 
 #ifdef HAL_SMBUS_MODULE_ENABLED
- #include "stm32h7xx_hal_smbus.h"
+#include "stm32h7xx_hal_smbus.h"
 #endif /* HAL_SMBUS_MODULE_ENABLED */
 
 #ifdef HAL_WWDG_MODULE_ENABLED
- #include "stm32h7xx_hal_wwdg.h"
+#include "stm32h7xx_hal_wwdg.h"
 #endif /* HAL_WWDG_MODULE_ENABLED */
 
 #ifdef HAL_PCD_MODULE_ENABLED
- #include "stm32h7xx_hal_pcd.h"
+#include "stm32h7xx_hal_pcd.h"
 #endif /* HAL_PCD_MODULE_ENABLED */
 
 #ifdef HAL_HCD_MODULE_ENABLED
- #include "stm32h7xx_hal_hcd.h"
+#include "stm32h7xx_hal_hcd.h"
 #endif /* HAL_HCD_MODULE_ENABLED */
 
 /* Exported macro ------------------------------------------------------------*/
@@ -408,11 +417,12 @@
  *         If expr is true, it returns no value.
  * @retval None
  */
-  #define assert_param(expr) ((expr) ? (void)0U : assert_failed((uint8_t *)__FILE__, __LINE__))
+#define assert_param(expr)                                                     \
+  ((expr) ? (void)0U : assert_failed((uint8_t *)__FILE__, __LINE__))
 /* Exported functions ------------------------------------------------------- */
-  void assert_failed(uint8_t* file, uint32_t line);
+void assert_failed(uint8_t *file, uint32_t line);
 #else
-  #define assert_param(expr) ((void)0U)
+#define assert_param(expr) ((void)0U)
 #endif /* USE_FULL_ASSERT */
 
 #ifdef __cplusplus
@@ -421,5 +431,4 @@
 
 #endif /* __STM32H7xx_HAL_CONF_H */
 
-
 /************************ (C) COPYRIGHT STMicroelectronics *****END OF FILE****/

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/stm32h7xx_it.h

@@ -22,7 +22,7 @@
 #define __STM32H7xx_IT_H
 
 #ifdef __cplusplus
- extern "C" {
+extern "C" {
 #endif
 
 /* Includes ------------------------------------------------------------------*/

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/stm32h7xx_nucleo.h

@@ -19,12 +19,16 @@
  ******************************************************************************
  */
 
+#ifndef USE_HAL_UART_REGISTER_CALLBACKS
+#define USE_HAL_UART_REGISTER_CALLBACKS 1
+#endif
+
 /* Define to prevent recursive inclusion -------------------------------------*/
 #ifndef STM32H7XX_NUCLEO_H
 #define STM32H7XX_NUCLEO_H
 
 #ifdef __cplusplus
- extern "C" {
+extern "C" {
 #endif
 
 /* Includes ------------------------------------------------------------------*/
@@ -32,11 +36,11 @@
 #include "stm32h7xx_nucleo_errno.h"
 
 #if (USE_BSP_COM_FEATURE > 0)
-  #if (USE_COM_LOG > 0)
+#if (USE_COM_LOG > 0)
-    #ifndef __GNUC__
+#ifndef __GNUC__
-      #include <stdio.h>
+#include <stdio.h>
-    #endif
+#endif
-  #endif
+#endif
 #endif
 
 /** @addtogroup BSP
@@ -54,26 +58,29 @@
 /**
  * @brief Define for STM32H7XX_NUCLEO board
  */
-#if !defined (USE_STM32H7XX_NUCLEO)
+#if !defined(USE_STM32H7XX_NUCLEO)
- #define USE_STM32H7XX_NUCLEO
+#define USE_STM32H7XX_NUCLEO
 #endif
 
-#if !defined (USE_NUCLEO_144)
+#if !defined(USE_NUCLEO_144)
- #error "Board Pin number not defined!! Add USE_NUCLEO_144 define within stm32h7xx_nucleo_conf.h file"
+#error                                                                         \
+    "Board Pin number not defined!! Add USE_NUCLEO_144 define within stm32h7xx_nucleo_conf.h file"
 #endif
 
-#if !defined (USE_NUCLEO_H745ZI_Q) && !defined (USE_NUCLEO_H743ZI) && !defined (USE_NUCLEO_H743ZI2) &&\
+#if !defined(USE_NUCLEO_H745ZI_Q) && !defined(USE_NUCLEO_H743ZI) &&            \
-    !defined (USE_NUCLEO_H7A3ZI_Q) && !defined (USE_NUCLEO_H723ZG)
+    !defined(USE_NUCLEO_H743ZI2) && !defined(USE_NUCLEO_H7A3ZI_Q) &&           \
- #error "Board Part number not defined!! Add one of the following define within stm32h7xx_nucleo_conf.h file:\
+    !defined(USE_NUCLEO_H723ZG)
+#error                                                                         \
+    "Board Part number not defined!! Add one of the following define within stm32h7xx_nucleo_conf.h file:\
          USE_NUCLEO_H745ZI_Q, USE_NUCLEO_H743ZI, USE_NUCLEO_H743ZI2, USE_NUCLEO_H7A3ZI_Q, USE_NUCLEO_H723ZG"
 #endif
 
 /** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_Types LOW LEVEL Exported Types
  * @{
  */
-#if defined (USE_NUCLEO_H745ZI_Q) || defined (USE_NUCLEO_H743ZI2) || defined (USE_NUCLEO_H7A3ZI_Q) || defined (USE_NUCLEO_H723ZG)
+#if defined(USE_NUCLEO_H745ZI_Q) || defined(USE_NUCLEO_H743ZI2) ||             \
-typedef enum
+    defined(USE_NUCLEO_H7A3ZI_Q) || defined(USE_NUCLEO_H723ZG)
-{
+typedef enum {
   LED1 = 0,
   LED_GREEN = LED1,
   LED2 = 1,
@@ -81,10 +88,9 @@ typedef enum
   LED3 = 2,
   LED_RED = LED3,
   LEDn
-}Led_TypeDef;
+} Led_TypeDef;
 #else /* USE_NUCLEO_H743ZI */
-typedef enum
+typedef enum {
-{
   LED1 = 0,
   LED_GREEN = LED1,
   LED2 = 1,
@@ -92,71 +98,53 @@ typedef enum
   LED3 = 2,
   LED_RED = LED3,
   LEDn
-}Led_TypeDef;
+} Led_TypeDef;
 #endif
 
-typedef enum
+typedef enum { BUTTON_USER = 0U, BUTTONn } Button_TypeDef;
-{
-  BUTTON_USER = 0U,
-  BUTTONn
-}Button_TypeDef;
 
-typedef enum
+typedef enum { BUTTON_MODE_GPIO = 0, BUTTON_MODE_EXTI = 1 } ButtonMode_TypeDef;
-{
-  BUTTON_MODE_GPIO = 0,
-  BUTTON_MODE_EXTI = 1
-}ButtonMode_TypeDef;
 
 #if (USE_BSP_COM_FEATURE > 0)
-typedef enum
+typedef enum { COM1 = 0U, COMn } COM_TypeDef;
-{
-  COM1 = 0U,
-  COMn
-}COM_TypeDef;
 
-typedef enum
+typedef enum {
-{
   COM_STOPBITS_1 = UART_STOPBITS_1,
   COM_STOPBITS_2 = UART_STOPBITS_2,
-}COM_StopBitsTypeDef;
+} COM_StopBitsTypeDef;
 
-typedef enum
+typedef enum {
-{
   COM_PARITY_NONE = UART_PARITY_NONE,
   COM_PARITY_EVEN = UART_PARITY_EVEN,
   COM_PARITY_ODD = UART_PARITY_ODD,
-}COM_ParityTypeDef;
+} COM_ParityTypeDef;
 
-typedef enum
+typedef enum {
-{
   COM_HWCONTROL_NONE = UART_HWCONTROL_NONE,
   COM_HWCONTROL_RTS = UART_HWCONTROL_RTS,
   COM_HWCONTROL_CTS = UART_HWCONTROL_CTS,
   COM_HWCONTROL_RTS_CTS = UART_HWCONTROL_RTS_CTS,
-}COM_HwFlowCtlTypeDef;
+} COM_HwFlowCtlTypeDef;
 
-typedef enum
+typedef enum {
-{
   COM_WORDLENGTH_7B = UART_WORDLENGTH_7B,
   COM_WORDLENGTH_8B = UART_WORDLENGTH_8B,
   COM_WORDLENGTH_9B = UART_WORDLENGTH_9B,
-}COM_WordLengthTypeDef;
+} COM_WordLengthTypeDef;
 
-typedef struct
+typedef struct {
-{
   uint32_t BaudRate;
   COM_WordLengthTypeDef WordLength;
   COM_StopBitsTypeDef StopBits;
   COM_ParityTypeDef Parity;
   COM_HwFlowCtlTypeDef HwFlowCtl;
-}COM_InitTypeDef;
+} COM_InitTypeDef;
 
 #if (USE_HAL_UART_REGISTER_CALLBACKS == 1)
-typedef struct
+typedef struct {
-{
+  void (*pMspInitCb)(UART_HandleTypeDef *);
-  void (* pMspInitCb)(UART_HandleTypeDef *);
+  void (*pMspDeInitCb)(UART_HandleTypeDef *);
-  void (* pMspDeInitCb)(UART_HandleTypeDef *);
+} BSP_COM_Cb_t;
-}BSP_COM_Cb_t;
 #endif /* (USE_HAL_UART_REGISTER_CALLBACKS == 1) */
 
 #define MX_UART_InitTypeDef COM_InitTypeDef
@@ -166,7 +154,8 @@ typedef struct
  * @}
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_Constants LOW LEVEL Exported Constants
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_Constants LOW LEVEL Exported
+ * Constants
  * @{
  */
 
@@ -176,11 +165,13 @@ typedef struct
 #define STM32H7XX_NUCLEO_BSP_VERSION_MAIN (0x01U) /*!< [31:24] main version */
 #define STM32H7XX_NUCLEO_BSP_VERSION_SUB1 (0x03U) /*!< [23:16] sub1 version */
 #define STM32H7XX_NUCLEO_BSP_VERSION_SUB2 (0x00U) /*!< [15:8]  sub2 version */
-#define STM32H7XX_NUCLEO_BSP_VERSION_RC     (0x00U) /*!< [7:0]  release candidate */
+#define STM32H7XX_NUCLEO_BSP_VERSION_RC (0x00U) /*!< [7:0]  release candidate  \
-#define STM32H7XX_NUCLEO_BSP_VERSION        ((STM32H7XX_NUCLEO_BSP_VERSION_MAIN << 24)\
+                                                 */
-                                            |(STM32H7XX_NUCLEO_BSP_VERSION_SUB1 << 16)\
+#define STM32H7XX_NUCLEO_BSP_VERSION                                           \
-                                            |(STM32H7XX_NUCLEO_BSP_VERSION_SUB2 << 8 )\
+  ((STM32H7XX_NUCLEO_BSP_VERSION_MAIN << 24) |                                 \
-                                            |(STM32H7XX_NUCLEO_BSP_VERSION_RC))
+   (STM32H7XX_NUCLEO_BSP_VERSION_SUB1 << 16) |                                 \
+   (STM32H7XX_NUCLEO_BSP_VERSION_SUB2 << 8) |                                  \
+   (STM32H7XX_NUCLEO_BSP_VERSION_RC))
 
 /** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_LED LOW LEVEL LED
  * @{
@@ -192,7 +183,8 @@ typedef struct
 #define LED1_GPIO_CLK_ENABLE() __HAL_RCC_GPIOB_CLK_ENABLE()
 #define LED1_GPIO_CLK_DISABLE() __HAL_RCC_GPIOB_CLK_DISABLE()
 
-#if defined (USE_NUCLEO_H745ZI_Q) || defined (USE_NUCLEO_H743ZI2) || defined (USE_NUCLEO_H7A3ZI_Q) || defined (USE_NUCLEO_H723ZG)
+#if defined(USE_NUCLEO_H745ZI_Q) || defined(USE_NUCLEO_H743ZI2) ||             \
+    defined(USE_NUCLEO_H7A3ZI_Q) || defined(USE_NUCLEO_H723ZG)
 #define LED2_PIN GPIO_PIN_1
 #define LED2_GPIO_PORT GPIOE
 #define LED2_GPIO_CLK_ENABLE() __HAL_RCC_GPIOE_CLK_ENABLE()
@@ -276,14 +268,15 @@ typedef struct
 extern EXTI_HandleTypeDef hpb_exti[];
 #if (USE_BSP_COM_FEATURE > 0)
 extern UART_HandleTypeDef hcom_uart[];
-extern USART_TypeDef* COM_USART[];
+extern USART_TypeDef *COM_USART[];
 #endif
 
 /**
  * @}
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_FunctionsPrototypes LOW LEVEL Exported Functions Prototypes
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_FunctionsPrototypes LOW LEVEL
+ * Exported Functions Prototypes
  * @{
  */
 int32_t BSP_GetVersion(void);
@@ -292,7 +285,7 @@ int32_t  BSP_LED_DeInit(Led_TypeDef Led);
 int32_t BSP_LED_On(Led_TypeDef Led);
 int32_t BSP_LED_Off(Led_TypeDef Led);
 int32_t BSP_LED_Toggle(Led_TypeDef Led);
-int32_t  BSP_LED_GetState (Led_TypeDef Led);
+int32_t BSP_LED_GetState(Led_TypeDef Led);
 int32_t BSP_PB_Init(Button_TypeDef Button, ButtonMode_TypeDef ButtonMode);
 int32_t BSP_PB_DeInit(Button_TypeDef Button);
 int32_t BSP_PB_GetState(Button_TypeDef Button);
@@ -303,14 +296,15 @@ void     BSP_PB_Callback(Button_TypeDef Button);
 int32_t BSP_COM_Init(COM_TypeDef COM, COM_InitTypeDef *COM_Init);
 int32_t BSP_COM_DeInit(COM_TypeDef COM);
 #if (USE_COM_LOG > 0)
-int32_t  BSP_COM_SelectLogPort (COM_TypeDef COM);
+int32_t BSP_COM_SelectLogPort(COM_TypeDef COM);
 #endif
 
 #if (USE_HAL_UART_REGISTER_CALLBACKS == 1)
 int32_t BSP_COM_RegisterDefaultMspCallbacks(COM_TypeDef COM);
-int32_t BSP_COM_RegisterMspCallbacks(COM_TypeDef COM , BSP_COM_Cb_t *Callback);
+int32_t BSP_COM_RegisterMspCallbacks(COM_TypeDef COM, BSP_COM_Cb_t *Callback);
 #endif /* USE_HAL_UART_REGISTER_CALLBACKS */
-HAL_StatusTypeDef MX_USART3_Init(UART_HandleTypeDef *huart, MX_UART_InitTypeDef *COM_Init);
+HAL_StatusTypeDef MX_USART3_Init(UART_HandleTypeDef *huart,
+                                 MX_UART_InitTypeDef *COM_Init);
 #endif /* USE_BSP_COM_FEATURE */
 
 /**

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/stm32h7xx_nucleo_conf.h

@@ -22,12 +22,12 @@
 #define STM32H7XX_NUCLEO_CONF_H
 
 #ifdef __cplusplus
- extern "C" {
+extern "C" {
 #endif
 
 /* Includes ------------------------------------------------------------------*/
-#include "stm32h7xx_hal.h"
 #include "stm32h743xx.h"
+#include "stm32h7xx_hal.h"
 
 /** @addtogroup BSP
  * @{

bsp_stm32h7_freertos/NUCLEO-H743ZI/Inc/stm32h7xx_nucleo_errno.h

@@ -22,7 +22,7 @@
 #define STM32H7XX_NUCLEO_ERRNO_H
 
 #ifdef __cplusplus
- extern "C" {
+extern "C" {
 #endif
 
 /* Common Error codes */

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/CMakeLists.txt

@@ -1,9 +1,9 @@
-target_sources(${TARGET_NAME} PRIVATE
+target_sources(
-	freertos.c
+  ${TARGET_NAME}
+  PRIVATE freertos.c
           hardware_init.cpp
           stm32h7xx_hal_timebase_tim.c
           stm32h7xx_nucleo.c
           lan8742.c
           stm32h7xx_it.c
-	syscalls.c
+          syscalls.c)
-)

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/freertos.c

@@ -48,13 +48,11 @@
 
 /* Includes ------------------------------------------------------------------*/
 #include "FreeRTOS.h"
-#include "task.h"
 #include "portmacro.h"
+#include "task.h"
 
 /* USER CODE BEGIN Includes */
-void vRequestContextSwitchFromISR() {
+void vRequestContextSwitchFromISR() { portYIELD_FROM_ISR(pdTRUE); }
-	portYIELD_FROM_ISR(pdTRUE);
-}
 
 /* USER CODE END Includes */
 

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/hardware_init.cpp

@@ -6,23 +6,22 @@
 #include "stm32h7xx_nucleo.h"
 
 #if OBSW_ADD_LWIP_COMPONENTS == 1
-#include <app_ethernet.h>
-#include "example_common/stm32h7/networking/ethernetif.h"
 #include "example_common/stm32h7/networking/app_ethernet.h"
-#include <lwip/netif.h>
+#include "example_common/stm32h7/networking/ethernetif.h"
 #include <lwip/init.h>
 #include <lwip/ip_addr.h>
+#include <lwip/netif.h>
 #include <netif/ethernet.h>
 #endif
 
 #include <boardconfig.h>
-#include <stdio.h>
+#include <cstdio>
 
 /* Forward declarations */
-void MPU_Config(void);
+void MPU_Config();
-void SystemClock_Config(void);
+void SystemClock_Config();
-void BSP_Config(void);
+void BSP_Config();
-void CPU_CACHE_Enable(void);
+void CPU_CACHE_Enable();
 void MX_USART3_UART_Init(uint32_t baudRate);
 
 /**
@@ -46,7 +45,7 @@ void performHardwareInit() {
   MX_USART3_UART_Init(DEBUG_UART_BAUDRATE);
 
   HAL_StatusTypeDef retval = HAL_Init();
-	if(retval != HAL_OK) {
+  if (retval != HAL_OK) {
     printf("Error: HAL initialization failed!\n\r");
   }
 
@@ -55,13 +54,12 @@ void performHardwareInit() {
 
   BSP_Config();
 }
-void MX_USART3_UART_Init(uint32_t baudRate)
+void MX_USART3_UART_Init(uint32_t baudRate) {
-{
   __HAL_RCC_USART3_CONFIG(RCC_USART3CLKSOURCE_HSI);
   __HAL_RCC_GPIOD_CLK_ENABLE();
   __HAL_RCC_USART3_CLK_ENABLE();
   /*Configure GPIO pins : PD8 PD9 */
-	gpio_uart_init_struct.Pin = GPIO_PIN_8|GPIO_PIN_9;
+  gpio_uart_init_struct.Pin = GPIO_PIN_8 | GPIO_PIN_9;
   gpio_uart_init_struct.Mode = GPIO_MODE_AF_PP;
   gpio_uart_init_struct.Pull = GPIO_NOPULL;
   gpio_uart_init_struct.Speed = GPIO_SPEED_FREQ_LOW;
@@ -79,19 +77,19 @@ void MX_USART3_UART_Init(uint32_t baudRate)
   huart3.Init.OverSampling = UART_OVERSAMPLING_16;
   huart3.Init.OneBitSampling = UART_ONE_BIT_SAMPLE_DISABLE;
   huart3.Init.ClockPrescaler = UART_PRESCALER_DIV1;
-	//huart3.Init.FIFOMode = UART_FIFOMODE_DISABLE;
+  // huart3.Init.FIFOMode = UART_FIFOMODE_DISABLE;
-	//huart3.Init.TXFIFOThreshold = UART_TXFIFO_THRESHOLD_1_8;
+  // huart3.Init.TXFIFOThreshold = UART_TXFIFO_THRESHOLD_1_8;
-	//huart3.Init.RXFIFOThreshold = UART_RXFIFO_THRESHOLD_1_8;
+  // huart3.Init.RXFIFOThreshold = UART_RXFIFO_THRESHOLD_1_8;
   huart3.AdvancedInit.AdvFeatureInit = UART_ADVFEATURE_NO_INIT;
-	// we can't do error handling (simple print out first) here because UART3 is our print interface
+  // we can't do error handling (simple print out first) here because UART3 is
+  // our print interface
   result = HAL_UART_Init(&huart3);
-	if(result == HAL_OK) {
+  if (result == HAL_OK) {
-		//print_uart3("\rUART3 configured successfully !\r\n\0");
+    // print_uart3("\rUART3 configured successfully !\r\n\0");
     debugAvailable = true;
   }
 }
 
-
 /**
  * @brief  System Clock Configuration
  *         The system Clock is configured as follow :
@@ -114,21 +112,23 @@ void MX_USART3_UART_Init(uint32_t baudRate)
  * @param  None
  * @retval None
  */
-void SystemClock_Config(void)
+void SystemClock_Config() {
-{
   RCC_ClkInitTypeDef RCC_ClkInitStruct;
   RCC_OscInitTypeDef RCC_OscInitStruct;
   HAL_StatusTypeDef ret = HAL_OK;
 
   /*!< Supply configuration update enable */
-	MODIFY_REG(PWR->CR3, PWR_CR3_SCUEN, 0); // @suppress("Field cannot be resolved")
+  MODIFY_REG(PWR->CR3, PWR_CR3_SCUEN,
+             0); // @suppress("Field cannot be resolved")
 
-	/* The voltage scaling allows optimizing the power consumption when the device is
+  /* The voltage scaling allows optimizing the power consumption when the device
-     clocked below the maximum system frequency, to update the voltage scaling value
+is clocked below the maximum system frequency, to update the voltage scaling
-     regarding system frequency refer to product datasheet.  */
+value regarding system frequency refer to product datasheet.  */
-	__HAL_PWR_VOLTAGESCALING_CONFIG(PWR_REGULATOR_VOLTAGE_SCALE1); // @suppress("Field cannot be resolved")
+  __HAL_PWR_VOLTAGESCALING_CONFIG(
+      PWR_REGULATOR_VOLTAGE_SCALE1); // @suppress("Field cannot be resolved")
 
-	while(!__HAL_PWR_GET_FLAG(PWR_FLAG_VOSRDY)) {} // @suppress("Field cannot be resolved")
+  while (!__HAL_PWR_GET_FLAG(PWR_FLAG_VOSRDY)) {
+  } // @suppress("Field cannot be resolved")
 
   /* Enable D2 domain SRAM3 Clock (0x30040000 AXI)*/
   __HAL_RCC_D2SRAM3_CLK_ENABLE(); // @suppress("Field cannot be resolved")
@@ -150,15 +150,15 @@ void SystemClock_Config(void)
   RCC_OscInitStruct.PLL.PLLVCOSEL = RCC_PLL1VCOWIDE;
   RCC_OscInitStruct.PLL.PLLRGE = RCC_PLL1VCIRANGE_2;
   ret = HAL_RCC_OscConfig(&RCC_OscInitStruct);
-	if(ret != HAL_OK)
+  if (ret != HAL_OK) {
-	{
+    while (1)
-		while(1);
+      ;
   }
 
   /* Select PLL as system clock source and configure  bus clocks dividers */
-	RCC_ClkInitStruct.ClockType = (RCC_CLOCKTYPE_SYSCLK | RCC_CLOCKTYPE_HCLK |
+  RCC_ClkInitStruct.ClockType =
-	        RCC_CLOCKTYPE_D1PCLK1 | RCC_CLOCKTYPE_PCLK1 | RCC_CLOCKTYPE_PCLK2  |
+      (RCC_CLOCKTYPE_SYSCLK | RCC_CLOCKTYPE_HCLK | RCC_CLOCKTYPE_D1PCLK1 |
-	        RCC_CLOCKTYPE_D3PCLK1);
+       RCC_CLOCKTYPE_PCLK1 | RCC_CLOCKTYPE_PCLK2 | RCC_CLOCKTYPE_D3PCLK1);
 
   RCC_ClkInitStruct.SYSCLKSource = RCC_SYSCLKSOURCE_PLLCLK;
   RCC_ClkInitStruct.SYSCLKDivider = RCC_SYSCLK_DIV1;
@@ -168,22 +168,21 @@ void SystemClock_Config(void)
   RCC_ClkInitStruct.APB2CLKDivider = RCC_APB2_DIV2;
   RCC_ClkInitStruct.APB4CLKDivider = RCC_APB4_DIV2;
   ret = HAL_RCC_ClockConfig(&RCC_ClkInitStruct, FLASH_LATENCY_4);
-	if(ret != HAL_OK)
+  if (ret != HAL_OK) {
-	{
+    while (1)
-		while(1);
+      ;
   }
 }
 
 /*Configure the MPU attributes */
-void MPU_Config(void)
+void MPU_Config() {
-{
   MPU_Region_InitTypeDef MPU_InitStruct;
 
   /* Disable the MPU */
   HAL_MPU_Disable();
 
   /* Configure the MPU attributes as Device not cacheable
-     for ETH DMA descriptors */
+for ETH DMA descriptors */
   MPU_InitStruct.Enable = MPU_REGION_ENABLE;
   MPU_InitStruct.BaseAddress = 0x30040000;
   MPU_InitStruct.Size = MPU_REGION_SIZE_256B;
@@ -199,7 +198,7 @@ void MPU_Config(void)
   HAL_MPU_ConfigRegion(&MPU_InitStruct);
 
   /* Configure the MPU attributes as Cacheable write through
-     for LwIP RAM heap which contains the Tx buffers */
+for LwIP RAM heap which contains the Tx buffers */
   MPU_InitStruct.Enable = MPU_REGION_ENABLE;
   MPU_InitStruct.BaseAddress = 0x30044000;
   MPU_InitStruct.Size = MPU_REGION_SIZE_16KB;
@@ -219,8 +218,7 @@ void MPU_Config(void)
 }
 
 /*CPU L1-Cache enable*/
-void CPU_CACHE_Enable(void)
+void CPU_CACHE_Enable() {
-{
   /* Enable I-Cache */
   SCB_EnableICache();
 
@@ -228,8 +226,7 @@ void CPU_CACHE_Enable(void)
   SCB_EnableDCache();
 }
 
-void BSP_Config(void)
+void BSP_Config() {
-{
   BSP_LED_Init(LED1);
   BSP_LED_Init(LED2);
   BSP_LED_Init(LED3);
@@ -237,8 +234,7 @@ void BSP_Config(void)
 
 #if OBSW_ADD_LWIP_COMPONENTS == 1
 
-void Netif_Config(void)
+void Netif_Config(void) {
-{
   ip_addr_t ipaddr;
   ip_addr_t netmask;
   ip_addr_t gw;
@@ -254,10 +250,10 @@ void Netif_Config(void)
 #endif
 
   /* add the network interface */
-	struct netif* netif_valid = netif_add(&gnetif, (ip4_addr_t*)&ipaddr,
+  struct netif *netif_valid =
-			(ip4_addr_t*)&netmask, (ip4_addr_t*) &gw, NULL, &ethernetif_init,
+      netif_add(&gnetif, (ip4_addr_t *)&ipaddr, (ip4_addr_t *)&netmask,
-			&ethernet_input);
+                (ip4_addr_t *)&gw, nullptr, &ethernetif_init, &ethernet_input);
-	if(netif_valid == NULL) {
+  if (netif_valid == nullptr) {
     printf("Error: netif initialization failed!\n\r");
     return;
   }
@@ -272,4 +268,3 @@ void Netif_Config(void)
 }
 
 #endif /* OBSW_ADD_LWIP_COMPONENTS == 1 */
-

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/lan8742.c

@@ -60,10 +60,8 @@
  * @retval LAN8742_STATUS_OK  if OK
  *         LAN8742_STATUS_ERROR if missing mandatory function
  */
-int32_t  LAN8742_RegisterBusIO(lan8742_Object_t *pObj, lan8742_IOCtx_t *ioctx)
+int32_t LAN8742_RegisterBusIO(lan8742_Object_t *pObj, lan8742_IOCtx_t *ioctx) {
-{
+  if (!pObj || !ioctx->ReadReg || !ioctx->WriteReg || !ioctx->GetTick) {
-  if(!pObj || !ioctx->ReadReg || !ioctx->WriteReg || !ioctx->GetTick)
-  {
     return LAN8742_STATUS_ERROR;
   }
 
@@ -85,15 +83,12 @@ int32_t  LAN8742_RegisterBusIO(lan8742_Object_t *pObj, lan8742_IOCtx_t *ioctx)
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  *         LAN8742_STATUS_RESET_TIMEOUT if cannot perform a software reset
  */
- int32_t LAN8742_Init(lan8742_Object_t *pObj)
+int32_t LAN8742_Init(lan8742_Object_t *pObj) {
- {
   uint32_t tickstart = 0, regvalue = 0, addr = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-   if(pObj->Is_Initialized == 0)
+  if (pObj->Is_Initialized == 0) {
-   {
+    if (pObj->IO.Init != 0) {
-     if(pObj->IO.Init != 0)
-     {
       /* GPIO and Clocks initialization */
       pObj->IO.Init();
     }
@@ -102,97 +97,76 @@ int32_t  LAN8742_RegisterBusIO(lan8742_Object_t *pObj, lan8742_IOCtx_t *ioctx)
     pObj->DevAddr = LAN8742_MAX_DEV_ADDR + 1;
 
     /* Get the device address from special mode register */
-     for(addr = 0; addr <= LAN8742_MAX_DEV_ADDR; addr ++)
+    for (addr = 0; addr <= LAN8742_MAX_DEV_ADDR; addr++) {
-     {
+      if (pObj->IO.ReadReg(addr, LAN8742_SMR, &regvalue) < 0) {
-       if(pObj->IO.ReadReg(addr, LAN8742_SMR, &regvalue) < 0)
-       { 
         status = LAN8742_STATUS_READ_ERROR;
         /* Can't read from this device address
            continue with next address */
         continue;
       }
 
-       if((regvalue & LAN8742_SMR_PHY_ADDR) == addr)
+      if ((regvalue & LAN8742_SMR_PHY_ADDR) == addr) {
-       {
         pObj->DevAddr = addr;
         status = LAN8742_STATUS_OK;
         break;
       }
     }
 
-     if(pObj->DevAddr > LAN8742_MAX_DEV_ADDR)
+    if (pObj->DevAddr > LAN8742_MAX_DEV_ADDR) {
-     {
       status = LAN8742_STATUS_ADDRESS_ERROR;
     }
 
     /* if device address is matched */
-     if(status == LAN8742_STATUS_OK)
+    if (status == LAN8742_STATUS_OK) {
-     {
       /* set a software reset  */
-       if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, LAN8742_BCR_SOFT_RESET) >= 0)
+      if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR,
-       { 
+                            LAN8742_BCR_SOFT_RESET) >= 0) {
         /* get software reset status */
-         if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &regvalue) >= 0)
+        if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &regvalue) >= 0) {
-         { 
           tickstart = pObj->IO.GetTick();
 
           /* wait until software reset is done or timeout occured  */
-           while(regvalue & LAN8742_BCR_SOFT_RESET)
+          while (regvalue & LAN8742_BCR_SOFT_RESET) {
-           {
+            if ((pObj->IO.GetTick() - tickstart) <= LAN8742_SW_RESET_TO) {
-             if((pObj->IO.GetTick() - tickstart) <= LAN8742_SW_RESET_TO)
+              if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &regvalue) < 0) {
-             {
-               if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &regvalue) < 0)
-               { 
                 status = LAN8742_STATUS_READ_ERROR;
                 break;
               }
-             }
+            } else {
-             else
-             {
               status = LAN8742_STATUS_RESET_TIMEOUT;
               break;
             }
           }
-         }
+        } else {
-         else
-         {
           status = LAN8742_STATUS_READ_ERROR;
         }
-       }
+      } else {
-       else
-       {
         status = LAN8742_STATUS_WRITE_ERROR;
       }
     }
   }
 
-   if(status == LAN8742_STATUS_OK)
+  if (status == LAN8742_STATUS_OK) {
-   {
     tickstart = pObj->IO.GetTick();
 
     /* Wait for 2s to perform initialization */
-     while((pObj->IO.GetTick() - tickstart) <= LAN8742_INIT_TO)
+    while ((pObj->IO.GetTick() - tickstart) <= LAN8742_INIT_TO) {
-     {
     }
     pObj->Is_Initialized = 1;
   }
 
   return status;
- }
+}
 
 /**
  * @brief  De-Initialize the lan8742 and it's hardware resources
  * @param  pObj: device object LAN8742_Object_t.
  * @retval None
  */
-int32_t LAN8742_DeInit(lan8742_Object_t *pObj)
+int32_t LAN8742_DeInit(lan8742_Object_t *pObj) {
-{
+  if (pObj->Is_Initialized) {
-  if(pObj->Is_Initialized)
+    if (pObj->IO.DeInit != 0) {
-  {
+      if (pObj->IO.DeInit() < 0) {
-    if(pObj->IO.DeInit != 0)
-    {
-      if(pObj->IO.DeInit() < 0)
-      {
         return LAN8742_STATUS_ERROR;
       }
     }
@@ -210,23 +184,18 @@ int32_t LAN8742_DeInit(lan8742_Object_t *pObj)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_DisablePowerDownMode(lan8742_Object_t *pObj)
+int32_t LAN8742_DisablePowerDownMode(lan8742_Object_t *pObj) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0) {
-  {
     readval &= ~LAN8742_BCR_POWER_DOWN;
 
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -240,23 +209,18 @@ int32_t LAN8742_DisablePowerDownMode(lan8742_Object_t *pObj)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_EnablePowerDownMode(lan8742_Object_t *pObj)
+int32_t LAN8742_EnablePowerDownMode(lan8742_Object_t *pObj) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0) {
-  {
     readval |= LAN8742_BCR_POWER_DOWN;
 
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -270,23 +234,18 @@ int32_t LAN8742_EnablePowerDownMode(lan8742_Object_t *pObj)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_StartAutoNego(lan8742_Object_t *pObj)
+int32_t LAN8742_StartAutoNego(lan8742_Object_t *pObj) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0) {
-  {
     readval |= LAN8742_BCR_AUTONEGO_EN;
 
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -306,80 +265,61 @@ int32_t LAN8742_StartAutoNego(lan8742_Object_t *pObj)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_GetLinkState(lan8742_Object_t *pObj)
+int32_t LAN8742_GetLinkState(lan8742_Object_t *pObj) {
-{
   uint32_t readval = 0;
 
   /* Read Status register  */
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BSR, &readval) < 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BSR, &readval) < 0) {
-  {
     return LAN8742_STATUS_READ_ERROR;
   }
 
   /* Read Status register again */
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BSR, &readval) < 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BSR, &readval) < 0) {
-  {
     return LAN8742_STATUS_READ_ERROR;
   }
 
-  if((readval & LAN8742_BSR_LINK_STATUS) == 0)
+  if ((readval & LAN8742_BSR_LINK_STATUS) == 0) {
-  {
     /* Return Link Down status */
     return LAN8742_STATUS_LINK_DOWN;
   }
 
   /* Check Auto negotiaition */
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) < 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) < 0) {
-  {
     return LAN8742_STATUS_READ_ERROR;
   }
 
-  if((readval & LAN8742_BCR_AUTONEGO_EN) != LAN8742_BCR_AUTONEGO_EN)
+  if ((readval & LAN8742_BCR_AUTONEGO_EN) != LAN8742_BCR_AUTONEGO_EN) {
-  {
+    if (((readval & LAN8742_BCR_SPEED_SELECT) == LAN8742_BCR_SPEED_SELECT) &&
-    if(((readval & LAN8742_BCR_SPEED_SELECT) == LAN8742_BCR_SPEED_SELECT) && ((readval & LAN8742_BCR_DUPLEX_MODE) == LAN8742_BCR_DUPLEX_MODE)) 
+        ((readval & LAN8742_BCR_DUPLEX_MODE) == LAN8742_BCR_DUPLEX_MODE)) {
-    {
       return LAN8742_STATUS_100MBITS_FULLDUPLEX;
-    }
+    } else if ((readval & LAN8742_BCR_SPEED_SELECT) ==
-    else if ((readval & LAN8742_BCR_SPEED_SELECT) == LAN8742_BCR_SPEED_SELECT)
+               LAN8742_BCR_SPEED_SELECT) {
-    {
       return LAN8742_STATUS_100MBITS_HALFDUPLEX;
-    }        
+    } else if ((readval & LAN8742_BCR_DUPLEX_MODE) == LAN8742_BCR_DUPLEX_MODE) {
-    else if ((readval & LAN8742_BCR_DUPLEX_MODE) == LAN8742_BCR_DUPLEX_MODE)
-    {
       return LAN8742_STATUS_10MBITS_FULLDUPLEX;
-    }
+    } else {
-    else
-    {
       return LAN8742_STATUS_10MBITS_HALFDUPLEX;
     }
-  }
+  } else /* Auto Nego enabled */
-  else /* Auto Nego enabled */
-  {
-    if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_PHYSCSR, &readval) < 0)
   {
+    if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_PHYSCSR, &readval) < 0) {
       return LAN8742_STATUS_READ_ERROR;
     }
 
     /* Check if auto nego not done */
-    if((readval & LAN8742_PHYSCSR_AUTONEGO_DONE) == 0)
+    if ((readval & LAN8742_PHYSCSR_AUTONEGO_DONE) == 0) {
-    {
       return LAN8742_STATUS_AUTONEGO_NOTDONE;
     }
 
-    if((readval & LAN8742_PHYSCSR_HCDSPEEDMASK) == LAN8742_PHYSCSR_100BTX_FD)
+    if ((readval & LAN8742_PHYSCSR_HCDSPEEDMASK) == LAN8742_PHYSCSR_100BTX_FD) {
-    {
       return LAN8742_STATUS_100MBITS_FULLDUPLEX;
-    }
+    } else if ((readval & LAN8742_PHYSCSR_HCDSPEEDMASK) ==
-    else if ((readval & LAN8742_PHYSCSR_HCDSPEEDMASK) == LAN8742_PHYSCSR_100BTX_HD)
+               LAN8742_PHYSCSR_100BTX_HD) {
-    {
       return LAN8742_STATUS_100MBITS_HALFDUPLEX;
-    }
+    } else if ((readval & LAN8742_PHYSCSR_HCDSPEEDMASK) ==
-    else if ((readval & LAN8742_PHYSCSR_HCDSPEEDMASK) == LAN8742_PHYSCSR_10BT_FD)
+               LAN8742_PHYSCSR_10BT_FD) {
-    {
       return LAN8742_STATUS_10MBITS_FULLDUPLEX;
-    }
+    } else {
-    else
-    {
       return LAN8742_STATUS_10MBITS_HALFDUPLEX;
     }
   }
@@ -398,44 +338,32 @@ int32_t LAN8742_GetLinkState(lan8742_Object_t *pObj)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_SetLinkState(lan8742_Object_t *pObj, uint32_t LinkState)
+int32_t LAN8742_SetLinkState(lan8742_Object_t *pObj, uint32_t LinkState) {
-{
   uint32_t bcrvalue = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &bcrvalue) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &bcrvalue) >= 0) {
-  {
     /* Disable link config (Auto nego, speed and duplex) */
-    bcrvalue &= ~(LAN8742_BCR_AUTONEGO_EN | LAN8742_BCR_SPEED_SELECT | LAN8742_BCR_DUPLEX_MODE);
+    bcrvalue &= ~(LAN8742_BCR_AUTONEGO_EN | LAN8742_BCR_SPEED_SELECT |
+                  LAN8742_BCR_DUPLEX_MODE);
 
-    if(LinkState == LAN8742_STATUS_100MBITS_FULLDUPLEX)
+    if (LinkState == LAN8742_STATUS_100MBITS_FULLDUPLEX) {
-    {
       bcrvalue |= (LAN8742_BCR_SPEED_SELECT | LAN8742_BCR_DUPLEX_MODE);
-    }
+    } else if (LinkState == LAN8742_STATUS_100MBITS_HALFDUPLEX) {
-    else if (LinkState == LAN8742_STATUS_100MBITS_HALFDUPLEX)
-    {
       bcrvalue |= LAN8742_BCR_SPEED_SELECT;
-    }
+    } else if (LinkState == LAN8742_STATUS_10MBITS_FULLDUPLEX) {
-    else if (LinkState == LAN8742_STATUS_10MBITS_FULLDUPLEX)
-    {
       bcrvalue |= LAN8742_BCR_DUPLEX_MODE;
-    }
+    } else {
-    else
-    {
       /* Wrong link status parameter */
       status = LAN8742_STATUS_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
-  if(status == LAN8742_STATUS_OK)
+  if (status == LAN8742_STATUS_OK) {
-  {
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, bcrvalue) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, bcrvalue) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
   }
@@ -450,23 +378,18 @@ int32_t LAN8742_SetLinkState(lan8742_Object_t *pObj, uint32_t LinkState)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_EnableLoopbackMode(lan8742_Object_t *pObj)
+int32_t LAN8742_EnableLoopbackMode(lan8742_Object_t *pObj) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0) {
-  {
     readval |= LAN8742_BCR_LOOPBACK;
 
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -480,23 +403,18 @@ int32_t LAN8742_EnableLoopbackMode(lan8742_Object_t *pObj)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_DisableLoopbackMode(lan8742_Object_t *pObj)
+int32_t LAN8742_DisableLoopbackMode(lan8742_Object_t *pObj) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_BCR, &readval) >= 0) {
-  {
     readval &= ~LAN8742_BCR_LOOPBACK;
 
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_BCR, readval) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -520,23 +438,18 @@ int32_t LAN8742_DisableLoopbackMode(lan8742_Object_t *pObj)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_EnableIT(lan8742_Object_t *pObj, uint32_t Interrupt)
+int32_t LAN8742_EnableIT(lan8742_Object_t *pObj, uint32_t Interrupt) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_IMR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_IMR, &readval) >= 0) {
-  {
     readval |= Interrupt;
 
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_IMR, readval) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_IMR, readval) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -560,23 +473,18 @@ int32_t LAN8742_EnableIT(lan8742_Object_t *pObj, uint32_t Interrupt)
  *         LAN8742_STATUS_READ_ERROR if connot read register
  *         LAN8742_STATUS_WRITE_ERROR if connot write to register
  */
-int32_t LAN8742_DisableIT(lan8742_Object_t *pObj, uint32_t Interrupt)
+int32_t LAN8742_DisableIT(lan8742_Object_t *pObj, uint32_t Interrupt) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_IMR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_IMR, &readval) >= 0) {
-  {
     readval &= ~Interrupt;
 
     /* Apply configuration */
-    if(pObj->IO.WriteReg(pObj->DevAddr, LAN8742_IMR, readval) < 0)
+    if (pObj->IO.WriteReg(pObj->DevAddr, LAN8742_IMR, readval) < 0) {
-    {
       status = LAN8742_STATUS_WRITE_ERROR;
     }
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -599,13 +507,11 @@ int32_t LAN8742_DisableIT(lan8742_Object_t *pObj, uint32_t Interrupt)
  * @retval LAN8742_STATUS_OK  if OK
  *         LAN8742_STATUS_READ_ERROR if connot read register
  */
-int32_t  LAN8742_ClearIT(lan8742_Object_t *pObj, uint32_t Interrupt)
+int32_t LAN8742_ClearIT(lan8742_Object_t *pObj, uint32_t Interrupt) {
-{
   uint32_t readval = 0;
   int32_t status = LAN8742_STATUS_OK;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_ISFR, &readval) < 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_ISFR, &readval) < 0) {
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 
@@ -629,17 +535,13 @@ int32_t  LAN8742_ClearIT(lan8742_Object_t *pObj, uint32_t Interrupt)
  *         0 IT flag is RESET
  *         LAN8742_STATUS_READ_ERROR if connot read register
  */
-int32_t LAN8742_GetITStatus(lan8742_Object_t *pObj, uint32_t Interrupt)
+int32_t LAN8742_GetITStatus(lan8742_Object_t *pObj, uint32_t Interrupt) {
-{
   uint32_t readval = 0;
   int32_t status = 0;
 
-  if(pObj->IO.ReadReg(pObj->DevAddr, LAN8742_ISFR, &readval) >= 0)
+  if (pObj->IO.ReadReg(pObj->DevAddr, LAN8742_ISFR, &readval) >= 0) {
-  {
     status = ((readval & Interrupt) == Interrupt);
-  }
+  } else {
-  else
-  {
     status = LAN8742_STATUS_READ_ERROR;
   }
 

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/stm32h7xx_hal_timebase_tim.c

@@ -4,37 +4,40 @@
  * @author  MCD Application Team
  * @brief   HAL time base based on the hardware TIM.
  *
-  *          This file overrides the native HAL time base functions (defined as weak)
+ *          This file overrides the native HAL time base functions (defined as
-  *          the TIM time base:
+ *weak) the TIM time base:
-  *           + Intializes the TIM peripheral generate a Period elapsed Event each 1ms
+ *           + Intializes the TIM peripheral generate a Period elapsed Event
-  *           + HAL_IncTick is called inside HAL_TIM_PeriodElapsedCallback ie each 1ms
+ *each 1ms
+ *           + HAL_IncTick is called inside HAL_TIM_PeriodElapsedCallback ie
+ *each 1ms
  *
  ******************************************************************************
  * @attention
  *
  * <h2><center>&copy; COPYRIGHT(c) 2017 STMicroelectronics</center></h2>
  *
-  * Redistribution and use in source and binary forms, with or without modification,
+ * Redistribution and use in source and binary forms, with or without
-  * are permitted provided that the following conditions are met:
+ *modification, are permitted provided that the following conditions are met:
  *   1. Redistributions of source code must retain the above copyright notice,
  *      this list of conditions and the following disclaimer.
-  *   2. Redistributions in binary form must reproduce the above copyright notice,
+ *   2. Redistributions in binary form must reproduce the above copyright
-  *      this list of conditions and the following disclaimer in the documentation
+ *notice, this list of conditions and the following disclaimer in the
-  *      and/or other materials provided with the distribution.
+ *documentation and/or other materials provided with the distribution.
  *   3. Neither the name of STMicroelectronics nor the names of its contributors
  *      may be used to endorse or promote products derived from this software
  *      without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
  * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
-  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
-  * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+ *ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
-  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ *LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
-  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ *CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
-  * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ *SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-  * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ *INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
-  * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ *CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
-  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ *POSSIBILITY OF SUCH DAMAGE.
  *
  ******************************************************************************
  */
@@ -55,20 +58,20 @@ void TIM6_DAC_IRQHandler(void);
  * @brief  This function configures the TIM6 as a time base source.
  *         The time source is configured to have 1ms time base with a dedicated
  *         Tick interrupt priority.
-  * @note   This function is called  automatically at the beginning of program after
+ * @note   This function is called  automatically at the beginning of program
-  *         reset by HAL_Init() or at any time when clock is configured, by HAL_RCC_ClockConfig(). 
+ * after reset by HAL_Init() or at any time when clock is configured, by
+ * HAL_RCC_ClockConfig().
  * @param  TickPriority: Tick interrupt priority.
  * @retval HAL status
  */
-HAL_StatusTypeDef HAL_InitTick (uint32_t TickPriority)
+HAL_StatusTypeDef HAL_InitTick(uint32_t TickPriority) {
-{
   RCC_ClkInitTypeDef clkconfig;
   uint32_t uwTimclock, uwAPB1Prescaler = 0U;
   uint32_t uwPrescalerValue = 0U;
   uint32_t pFLatency;
 
   /*Configure the TIM6 IRQ priority */
-  HAL_NVIC_SetPriority(TIM6_DAC_IRQn, TickPriority ,0U);
+  HAL_NVIC_SetPriority(TIM6_DAC_IRQn, TickPriority, 0U);
 
   /* Enable the TIM6 global Interrupt */
   HAL_NVIC_EnableIRQ(TIM6_DAC_IRQn);
@@ -83,17 +86,14 @@ HAL_StatusTypeDef HAL_InitTick (uint32_t TickPriority)
   uwAPB1Prescaler = clkconfig.APB1CLKDivider;
 
   /* Compute TIM6 clock */
-  if (uwAPB1Prescaler == RCC_HCLK_DIV1) 
+  if (uwAPB1Prescaler == RCC_HCLK_DIV1) {
-  {
     uwTimclock = HAL_RCC_GetPCLK1Freq();
-  }
+  } else {
-  else
+    uwTimclock = 2 * HAL_RCC_GetPCLK1Freq();
-  {
-    uwTimclock = 2*HAL_RCC_GetPCLK1Freq();
   }
 
   /* Compute the prescaler value to have TIM6 counter clock equal to 1MHz */
-  uwPrescalerValue = (uint32_t) ((uwTimclock / 1000000U) - 1U);
+  uwPrescalerValue = (uint32_t)((uwTimclock / 1000000U) - 1U);
 
   /* Initialize TIM6 */
   TimHandle.Instance = TIM6;
@@ -108,8 +108,7 @@ HAL_StatusTypeDef HAL_InitTick (uint32_t TickPriority)
   TimHandle.Init.Prescaler = uwPrescalerValue;
   TimHandle.Init.ClockDivision = 0;
   TimHandle.Init.CounterMode = TIM_COUNTERMODE_UP;
-  if(HAL_TIM_Base_Init(&TimHandle) == HAL_OK)
+  if (HAL_TIM_Base_Init(&TimHandle) == HAL_OK) {
-  {
     /* Start the TIM time Base generation in interrupt mode */
     return HAL_TIM_Base_Start_IT(&TimHandle);
   }
@@ -124,8 +123,7 @@ HAL_StatusTypeDef HAL_InitTick (uint32_t TickPriority)
  * @param  None
  * @retval None
  */
-void HAL_SuspendTick(void)
+void HAL_SuspendTick(void) {
-{
   /* Disable TIM6 update Interrupt */
   __HAL_TIM_DISABLE_IT(&TimHandle, TIM_IT_UPDATE);
 }
@@ -136,8 +134,7 @@ void HAL_SuspendTick(void)
  * @param  None
  * @retval None
  */
-void HAL_ResumeTick(void)
+void HAL_ResumeTick(void) {
-{
   /* Enable TIM6 Update interrupt */
   __HAL_TIM_ENABLE_IT(&TimHandle, TIM_IT_UPDATE);
 }
@@ -150,19 +147,13 @@ void HAL_ResumeTick(void)
  * @param  htim : TIM handle
  * @retval None
  */
-void HAL_TIM_PeriodElapsedCallback(TIM_HandleTypeDef *htim)
+void HAL_TIM_PeriodElapsedCallback(TIM_HandleTypeDef *htim) { HAL_IncTick(); }
-{
-  HAL_IncTick();
-}
 
 /**
  * @brief  This function handles TIM interrupt request.
  * @param  None
  * @retval None
  */
-void TIM6_DAC_IRQHandler(void)
+void TIM6_DAC_IRQHandler(void) { HAL_TIM_IRQHandler(&TimHandle); }
-{
-  HAL_TIM_IRQHandler(&TimHandle);
-}
 
 /************************ (C) COPYRIGHT STMicroelectronics *****END OF FILE****/

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/stm32h7xx_it.c

@@ -8,34 +8,35 @@
  *
  * <h2><center>&copy; COPYRIGHT(c) 2017 STMicroelectronics</center></h2>
  *
-  * Redistribution and use in source and binary forms, with or without modification,
+ * Redistribution and use in source and binary forms, with or without
-  * are permitted provided that the following conditions are met:
+ *modification, are permitted provided that the following conditions are met:
  *   1. Redistributions of source code must retain the above copyright notice,
  *      this list of conditions and the following disclaimer.
-  *   2. Redistributions in binary form must reproduce the above copyright notice,
+ *   2. Redistributions in binary form must reproduce the above copyright
-  *      this list of conditions and the following disclaimer in the documentation
+ *notice, this list of conditions and the following disclaimer in the
-  *      and/or other materials provided with the distribution.
+ *documentation and/or other materials provided with the distribution.
  *   3. Neither the name of STMicroelectronics nor the names of its contributors
  *      may be used to endorse or promote products derived from this software
  *      without specific prior written permission.
  *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
  * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
-  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
-  * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+ *ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
-  * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ *LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
-  * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ *CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
-  * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ *SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
-  * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ *INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
-  * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ *CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
-  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ *POSSIBILITY OF SUCH DAMAGE.
  *
  ******************************************************************************
  */
 
 /* Includes ------------------------------------------------------------------*/
-#include "OBSWConfig.h"
 #include "stm32h7xx_it.h"
+#include "OBSWConfig.h"
 
 #include "cmsis_os.h"
 #include "main.h"
@@ -61,20 +62,16 @@ void ETH_IRQHandler(void);
  * @param  None
  * @retval None
  */
-void NMI_Handler(void)
+void NMI_Handler(void) {}
-{
-}
 
 /**
  * @brief  This function handles Hard Fault exception.
  * @param  None
  * @retval None
  */
-void HardFault_Handler(void)
+void HardFault_Handler(void) {
-{
   /* Go to infinite loop when Hard Fault exception occurs */
-  while (1)
+  while (1) {
-  {
   }
 }
 
@@ -83,11 +80,9 @@ void HardFault_Handler(void)
  * @param  None
  * @retval None
  */
-void MemManage_Handler(void)
+void MemManage_Handler(void) {
-{
   /* Go to infinite loop when Memory Manage exception occurs */
-  while (1)
+  while (1) {
-  {
   }
 }
 
@@ -96,11 +91,9 @@ void MemManage_Handler(void)
  * @param  None
  * @retval None
  */
-void BusFault_Handler(void)
+void BusFault_Handler(void) {
-{
   /* Go to infinite loop when Bus Fault exception occurs */
-  while (1)
+  while (1) {
-  {
   }
 }
 
@@ -109,34 +102,25 @@ void BusFault_Handler(void)
  * @param  None
  * @retval None
  */
-void UsageFault_Handler(void)
+void UsageFault_Handler(void) {
-{
   /* Go to infinite loop when Usage Fault exception occurs */
-  while (1)
+  while (1) {
-  {
   }
 }
 
-
 /**
  * @brief  This function handles Debug Monitor exception.
  * @param  None
  * @retval None
  */
-void DebugMon_Handler(void)
+void DebugMon_Handler(void) {}
-{
-}
 
 /**
  * @brief  This function handles SysTick Handler.
  * @param  None
  * @retval None
  */
-void SysTick_Handler(void)
+void SysTick_Handler(void) { osSystickHandler(); }
-{
-  osSystickHandler();
-}
-  
 
 /******************************************************************************/
 /*                 STM32H7xx Peripherals Interrupt Handlers                   */
@@ -149,14 +133,12 @@ void SysTick_Handler(void)
  * @param  None
  * @retval None
  */
-void ETH_IRQHandler(void)
+void ETH_IRQHandler(void) {
-{
 #if OBSW_ADD_LWIP_COMPONENTS == 1
   HAL_ETH_IRQHandler(&EthHandle);
 #endif
 }
 
-
 /**
  * @}
  */

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/stm32h7xx_nucleo.c

@@ -31,12 +31,13 @@
  */
 
 /** @addtogroup STM32H7XX_NUCLEO_LOW_LEVEL
-  * @brief This file provides set of firmware functions to manage Leds and push-button
+ * @brief This file provides set of firmware functions to manage Leds and
-  *        available on STM32H7xx-Nucleo Kit from STMicroelectronics.
+ * push-button available on STM32H7xx-Nucleo Kit from STMicroelectronics.
  * @{
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_Defines LOW LEVEL Private Defines
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_Defines LOW LEVEL Private
+ * Defines
  * @{
  */
 
@@ -44,56 +45,55 @@
  * @}
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_TypesDefinitions LOW LEVEL Private Typedef
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_TypesDefinitions LOW LEVEL
+ * Private Typedef
  * @{
  */
-typedef void (* BSP_EXTI_LineCallback) (void);
+typedef void (*BSP_EXTI_LineCallback)(void);
 /**
  * @}
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_Variables LOW LEVEL Exported Variables
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_Variables LOW LEVEL Exported
+ * Variables
  * @{
  */
 EXTI_HandleTypeDef hpb_exti[BUTTONn];
 #if (USE_BSP_COM_FEATURE > 0)
 UART_HandleTypeDef hcom_uart[COMn];
-USART_TypeDef* COM_USART[COMn]   = {COM1_UART};
+USART_TypeDef *COM_USART[COMn] = {COM1_UART};
 #endif
 /**
  * @}
  */
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_Variables LOW LEVEL Private Variables
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_Variables LOW LEVEL Private
+ * Variables
  * @{
  */
-static GPIO_TypeDef* LED_PORT[LEDn] = {LED1_GPIO_PORT,
+static GPIO_TypeDef *LED_PORT[LEDn] = {LED1_GPIO_PORT, LED2_GPIO_PORT,
-                                       LED2_GPIO_PORT,
                                        LED3_GPIO_PORT};
 
-static const uint16_t LED_PIN[LEDn] = {LED1_PIN,
+static const uint16_t LED_PIN[LEDn] = {LED1_PIN, LED2_PIN, LED3_PIN};
-                                       LED2_PIN,
-                                       LED3_PIN};
 
-static GPIO_TypeDef* BUTTON_PORT[BUTTONn] = {BUTTON_USER_GPIO_PORT};
+static GPIO_TypeDef *BUTTON_PORT[BUTTONn] = {BUTTON_USER_GPIO_PORT};
 static const uint16_t BUTTON_PIN[BUTTONn] = {BUTTON_USER_PIN};
 static const IRQn_Type BUTTON_IRQn[BUTTONn] = {BUTTON_USER_EXTI_IRQn};
 
-
-
 #if (USE_BSP_COM_FEATURE > 0)
-  #if (USE_COM_LOG > 0)
+#if (USE_COM_LOG > 0)
-     static COM_TypeDef COM_ActiveLogPort = COM1;
+static COM_TypeDef COM_ActiveLogPort = COM1;
-  #endif
+#endif
-  #if (USE_HAL_UART_REGISTER_CALLBACKS == 1)
+#if (USE_HAL_UART_REGISTER_CALLBACKS == 1)
-     static uint32_t IsComMspCbValid[COMn] = {0};
+static uint32_t IsComMspCbValid[COMn] = {0};
-  #endif
+#endif
 #endif
 
 /**
  * @}
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_FunctionPrototypes LOW LEVEL Private functions Prototypes
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_FunctionPrototypes LOW LEVEL
+ * Private functions Prototypes
  * @{
  */
 static void BUTTON_USER_EXTI_Callback(void);
@@ -105,7 +105,8 @@ static void COM1_MspDeInit(UART_HandleTypeDef *huart);
  * @}
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_Functions LOW LEVEL Exported Functions
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Exported_Functions LOW LEVEL Exported
+ * Functions
  * @{
  */
 
@@ -113,10 +114,7 @@ static void COM1_MspDeInit(UART_HandleTypeDef *huart);
  * @brief  This method returns the STM32H7XX NUCLEO BSP Driver revision
  * @retval version: 0xXYZR (8bits for each decimal, R for RC)
  */
-int32_t BSP_GetVersion(void)
+int32_t BSP_GetVersion(void) { return (int32_t)STM32H7XX_NUCLEO_BSP_VERSION; }
-{
-  return (int32_t)STM32H7XX_NUCLEO_BSP_VERSION;
-}
 
 /**
  * @brief  Configures LED GPIO.
@@ -127,28 +125,19 @@ int32_t BSP_GetVersion(void)
  *     @arg  LED3
  * @retval BSP status
  */
-int32_t BSP_LED_Init(Led_TypeDef Led)
+int32_t BSP_LED_Init(Led_TypeDef Led) {
-{
   int32_t ret = BSP_ERROR_NONE;
   GPIO_InitTypeDef gpio_init_structure;
 
-  if((Led != LED1) && (Led != LED2) && (Led != LED3))
+  if ((Led != LED1) && (Led != LED2) && (Led != LED3)) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     /* Enable the GPIO LED Clock */
-    if(Led == LED1)
+    if (Led == LED1) {
-    {
       LED1_GPIO_CLK_ENABLE();
-    }
+    } else if (Led == LED2) {
-    else if(Led == LED2)
-    {
       LED2_GPIO_CLK_ENABLE();
-    }
+    } else {
-    else
-    {
       LED3_GPIO_CLK_ENABLE();
     }
     /* Configure the GPIO_LED pin */
@@ -174,17 +163,13 @@ int32_t BSP_LED_Init(Led_TypeDef Led)
  * @note Led DeInit does not disable the GPIO clock nor disable the Mfx
  * @retval BSP status
  */
-int32_t BSP_LED_DeInit(Led_TypeDef Led)
+int32_t BSP_LED_DeInit(Led_TypeDef Led) {
-{
   int32_t ret = BSP_ERROR_NONE;
   GPIO_InitTypeDef gpio_init_structure;
 
-  if((Led != LED1) && (Led != LED2) && (Led != LED3))
+  if ((Led != LED1) && (Led != LED2) && (Led != LED3)) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     /* Turn off LED */
     HAL_GPIO_WritePin(LED_PORT[Led], LED_PIN[Led], GPIO_PIN_RESET);
     /* DeInit the GPIO_LED pin */
@@ -204,16 +189,12 @@ int32_t BSP_LED_DeInit(Led_TypeDef Led)
  *     @arg  LED3
  * @retval BSP status
  */
-int32_t BSP_LED_On(Led_TypeDef Led)
+int32_t BSP_LED_On(Led_TypeDef Led) {
-{
   int32_t ret = BSP_ERROR_NONE;
 
-  if((Led != LED1) && (Led != LED2) && (Led != LED3))
+  if ((Led != LED1) && (Led != LED2) && (Led != LED3)) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     HAL_GPIO_WritePin(LED_PORT[Led], LED_PIN[Led], GPIO_PIN_SET);
   }
 
@@ -229,16 +210,12 @@ int32_t BSP_LED_On(Led_TypeDef Led)
  *     @arg  LED3
  * @retval BSP status
  */
-int32_t BSP_LED_Off(Led_TypeDef Led)
+int32_t BSP_LED_Off(Led_TypeDef Led) {
-{
   int32_t ret = BSP_ERROR_NONE;
 
-  if((Led != LED1) && (Led != LED2) && (Led != LED3))
+  if ((Led != LED1) && (Led != LED2) && (Led != LED3)) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     HAL_GPIO_WritePin(LED_PORT[Led], LED_PIN[Led], GPIO_PIN_RESET);
   }
 
@@ -254,16 +231,12 @@ int32_t BSP_LED_Off(Led_TypeDef Led)
  *     @arg  LED3
  * @retval BSP status
  */
-int32_t BSP_LED_Toggle(Led_TypeDef Led)
+int32_t BSP_LED_Toggle(Led_TypeDef Led) {
-{
   int32_t ret = BSP_ERROR_NONE;
 
-  if((Led != LED1) && (Led != LED2) && (Led != LED3))
+  if ((Led != LED1) && (Led != LED2) && (Led != LED3)) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     HAL_GPIO_TogglePin(LED_PORT[Led], LED_PIN[Led]);
   }
 
@@ -279,17 +252,13 @@ int32_t BSP_LED_Toggle(Led_TypeDef Led)
  *     @arg  LED3
  * @retval LED status
  */
-int32_t BSP_LED_GetState (Led_TypeDef Led)
+int32_t BSP_LED_GetState(Led_TypeDef Led) {
-{
   int32_t ret;
 
-  if((Led != LED1) && (Led != LED2) && (Led != LED3))
+  if ((Led != LED1) && (Led != LED2) && (Led != LED3)) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
+    ret = (int32_t)HAL_GPIO_ReadPin(LED_PORT[Led], LED_PIN[Led]);
-  {
-    ret = (int32_t)HAL_GPIO_ReadPin (LED_PORT [Led], LED_PIN [Led]);
   }
 
   return ret;
@@ -306,27 +275,25 @@ int32_t BSP_LED_GetState (Led_TypeDef Led)
  *            @arg  BUTTON_MODE_EXTI: Button will be connected to EXTI line
  *                                    with interrupt generation capability
  */
-int32_t BSP_PB_Init(Button_TypeDef Button, ButtonMode_TypeDef ButtonMode)
+int32_t BSP_PB_Init(Button_TypeDef Button, ButtonMode_TypeDef ButtonMode) {
-{
   GPIO_InitTypeDef gpio_init_structure;
-  static BSP_EXTI_LineCallback ButtonCallback[BUTTONn] = {BUTTON_USER_EXTI_Callback};
+  static BSP_EXTI_LineCallback ButtonCallback[BUTTONn] = {
-  static uint32_t  BSP_BUTTON_PRIO [BUTTONn] = {BSP_BUTTON_USER_IT_PRIORITY};
+      BUTTON_USER_EXTI_Callback};
+  static uint32_t BSP_BUTTON_PRIO[BUTTONn] = {BSP_BUTTON_USER_IT_PRIORITY};
   static const uint32_t BUTTON_EXTI_LINE[BUTTONn] = {BUTTON_USER_EXTI_LINE};
 
   /* Enable the BUTTON clock */
   BUTTON_USER_GPIO_CLK_ENABLE();
 
-  gpio_init_structure.Pin = BUTTON_PIN [Button];
+  gpio_init_structure.Pin = BUTTON_PIN[Button];
   gpio_init_structure.Pull = GPIO_PULLDOWN;
   gpio_init_structure.Speed = GPIO_SPEED_FREQ_HIGH;
 
-  if(ButtonMode == BUTTON_MODE_GPIO)
+  if (ButtonMode == BUTTON_MODE_GPIO) {
-  {
     /* Configure Button pin as input */
     gpio_init_structure.Mode = GPIO_MODE_INPUT;
-    HAL_GPIO_Init(BUTTON_PORT [Button], &gpio_init_structure);
+    HAL_GPIO_Init(BUTTON_PORT[Button], &gpio_init_structure);
-  }
+  } else /* (ButtonMode == BUTTON_MODE_EXTI) */
-  else /* (ButtonMode == BUTTON_MODE_EXTI) */
   {
     /* Configure Button pin as input with External interrupt */
     gpio_init_structure.Mode = GPIO_MODE_IT_RISING;
@@ -334,7 +301,8 @@ int32_t BSP_PB_Init(Button_TypeDef Button, ButtonMode_TypeDef ButtonMode)
     HAL_GPIO_Init(BUTTON_PORT[Button], &gpio_init_structure);
 
     (void)HAL_EXTI_GetHandle(&hpb_exti[Button], BUTTON_EXTI_LINE[Button]);
-    (void)HAL_EXTI_RegisterCallback(&hpb_exti[Button],  HAL_EXTI_COMMON_CB_ID, ButtonCallback[Button]);
+    (void)HAL_EXTI_RegisterCallback(&hpb_exti[Button], HAL_EXTI_COMMON_CB_ID,
+                                    ButtonCallback[Button]);
 
     /* Enable and set Button EXTI Interrupt to the lowest priority */
     HAL_NVIC_SetPriority((BUTTON_IRQn[Button]), BSP_BUTTON_PRIO[Button], 0x00);
@@ -351,8 +319,7 @@ int32_t BSP_PB_Init(Button_TypeDef Button, ButtonMode_TypeDef ButtonMode)
  *            @arg  BUTTON_USER: Wakeup Push Button
  * @note PB DeInit does not disable the GPIO clock
  */
-int32_t BSP_PB_DeInit(Button_TypeDef Button)
+int32_t BSP_PB_DeInit(Button_TypeDef Button) {
-{
   GPIO_InitTypeDef gpio_init_structure;
 
   gpio_init_structure.Pin = BUTTON_PIN[Button];
@@ -369,8 +336,7 @@ int32_t BSP_PB_DeInit(Button_TypeDef Button)
  *            @arg  BUTTON_USER: Wakeup Push Button
  * @retval The Button GPIO pin value (GPIO_PIN_RESET = button pressed)
  */
-int32_t BSP_PB_GetState(Button_TypeDef Button)
+int32_t BSP_PB_GetState(Button_TypeDef Button) {
-{
   return (int32_t)HAL_GPIO_ReadPin(BUTTON_PORT[Button], BUTTON_PIN[Button]);
 }
 
@@ -379,8 +345,7 @@ int32_t BSP_PB_GetState(Button_TypeDef Button)
  * @param  Button Can only be BUTTON_USER
  * @retval None
  */
-void BSP_PB_IRQHandler(Button_TypeDef Button)
+void BSP_PB_IRQHandler(Button_TypeDef Button) {
-{
   HAL_EXTI_IRQHandler(&hpb_exti[Button]);
 }
 
@@ -389,8 +354,7 @@ void BSP_PB_IRQHandler(Button_TypeDef Button)
  * @param  Button Specifies the pin connected EXTI line
  * @retval None
  */
-__weak void BSP_PB_Callback(Button_TypeDef Button)
+__weak void BSP_PB_Callback(Button_TypeDef Button) {
-{
   /* Prevent unused argument(s) compilation warning */
   UNUSED(Button);
 
@@ -404,34 +368,28 @@ __weak void BSP_PB_Callback(Button_TypeDef Button)
  * @param  COM COM port to be configured.
  *          This parameter can be COM1
  * @param  COM_Init Pointer to a UART_HandleTypeDef structure that contains the
-  *                  configuration information for the specified USART peripheral.
+ *                  configuration information for the specified USART
+ * peripheral.
  * @retval BSP error code
  */
-int32_t BSP_COM_Init(COM_TypeDef COM, COM_InitTypeDef *COM_Init)
+int32_t BSP_COM_Init(COM_TypeDef COM, COM_InitTypeDef *COM_Init) {
-{
   int32_t ret = BSP_ERROR_NONE;
 
-  if(COM >= COMn)
+  if (COM >= COMn) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
 #if (USE_HAL_UART_REGISTER_CALLBACKS == 0)
     /* Init the UART Msp */
     COM1_MspInit(&hcom_uart[COM]);
 #else
-    if(IsComMspCbValid[COM] == 0U)
+    if (IsComMspCbValid[COM] == 0U) {
-    {
+      if (BSP_COM_RegisterDefaultMspCallbacks(COM) != BSP_ERROR_NONE) {
-      if(BSP_COM_RegisterDefaultMspCallbacks(COM) != BSP_ERROR_NONE)
-      {
         return BSP_ERROR_MSP_FAILURE;
       }
     }
 #endif
 
-    if(MX_USART3_Init(&hcom_uart[COM], COM_Init) != HAL_OK)
+    if (MX_USART3_Init(&hcom_uart[COM], COM_Init) != HAL_OK) {
-    {
       ret = BSP_ERROR_PERIPH_FAILURE;
     }
   }
@@ -445,16 +403,12 @@ int32_t BSP_COM_Init(COM_TypeDef COM, COM_InitTypeDef *COM_Init)
  *          This parameter can be COM1
  * @retval BSP status
  */
-int32_t BSP_COM_DeInit(COM_TypeDef COM)
+int32_t BSP_COM_DeInit(COM_TypeDef COM) {
-{
   int32_t ret = BSP_ERROR_NONE;
 
-  if(COM >= COMn)
+  if (COM >= COMn) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     /* USART configuration */
     hcom_uart[COM].Instance = COM_USART[COM];
 
@@ -462,8 +416,7 @@ int32_t BSP_COM_DeInit(COM_TypeDef COM)
     COM1_MspDeInit(&hcom_uart[COM]);
 #endif /* (USE_HAL_UART_REGISTER_CALLBACKS == 0) */
 
-    if(HAL_UART_DeInit(&hcom_uart[COM]) != HAL_OK)
+    if (HAL_UART_DeInit(&hcom_uart[COM]) != HAL_OK) {
-    {
       ret = BSP_ERROR_PERIPH_FAILURE;
     }
   }
@@ -475,11 +428,12 @@ int32_t BSP_COM_DeInit(COM_TypeDef COM)
  * @brief  Configures COM port.
  * @param  huart USART handle
  * @param  COM_Init Pointer to a UART_HandleTypeDef structure that contains the
-  *                  configuration information for the specified USART peripheral.
+ *                  configuration information for the specified USART
+ * peripheral.
  * @retval HAL error code
  */
-__weak HAL_StatusTypeDef MX_USART3_Init(UART_HandleTypeDef *huart, MX_UART_InitTypeDef *COM_Init)
+__weak HAL_StatusTypeDef MX_USART3_Init(UART_HandleTypeDef *huart,
-{
+                                        MX_UART_InitTypeDef *COM_Init) {
   /* USART configuration */
   huart->Instance = COM_USART[COM1];
   huart->Init.BaudRate = COM_Init->BaudRate;
@@ -500,29 +454,23 @@ __weak HAL_StatusTypeDef MX_USART3_Init(UART_HandleTypeDef *huart, MX_UART_InitT
  *          This parameter can be COM1
  * @retval BSP status
  */
-int32_t BSP_COM_RegisterDefaultMspCallbacks(COM_TypeDef COM)
+int32_t BSP_COM_RegisterDefaultMspCallbacks(COM_TypeDef COM) {
-{
   int32_t ret = BSP_ERROR_NONE;
 
-  if(COM >= COMn)
+  if (COM >= COMn) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     __HAL_UART_RESET_HANDLE_STATE(&hcom_uart[COM]);
 
     /* Register default MspInit/MspDeInit Callback */
-    if(HAL_UART_RegisterCallback(&hcom_uart[COM], HAL_UART_MSPINIT_CB_ID, COM1_MspInit) != HAL_OK)
+    if (HAL_UART_RegisterCallback(&hcom_uart[COM], HAL_UART_MSPINIT_CB_ID,
-    {
+                                  COM1_MspInit) != HAL_OK) {
       ret = BSP_ERROR_PERIPH_FAILURE;
-    }
+    } else if (HAL_UART_RegisterCallback(&hcom_uart[COM],
-    else if(HAL_UART_RegisterCallback(&hcom_uart[COM], HAL_UART_MSPDEINIT_CB_ID, COM1_MspDeInit) != HAL_OK)
+                                         HAL_UART_MSPDEINIT_CB_ID,
-    {
+                                         COM1_MspDeInit) != HAL_OK) {
       ret = BSP_ERROR_PERIPH_FAILURE;
-    }
+    } else {
-    else
-    {
       IsComMspCbValid[COM] = 1U;
     }
   }
@@ -538,29 +486,23 @@ int32_t BSP_COM_RegisterDefaultMspCallbacks(COM_TypeDef COM)
  * @param Callbacks     pointer to COM1 MspInit/MspDeInit callback functions
  * @retval BSP status
  */
-int32_t BSP_COM_RegisterMspCallbacks(COM_TypeDef COM , BSP_COM_Cb_t *Callback)
+int32_t BSP_COM_RegisterMspCallbacks(COM_TypeDef COM, BSP_COM_Cb_t *Callback) {
-{
   int32_t ret = BSP_ERROR_NONE;
 
-  if(COM >= COMn)
+  if (COM >= COMn) {
-  {
     ret = BSP_ERROR_WRONG_PARAM;
-  }
+  } else {
-  else
-  {
     __HAL_UART_RESET_HANDLE_STATE(&hcom_uart[COM]);
 
     /* Register MspInit/MspDeInit Callbacks */
-    if(HAL_UART_RegisterCallback(&hcom_uart[COM], HAL_UART_MSPINIT_CB_ID, Callback->pMspInitCb) != HAL_OK)
+    if (HAL_UART_RegisterCallback(&hcom_uart[COM], HAL_UART_MSPINIT_CB_ID,
-    {
+                                  Callback->pMspInitCb) != HAL_OK) {
       ret = BSP_ERROR_PERIPH_FAILURE;
-    }
+    } else if (HAL_UART_RegisterCallback(&hcom_uart[COM],
-    else if(HAL_UART_RegisterCallback(&hcom_uart[COM], HAL_UART_MSPDEINIT_CB_ID, Callback->pMspDeInitCb) != HAL_OK)
+                                         HAL_UART_MSPDEINIT_CB_ID,
-    {
+                                         Callback->pMspDeInitCb) != HAL_OK) {
       ret = BSP_ERROR_PERIPH_FAILURE;
-    }
+    } else {
-    else
-    {
       IsComMspCbValid[COM] = 1U;
     }
   }
@@ -576,10 +518,8 @@ int32_t BSP_COM_RegisterMspCallbacks(COM_TypeDef COM , BSP_COM_Cb_t *Callback)
  *          This parameter can be COM1
  * @retval BSP status
  */
-int32_t BSP_COM_SelectLogPort(COM_TypeDef COM)
+int32_t BSP_COM_SelectLogPort(COM_TypeDef COM) {
-{
+  if (COM_ActiveLogPort != COM) {
-  if(COM_ActiveLogPort != COM)
-  {
     COM_ActiveLogPort = COM;
   }
   return BSP_ERROR_NONE;
@@ -588,13 +528,14 @@ int32_t BSP_COM_SelectLogPort(COM_TypeDef COM)
 /**
  * @brief  Redirect console output to COM
  */
- #ifdef __GNUC__
+#ifdef __GNUC__
- int __io_putchar (int ch)
+int __io_putchar(int ch)
- #else
+#else
- int fputc (int ch, FILE *f)
+int fputc(int ch, FILE *f)
- #endif /* __GNUC__ */
+#endif /* __GNUC__ */
 {
-  HAL_UART_Transmit (&hcom_uart [COM_ActiveLogPort], (uint8_t *) &ch, 1, COM_POLL_TIMEOUT);
+  HAL_UART_Transmit(&hcom_uart[COM_ActiveLogPort], (uint8_t *)&ch, 1,
+                    COM_POLL_TIMEOUT);
   return ch;
 }
 #endif /* USE_COM_LOG */
@@ -604,17 +545,15 @@ int32_t BSP_COM_SelectLogPort(COM_TypeDef COM)
  * @}
  */
 
-/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_Functions LOW LEVEL Private functions
+/** @defgroup STM32H7XX_NUCLEO_LOW_LEVEL_Private_Functions LOW LEVEL Private
+ * functions
  * @{
  */
 /**
  * @brief  Key EXTI line detection callbacks.
  * @retval BSP status
  */
-static void BUTTON_USER_EXTI_Callback(void)
+static void BUTTON_USER_EXTI_Callback(void) { BSP_PB_Callback(BUTTON_USER); }
-{
-  BSP_PB_Callback(BUTTON_USER);
-}
 
 #if (USE_BSP_COM_FEATURE > 0)
 /**
@@ -622,8 +561,7 @@ static void BUTTON_USER_EXTI_Callback(void)
  * @param  huart UART handle
  * @retval BSP status
  */
-static void COM1_MspInit(UART_HandleTypeDef *huart)
+static void COM1_MspInit(UART_HandleTypeDef *huart) {
-{
   GPIO_InitTypeDef gpio_init_structure;
 
   /* Prevent unused argument(s) compilation warning */
@@ -656,8 +594,7 @@ static void COM1_MspInit(UART_HandleTypeDef *huart)
  * @param  huart UART handle
  * @retval BSP status
  */
-static void COM1_MspDeInit(UART_HandleTypeDef *huart)
+static void COM1_MspDeInit(UART_HandleTypeDef *huart) {
-{
   GPIO_InitTypeDef gpio_init_structure;
 
   /* Prevent unused argument(s) compilation warning */

bsp_stm32h7_freertos/NUCLEO-H743ZI/Src/syscalls.c

@@ -4,19 +4,18 @@
  * the debug port.
  */
 
-#include <stdlib.h>
-#include <stdint.h>
-#include <unistd.h>
-#include <fcntl.h>
 #include <errno.h>
+#include <fcntl.h>
+#include <stdint.h>
+#include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 #include <sys/stat.h>
 #include <sys/types.h>
-#include <stdio.h>
+#include <unistd.h>
 
-#include <hardware_init.h>
 #include <boardconfig.h>
-
+#include <hardware_init.h>
 
 #define AUTO_RETURN_AFTER_NEWLINE 1
 
@@ -24,7 +23,7 @@
 // If the At91lib\utility\stdio.c printf implementation is not used anymore:
 // remove if there are spurious newlines!
 
-//int _read(int file, void *ptr, size_t len) {
+// int _read(int file, void *ptr, size_t len) {
 //	(void)len;
 //	if(file <= STDERR_FILENO) {
 //		*(char *) ptr = DBGU_GetChar();
@@ -43,32 +42,30 @@
 //		return -1;
 //	}
 //
-//}
+// }
 
 int _write(int file, const void *ptr, size_t len) {
   size_t i;
 
-	if(file <= STDERR_FILENO) {
+  if (file <= STDERR_FILENO) {
-		for(i=0; i<len; i++) {
+    for (i = 0; i < len; i++) {
 #if AUTO_RETURN_AFTER_NEWLINE
-			if(((const char *) ptr)[i] == '\n' &&
+      if (((const char *)ptr)[i] == '\n' &&
-					((const char *) ptr)[i+1] != '\r') {
+          ((const char *)ptr)[i + 1] != '\r') {
-				HAL_UART_Transmit(&huart3, (uint8_t*)"\r", 1 ,
+        HAL_UART_Transmit(&huart3, (uint8_t *)"\r", 1, DEBUG_UART_MS_TIMEOUT);
-						DEBUG_UART_MS_TIMEOUT);
       }
 #endif
-			uint8_t* character = (uint8_t*) ptr;
+      const uint8_t *character = (const uint8_t *)ptr;
-			HAL_UART_Transmit(&huart3, (uint8_t*) (character + i),
+      HAL_UART_Transmit(&huart3, (const uint8_t *)(character + i), 1,
-					1 , DEBUG_UART_MS_TIMEOUT);
+                        DEBUG_UART_MS_TIMEOUT);
     }
     return len;
   } else {
     return -1;
   }
-
 }
 
-int _lseek (int file, int ptr, int dir) {
+int _lseek(int file, int ptr, int dir) {
   (void)file;
   (void)ptr;
   (void)dir;
@@ -121,34 +118,29 @@ int _stat(const char *path, struct stat *st) {
 }
 
 int _isatty(int fd) {
-	if(fd <= STDERR_FILENO) {
+  if (fd <= STDERR_FILENO) {
     return 1;
-	}
+  } else {
-	else {
     return 0;
   }
 }
 
 void exit(int n) {
   printf("\n\r EXITING WITH CODE: %u \n\r", n);
-	//restart();
+  // restart();
-	while(1);
+  while (1)
+    ;
 }
 
 void _exit(int n) {
   printf("\n\r EXITING WITH CODE: %u \n\r", n);
-	//restart();
+  // restart();
-	while(1);
+  while (1)
+    ;
 }
 
-int _kill() {
+int _kill() { return -1; }
-	return -1;
-}
 
-int _getpid() {
+int _getpid() { return 1; }
-	return 1;
-}
 
-int _gettimeofday() {
+int _gettimeofday() { return -1; }
-	return -1;
-}

bsp_stm32h7_freertos/boardtest/CMakeLists.txt

@@ -1,4 +1 @@
-target_sources(${TARGET_NAME} PRIVATE
+target_sources(${TARGET_NAME} PRIVATE SpiTest.cpp)
-    SpiTest.cpp
-)
-

bsp_stm32h7_freertos/boardtest/SpiTest.cpp

@@ -1,21 +1,23 @@
 #include "SpiTest.h"
 #include "fsfw_hal/stm32h7/spi/stm32h743zi.h"
 
-SpiTest::SpiTest(SpiComIF &spiComIF):
+SpiTest::SpiTest(SpiComIF &spiComIF)
-        spiComIF(spiComIF), spiCookie(TEST_ADDRESS, spi::SpiBus::SPI_1,
+    : spiComIF(spiComIF),
-        spi::TransferModes::POLLING, &mspCfg, 1000000, spi::SpiModes::MODE_0, 32) {
+      spiCookie(TEST_ADDRESS, spi::SpiBus::SPI_1, spi::TransferModes::POLLING,
+                &mspCfg, 1000000, spi::SpiModes::MODE_0, 32) {
   stm32h7::h743zi::standardPollingCfg(mspCfg);
   spiComIF.initializeInterface(&spiCookie);
 }
 
 ReturnValue_t SpiTest::performOperation(uint8_t opCode) {
   std::array<uint8_t, 3> dummyData = {0x01, 0x02, 0x03};
-    ReturnValue_t result = spiComIF.sendMessage(&spiCookie, dummyData.data(), dummyData.size());
+  ReturnValue_t result =
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+      spiComIF.sendMessage(&spiCookie, dummyData.data(), dummyData.size());
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     return result;
   }
-    std::array<uint8_t, 3> recBuf;
+  std::array<uint8_t, 3> recBuf{};
-    uint8_t* recPtr = recBuf.data();
+  uint8_t *recPtr = recBuf.data();
   size_t readLen = 0;
   return spiComIF.readReceivedMessage(&spiCookie, &recPtr, &readLen);
 }

bsp_stm32h7_freertos/boardtest/SpiTest.h

@@ -7,13 +7,14 @@
 
 class SpiTest {
 public:
-    SpiTest(SpiComIF& spiComIF);
+  explicit SpiTest(SpiComIF &spiComIF);
 
   ReturnValue_t performOperation(uint8_t opCode = 0);
+
 private:
   static constexpr address_t TEST_ADDRESS = 0x00;
   spi::MspPollingConfigStruct mspCfg;
-    SpiComIF& spiComIF;
+  SpiComIF &spiComIF;
   SpiCookie spiCookie;
 };
 

bsp_stm32h7_freertos/core/CMakeLists.txt

@@ -1,5 +1 @@
-target_sources(${TARGET_NAME} PRIVATE
+target_sources(${TARGET_NAME} PRIVATE InitMission.cpp ObjectFactory.cpp)
-	InitMission.cpp
-	ObjectFactory.cpp
-)
-

bsp_stm32h7_freertos/core/InitMission.cpp

@@ -6,160 +6,175 @@
 #include "example/utility/TaskCreation.h"
 #include "fsfw_tests/integration/assemblies/TestAssembly.h"
 
+#include "fsfw/devicehandlers/DeviceHandlerIF.h"
+#include "fsfw/modes/HasModesIF.h"
+#include "fsfw/modes/ModeMessage.h"
 #include "fsfw/returnvalues/HasReturnvaluesIF.h"
 #include "fsfw/serviceinterface/ServiceInterface.h"
 #include "fsfw/tasks/FixedTimeslotTaskIF.h"
 #include "fsfw/tasks/PeriodicTaskIF.h"
 #include "fsfw/tasks/TaskFactory.h"
-#include "fsfw/devicehandlers/DeviceHandlerIF.h"
-#include "fsfw/modes/HasModesIF.h"
-#include "fsfw/modes/ModeMessage.h"
 
 #include "FreeRTOS.h"
 
-void InitMission::createTasks() {
+void assemlyDemo();
-    TaskFactory* taskFactory = TaskFactory::instance();
 
+void InitMission::createTasks() {
+  TaskFactory *taskFactory = TaskFactory::instance();
+  ReturnValue_t result = HasReturnvaluesIF::RETURN_OK;
+  static_cast<void>(taskFactory);
+  static_cast<void>(result);
 #if OBSW_ADD_CORE_COMPONENTS == 1
   /* TMTC Distribution */
-    PeriodicTaskIF* distributerTask = taskFactory->createPeriodicTask(
+  PeriodicTaskIF *distributerTask =
-            "DIST", 5, 1024 * 2, 0.2, nullptr);
+      taskFactory->createPeriodicTask("DIST", 5, 1024 * 2, 0.2, nullptr);
-    ReturnValue_t result = distributerTask->addComponent(objects::CCSDS_DISTRIBUTOR);
+  result = distributerTask->addComponent(objects::CCSDS_DISTRIBUTOR);
-    if(result!=HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("CCSDS distributor", objects::CCSDS_DISTRIBUTOR);
   }
   result = distributerTask->addComponent(objects::PUS_DISTRIBUTOR);
-    if(result!=HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS distributor", objects::PUS_DISTRIBUTOR);
   }
   result = distributerTask->addComponent(objects::TM_FUNNEL);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("TM funnel", objects::TM_FUNNEL);
   }
 
+#if OBSW_ADD_LWIP_COMPONENTS == 1
   /* UDP bridge */
-    PeriodicTaskIF* udpBridgeTask = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *udpBridgeTask = TaskFactory::instance()->createPeriodicTask(
       "UDP_UNIX_BRIDGE", 6, 1024 * 2, 0.2, nullptr);
   result = udpBridgeTask->addComponent(objects::UDP_BRIDGE);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("UDP bridge", objects::UDP_BRIDGE);
   }
-    PeriodicTaskIF* udpPollingTask = TaskFactory::instance()->
+  PeriodicTaskIF *udpPollingTask = TaskFactory::instance()->createPeriodicTask(
-            createPeriodicTask("UDP_POLLING", 8, 1024, 0.1, nullptr);
+      "UDP_POLLING", 8, 1024, 0.1, nullptr);
   result = udpPollingTask->addComponent(objects::UDP_POLLING_TASK);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("UDP polling task", objects::UDP_POLLING_TASK);
   }
+#endif
 
-    PeriodicTaskIF* eventManagerTask = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *eventManagerTask =
-            "EVENT_MGMT", 4, 1024 * 2, 0.1, nullptr);
+      TaskFactory::instance()->createPeriodicTask("EVENT_MGMT", 4, 1024 * 2,
+                                                  0.1, nullptr);
   result = eventManagerTask->addComponent(objects::EVENT_MANAGER);
-    if(result!=HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("Event Manager", objects::EVENT_MANAGER);
   }
 #endif /* OBSW_ADD_CORE_COMPONENTS == 1 */
 
 #if OBSW_ADD_PUS_STACK == 1
   /* PUS Services */
-    PeriodicTaskIF* pusVerification = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *pusVerification = TaskFactory::instance()->createPeriodicTask(
       "PUS_VERIF_1", 4, 1024 * 2, 0.2, nullptr);
   result = pusVerification->addComponent(objects::PUS_SERVICE_1_VERIFICATION);
-    if(result != HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS 1", objects::PUS_SERVICE_1_VERIFICATION);
   }
 
-    PeriodicTaskIF* pusHighPrio = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *pusHighPrio = TaskFactory::instance()->createPeriodicTask(
       "PUS_HIGH_PRIO", 5, 1024 * 2, 0.2, nullptr);
   result = pusHighPrio->addComponent(objects::PUS_SERVICE_2_DEVICE_ACCESS);
-    if(result!=HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS 2", objects::PUS_SERVICE_2_DEVICE_ACCESS);
   }
   result = pusHighPrio->addComponent(objects::PUS_SERVICE_5_EVENT_REPORTING);
-    if(result != HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS 5", objects::PUS_SERVICE_5_EVENT_REPORTING);
   }
   result = pusHighPrio->addComponent(objects::PUS_SERVICE_9_TIME_MGMT);
-    if(result!=HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS 9", objects::PUS_SERVICE_9_TIME_MGMT);
   }
 
-    PeriodicTaskIF* pusMedPrio = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *pusMedPrio = TaskFactory::instance()->createPeriodicTask(
       "PUS_HIGH_PRIO", 4, 1024 * 2, 0.8, nullptr);
   result = pusMedPrio->addComponent(objects::PUS_SERVICE_8_FUNCTION_MGMT);
-    if(result!=HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS 8", objects::PUS_SERVICE_8_FUNCTION_MGMT);
   }
   result = pusMedPrio->addComponent(objects::PUS_SERVICE_200_MODE_MGMT);
-    if(result!=HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS 200", objects::PUS_SERVICE_200_MODE_MGMT);
   }
 
-    PeriodicTaskIF* pusLowPrio = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *pusLowPrio = TaskFactory::instance()->createPeriodicTask(
       "PUS_LOW_PRIO", 3, 1024 * 2, 1.6, nullptr);
   result = pusLowPrio->addComponent(objects::PUS_SERVICE_17_TEST);
-    if(result!=HasReturnvaluesIF::RETURN_OK){
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("PUS 17", objects::PUS_SERVICE_17_TEST);
   }
 #endif /* OBSW_ADD_PUS_STACK == 1 */
 
 #if OBSW_ADD_TASK_EXAMPLE == 1
-    FixedTimeslotTaskIF* timeslotDemoTask = TaskFactory::instance()->createFixedTimeslotTask(
+  FixedTimeslotTaskIF *timeslotDemoTask =
-            "PST_TASK", 6, 1024 * 2, 0.5, nullptr);
+      TaskFactory::instance()->createFixedTimeslotTask("PST_TASK", 6, 1024 * 2,
+                                                       0.5, nullptr);
   result = pst::pollingSequenceExamples(timeslotDemoTask);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
 #if FSFW_CPP_OSTREAM_ENABLED == 1
-        sif::error << "InitMission::createTasks: PST initialization failed!" << std::endl;
+    sif::error << "InitMission::createTasks: PST initialization failed!"
+               << std::endl;
 #else
-        sif::printError( "InitMission::createTasks: PST initialization failed!\n");
+    sif::printError("InitMission::createTasks: PST initialization failed!\n");
 #endif
   }
 
-    PeriodicTaskIF* readerTask = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *readerTask = TaskFactory::instance()->createPeriodicTask(
       "READER_TASK", 3, 1024, 1.0, nullptr);
   result = readerTask->addComponent(objects::TEST_DUMMY_4);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("TEST_DUMMY_4", objects::TEST_DUMMY_4);
   }
 #endif /* OBSW_ADD_TASK_EXAMPLE == 1 */
 
 #if OBSW_ADD_DEVICE_HANDLER_DEMO == 1
-    FixedTimeslotTaskIF* testDevicesTask = TaskFactory::instance()->createFixedTimeslotTask(
+  FixedTimeslotTaskIF *testDevicesTask =
-            "PST_TEST_TASK", 7, 1024 * 2, 1.0, nullptr);
+      TaskFactory::instance()->createFixedTimeslotTask("PST_TEST_TASK", 7,
+                                                       1024 * 2, 1.0, nullptr);
   result = pst::pollingSequenceDevices(testDevicesTask);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
 #if FSFW_CPP_OSTREAM_ENABLED == 1
-        sif::error << "InitMission::createTasks: Test PST initialization failed!" << std::endl;
+    sif::error << "InitMission::createTasks: Test PST initialization failed!"
+               << std::endl;
 #else
-        sif::printError("InitMission::createTasks: Test PST initialization faiedl!\n");
+    sif::printError(
+        "InitMission::createTasks: Test PST initialization faiedl!\n");
 #endif
   }
-    PeriodicTaskIF* assemblyTask = taskFactory->createPeriodicTask("ASS_TASK", 3,
+  PeriodicTaskIF *assemblyTask =
-            1024, 2.0, nullptr);
+      taskFactory->createPeriodicTask("ASS_TASK", 3, 1024, 2.0, nullptr);
-    if(assemblyTask == nullptr){
+  if (assemblyTask == nullptr) {
     task::printInitError("ASS_TASK", objects::TEST_ASSEMBLY);
   }
   result = assemblyTask->addComponent(objects::TEST_ASSEMBLY);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("ASS_TASK", objects::TEST_ASSEMBLY);
   }
 #endif /* OBSW_ADD_DEVICE_HANDLER_DEMO == 1 */
 
-    PeriodicTaskIF* testTask = TaskFactory::instance()->createPeriodicTask(
+  PeriodicTaskIF *testTask = TaskFactory::instance()->createPeriodicTask(
       "TEST", 2, 2048, 1.0, nullptr);
   result = testTask->addComponent(objects::TEST_TASK);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
     task::printInitError("Test Task", objects::TEST_TASK);
   }
 
 #if OBSW_PERIPHERAL_PST == 1
-    FixedTimeslotTaskIF* peripheralPst = TaskFactory::instance()->createFixedTimeslotTask(
+  FixedTimeslotTaskIF *peripheralPst =
-            "PST_PERIPHERAL_TASK", 9, 1024 * 2, 2.0, nullptr);
+      TaskFactory::instance()->createFixedTimeslotTask("PST_PERIPHERAL_TASK", 9,
+                                                       1024 * 2, 2.0, nullptr);
   result = pst::pstPeripheralsTest(peripheralPst);
-    if(result != HasReturnvaluesIF::RETURN_OK) {
+  if (result != HasReturnvaluesIF::RETURN_OK) {
 #if FSFW_CPP_OSTREAM_ENABLED == 1
-        sif::error << "InitMission::createTasks: Test PST initialization failed!" << std::endl;
+    sif::error << "InitMission::createTasks: Test PST initialization failed!"
+               << std::endl;
 #else
-        sif::printError("InitMission::createTasks: Test PST initialization faiedl!\n");
+    sif::printError(
+        "InitMission::createTasks: Test PST initialization faiedl!\n");
 #endif
   }
 #endif
@@ -173,8 +188,10 @@ void InitMission::createTasks() {
 #if OBSW_ADD_CORE_COMPONENTS == 1
   distributerTask->startTask();
   eventManagerTask->startTask();
+#if OBSW_ADD_LWIP_COMPONENTS == 1
   udpBridgeTask->startTask();
   udpPollingTask->startTask();
+#endif
 #endif /* OBSW_ADD_CORE_COMPONENTS == 1 */
 
 #if OBSW_ADD_PUS_STACK == 1
@@ -201,8 +218,8 @@ void InitMission::createTasks() {
   testTask->startTask();
 
 #if FSFW_CPP_OSTREAM_ENABLED == 1
-    sif::info << "Remaining FreeRTOS heap size: " << std::dec << xPortGetFreeHeapSize() <<
+  sif::info << "Remaining FreeRTOS heap size: " << std::dec
-            " bytes." << std::endl;
+            << xPortGetFreeHeapSize() << " bytes." << std::endl;
   sif::info << "Tasks started.." << std::endl;
 #else
   sif::printInfo("Remaining FreeRTOS heap size: %lu\n",
@@ -211,20 +228,29 @@ void InitMission::createTasks() {
 #endif
 
 #if OBSW_ADD_DEVICE_HANDLER_DEMO
-    HasModesIF* assembly = ObjectManager::instance()->get<HasModesIF>(objects::TEST_ASSEMBLY);
+  auto *assembly = ObjectManager::instance()->get<HasModesIF>(objects::TEST_ASSEMBLY);
-    if (assembly == nullptr){
+  if (assembly == nullptr) {
     return;
   }
 #if FSFW_CPP_OSTREAM_ENABLED == 1
-    sif::info << "Waiting 5 Seconds and then command Test Assembly to Normal, Dual" << std::endl;
+  sif::info
+      << "Waiting 5 Seconds and then command Test Assembly to Normal, Dual"
+      << std::endl;
 #else
-    sif::printInfo("Waiting 5 Seconds and then command Test Assembly to Normal, Dual \n");
+  sif::printInfo(
+      "Waiting 5 Seconds and then command Test Assembly to Normal, Dual \n");
 #endif
 
   TaskFactory::delayTask(5000);
+
+#endif /* OBSW_ADD_DEVICE_HANDLER_DEMO */
+}
+
+void assemblyDemo(HasModesIF* assembly) {
     CommandMessage modeMessage;
     ModeMessage::setModeMessage(&modeMessage, ModeMessage::CMD_MODE_COMMAND,
-            DeviceHandlerIF::MODE_NORMAL, TestAssembly::submodes::DUAL);
+                                DeviceHandlerIF::MODE_NORMAL,
+                                TestAssembly::submodes::DUAL);
 #if FSFW_CPP_OSTREAM_ENABLED == 1
     sif::info << "Commanding Test Assembly to Normal, Dual" << std::endl;
 #else
@@ -232,6 +258,4 @@ void InitMission::createTasks() {
 #endif
     MessageQueueSenderIF::sendMessage(assembly->getCommandQueue(), &modeMessage,
                                       MessageQueueIF::NO_QUEUE);
-#endif /* OBSW_ADD_DEVICE_HANDLER_DEMO */
 }
-

bsp_stm32h7_freertos/core/ObjectFactory.cpp

@@ -1,100 +1,92 @@
 #include "ObjectFactory.h"
 #include "OBSWConfig.h"
-#include "devices/devAddresses.h"
-#include "objects/systemObjectList.h"
 #include "hardware_init.h"
+#include "objects/systemObjectList.h"
 
-#include "example/utility/TmFunnel.h"
 #include "example/core/GenericFactory.h"
 
-#include "example_common/stm32h7/networking/UdpTcLwIpPollingTask.h"
-#include "example_common/stm32h7/networking/TmTcLwIpUdpBridge.h"
 #include "example_common/stm32h7/STM32TestTask.h"
-
+#include "example_common/stm32h7/networking/TmTcLwIpUdpBridge.h"
-#include "fsfw/datapoollocal/LocalDataPoolManager.h"
+#include "example_common/stm32h7/networking/UdpTcLwIpPollingTask.h"
-#include "fsfw/monitoring/MonitoringMessageContent.h"
 #include "fsfw/storagemanager/PoolManager.h"
-#include "fsfw/tmtcpacket/pus/tm.h"
-#include "fsfw/tmtcservices/CommandingServiceBase.h"
 #include "fsfw/tmtcservices/PusServiceBase.h"
 
-
 #if OBSW_PERFORM_L3GD20H_TEST == 1
-#include "fsfw/hal/stm32h7/spi/SpiCookie.h"
-#include "fsfw/hal/stm32h7/spi/SpiComIF.h"
 #include "fsfw/hal/devicehandlers/GyroL3GD20Handler.h"
-#include "fsfw/hal/stm32h7/spi/stm32h743ziSpi.h"
+#include "fsfw/hal/stm32h7/spi/SpiComIF.h"
+#include "fsfw/hal/stm32h7/spi/SpiCookie.h"
 #include "fsfw/hal/stm32h7/spi/spiCore.h"
-DMA_HandleTypeDef* txHandle = nullptr;
+#include "fsfw/hal/stm32h7/spi/stm32h743ziSpi.h"
-DMA_HandleTypeDef* rxHandle = nullptr;
+DMA_HandleTypeDef *txHandle = nullptr;
+DMA_HandleTypeDef *rxHandle = nullptr;
 #endif
 
-void ObjectFactory::produce(void* args) {
+void ObjectFactory::produce(void *args) {
   /* Located inside GenericFactory source file */
   Factory::setStaticFrameworkObjectIds();
 
 #if OBSW_ADD_CORE_COMPONENTS == 1
   {
     LocalPool::LocalPoolConfig poolCfg = {
-				{100, 32}, {30, 64}, {10, 128}, {2, 1024}
+        {100, 32}, {30, 64}, {10, 128}, {2, 1024}};
-		};
     new PoolManager(objects::TC_STORE, poolCfg);
   }
 
   {
     LocalPool::LocalPoolConfig poolCfg = {
-				{100, 32}, {30, 64}, {10, 128}, {2, 1024}
+        {100, 32}, {30, 64}, {10, 128}, {2, 1024}};
-		};
     new PoolManager(objects::TM_STORE, poolCfg);
   }
 
   {
     LocalPool::LocalPoolConfig poolCfg = {
-				{100, 32}, {30, 64}, {10, 128}, {2, 1024}
+        {100, 32}, {30, 64}, {10, 128}, {2, 1024}};
-		};
     new PoolManager(objects::IPC_STORE, poolCfg);
   }
 
+#if OBSW_ADD_LWIP_COMPONENTS == 1
   /* UDP Server */
-	new TmTcLwIpUdpBridge(objects::UDP_BRIDGE,
+  new TmTcLwIpUdpBridge(objects::UDP_BRIDGE, objects::CCSDS_DISTRIBUTOR,
-			objects::CCSDS_DISTRIBUTOR, objects::TM_STORE, objects::TC_STORE);
+                        objects::TM_STORE, objects::TC_STORE);
-	new UdpTcLwIpPollingTask(objects::UDP_POLLING_TASK, objects::UDP_BRIDGE, &gnetif);
+  new UdpTcLwIpPollingTask(objects::UDP_POLLING_TASK, objects::UDP_BRIDGE,
+                           &gnetif);
+#endif
 #endif /* OBSW_ADD_CORE_COMPONENTS == 1 */
 
   ObjectFactory::produceGenericObjects();
 
   /* Test Device Handler */
-    new STM32TestTask(objects::TEST_TASK, false, true);
+  new STM32TestTask(objects::TEST_TASK, true, true);
 
 #if OBSW_PERFORM_L3GD20H_TEST == 1
-
+  spi::MspCfgBase *mspCfg = nullptr;
-    spi::MspCfgBase* mspCfg = nullptr;
   spi::TransferModes transferMode = spi::TransferModes::DMA;
-    if(transferMode == spi::TransferModes::POLLING) {
+  if (transferMode == spi::TransferModes::POLLING) {
     auto typedCfg = new spi::MspPollingConfigStruct();
     spi::h743zi::standardPollingCfg(*typedCfg);
     mspCfg = typedCfg;
-    }
+  } else if (transferMode == spi::TransferModes::INTERRUPT) {
-    else if(transferMode == spi::TransferModes::INTERRUPT) {
     auto typedCfg = new spi::MspIrqConfigStruct();
-        spi::h743zi::standardInterruptCfg(*typedCfg, IrqPriorities::HIGHEST_FREERTOS);
+    spi::h743zi::standardInterruptCfg(*typedCfg,
+                                      IrqPriorities::HIGHEST_FREERTOS);
     mspCfg = typedCfg;
-    }
+  } else if (transferMode == spi::TransferModes::DMA) {
-    else if(transferMode == spi::TransferModes::DMA) {
     auto typedCfg = new spi::MspDmaConfigStruct();
     txHandle = new DMA_HandleTypeDef();
     rxHandle = new DMA_HandleTypeDef();
     spi::setDmaHandles(txHandle, rxHandle);
     spi::h743zi::standardDmaCfg(*typedCfg, IrqPriorities::HIGHEST_FREERTOS,
-                IrqPriorities::HIGHEST_FREERTOS, IrqPriorities::HIGHEST_FREERTOS);
+                                IrqPriorities::HIGHEST_FREERTOS,
+                                IrqPriorities::HIGHEST_FREERTOS);
     mspCfg = typedCfg;
   }
 
   new SpiComIF(objects::SPI_COM_IF);
-    auto spiCookie = new SpiCookie(devaddress::L3GD20H, spi::SpiBus::SPI_1, transferMode, mspCfg,
+  auto spiCookie = new SpiCookie(devaddress::L3GD20H, spi::SpiBus::SPI_1,
-            3900000, spi::SpiModes::MODE_3, GPIO_PIN_14, GPIOD, 32);
+                                 transferMode, mspCfg, 3900000,
-    auto gyroDevice = new GyroHandlerL3GD20H(objects::SPI_DEVICE_TEST, objects::SPI_COM_IF,
+                                 spi::SpiModes::MODE_3, GPIO_PIN_14, GPIOD, 32);
-            spiCookie);
+  auto gyroDevice = new GyroHandlerL3GD20H(objects::SPI_DEVICE_TEST,
+                                           objects::SPI_COM_IF, spiCookie);
   gyroDevice->setStartUpImmediately();
   gyroDevice->setGoNormalModeAtStartup();
 #endif

bsp_stm32h7_freertos/core/ObjectFactory.h

@@ -1,10 +1,9 @@
 #ifndef MISSION_CORE_OBJECTFACTORY_H_
 #define MISSION_CORE_OBJECTFACTORY_H_
 
-
 namespace ObjectFactory {
-	void setStatics();
+void setStatics();
-	void produce(void* args);
+void produce(void *args);
-};
+}; // namespace ObjectFactory
 
 #endif /* MISSION_CORE_OBJECTFACTORY_H_ */

bsp_stm32h7_freertos/fsfwconfig/CMakeLists.txt

@@ -1,11 +1,6 @@
-target_sources(${TARGET_NAME} 
+target_sources(
-	PRIVATE
+  ${TARGET_NAME} PRIVATE ipc/missionMessageTypes.cpp
-		ipc/missionMessageTypes.cpp
+                         pollingsequence/pollingSequenceFactory.cpp)
-		pollingsequence/pollingSequenceFactory.cpp
-)
 
 # Add include paths for the executable
-target_include_directories(${TARGET_NAME} 
+target_include_directories(${TARGET_NAME} PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})
-	PUBLIC 
-		${CMAKE_CURRENT_SOURCE_DIR}
-)

bsp_stm32h7_freertos/fsfwconfig/devices/devAddresses.h

@@ -4,11 +4,8 @@
 #include <cstdint>
 
 namespace devaddress {
-enum devaddress: uint32_t {
+enum devaddress : uint32_t { L3GD20H = 1 };
-    L3GD20H = 1
-};
 
 }
 
-
 #endif /* BSP_STM32_FREERTOS_FSFWCONFIG_DEVICES_DEVADDRESSES_H_ */

bsp_stm32h7_freertos/fsfwconfig/events/subsystemIdRanges.h

@@ -4,8 +4,7 @@
 #include "commonSubsystemIds.h"
 
 namespace SUBSYSTEM_ID {
-enum subsystemId: uint8_t {
+enum subsystemId : uint8_t {};
-};
 }
 
 #endif /* FSFWCONFIG_TMTC_SUBSYSTEMIDRANGES_H_ */

bsp_stm32h7_freertos/fsfwconfig/ipc/missionMessageTypes.cpp

@@ -2,8 +2,8 @@
 #include <fsfw/ipc/CommandMessage.h>
 #include <fsfw/ipc/CommandMessageCleaner.h>
 
-void messagetypes::clearMissionMessage(CommandMessage* message) {
+void messagetypes::clearMissionMessage(CommandMessage *message) {
-	switch((message->getMessageType())) {
+  switch ((message->getMessageType())) {
   default:
     message->setCommand(CommandMessage::CMD_NONE);
     break;

bsp_stm32h7_freertos/fsfwconfig/ipc/missionMessageTypes.h

@@ -7,13 +7,14 @@ class CommandMessage;
 
 namespace messagetypes {
 /* First type must have number MESSAGE_TYPE::FW_MESSAGES_COUNT! */
-/* Remember to add new message types to the clearMissionMessage function below! */
+/* Remember to add new message types to the clearMissionMessage function below!
+ */
 enum MISSION_MESSAGE_TYPE {
   COSTUM_MESSAGE = FW_MESSAGES_COUNT,
 };
 
-void clearMissionMessage(CommandMessage* message);
+void clearMissionMessage(CommandMessage *message);
 
-}
+} // namespace messagetypes
 
 #endif /* FSFWCONFIG_IPC_MISSIONMESSAGETYPES_H_ */

bsp_stm32h7_freertos/fsfwconfig/objects/systemObjectList.h

@@ -1,7 +1,7 @@
 #ifndef FSFWCONFIG_OBJECTS_SYSTEMOBJECTLIST_H_
 #define FSFWCONFIG_OBJECTS_SYSTEMOBJECTLIST_H_
 
-#include <commonSystemObjects.h>
+#include <commonObjects.h>
 
 namespace objects {
 enum mission_objects {

bsp_stm32h7_freertos/fsfwconfig/pollingsequence/pollingSequenceFactory.cpp

@@ -12,22 +12,26 @@ ReturnValue_t pst::pstPeripheralsTest(FixedTimeslotTaskIF *thisSequence) {
   static_cast<void>(length);
 
 #if OBSW_PERFORM_L3GD20H_TEST == 1
-    thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0, DeviceHandlerIF::PERFORM_OPERATION);
+  thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0,
-    thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0.3, DeviceHandlerIF::SEND_WRITE);
+                        DeviceHandlerIF::PERFORM_OPERATION);
+  thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0.3,
+                        DeviceHandlerIF::SEND_WRITE);
   thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0.45 * length,
                         DeviceHandlerIF::GET_WRITE);
-    thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0.6 * length, DeviceHandlerIF::SEND_READ);
+  thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0.6 * length,
-    thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0.8 * length, DeviceHandlerIF::GET_READ);
+                        DeviceHandlerIF::SEND_READ);
+  thisSequence->addSlot(objects::SPI_DEVICE_TEST, 0.8 * length,
+                        DeviceHandlerIF::GET_READ);
 #endif
 
   if (thisSequence->checkSequence() == HasReturnvaluesIF::RETURN_OK) {
     return HasReturnvaluesIF::RETURN_OK;
   }
 #if FSFW_CPP_OSTREAM_ENABLED == 1
-    sif::error << "pst::pollingSequenceInitFunction: Initialization errors!" << std::endl;
+  sif::error << "pst::pollingSequenceInitFunction: Initialization errors!"
+             << std::endl;
 #else
   sif::printError("pst::pollingSequenceInitFunction: Initialization errors!\n");
 #endif
   return HasReturnvaluesIF::RETURN_FAILED;
-
 }

bsp_stm32h7_freertos/fsfwconfig/pollingsequence/pollingSequenceFactory.h

@@ -9,7 +9,7 @@ class FixedTimeslotTaskIF;
 namespace pst {
 ReturnValue_t pstPeripheralsTest(FixedTimeslotTaskIF *thisSequence);
 ReturnValue_t pollingSequenceExamples(FixedTimeslotTaskIF *thisSequence);
-ReturnValue_t pollingSequenceDevices(FixedTimeslotTaskIF* thisSequence);
+ReturnValue_t pollingSequenceDevices(FixedTimeslotTaskIF *thisSequence);
-}
+} // namespace pst
 
 #endif /* POLLINGSEQUENCE_POLLINGSEQUENCFACTORY_H_ */

bsp_stm32h7_freertos/fsfwconfig/returnvalues/classIds.h

@@ -4,9 +4,7 @@
 #include <commonConfig.h>
 
 namespace CLASS_ID {
-enum classIds: uint8_t {
+enum classIds : uint8_t {};
-};
 }
 
-
 #endif /* FSFWCONFIG_RETURNVALUES_CLASSIDS_H_ */

bsp_stm32h7_freertos/fsfwconfig/tmtc/apid.h

@@ -1,8 +1,8 @@
 #ifndef FSFWCONFIG_TMTC_APID_H_
 #define FSFWCONFIG_TMTC_APID_H_
 
-#include <cstdint>
 #include <commonConfig.h>
+#include <cstdint>
 
 namespace apid {
 static const uint16_t APID = COMMON_APID;

bsp_stm32h7_freertos/fsfwconfig/tmtc/pusIds.h

@@ -3,5 +3,4 @@
 
 #include <commonConfig.h>
 
-
 #endif /* FSFWCONFIG_TMTC_PUSIDS_H_ */

bsp_stm32h7_freertos/main.cpp

@@ -1,7 +1,5 @@
-#include "hardware_init.h"
-#include "boardconfig.h"
-#include "OBSWVersion.h"
 #include "OBSWConfig.h"
+#include "hardware_init.h"
 
 #if OBSW_ADD_LWIP_COMPONENTS == 1
 #include <lwip/init.h>
@@ -16,16 +14,14 @@
 #include <fsfw/serviceinterface/ServiceInterface.h>
 #include <fsfw/tasks/TaskFactory.h>
 
-#include "FreeRTOS.h"
-#include "task.h"
 #include "cmsis_os.h"
+#include "task.h"
 
 #include <cstdio>
 #if FSFW_CPP_OSTREAM_ENABLED == 1
 #include <iostream>
 #endif
 
-
 #if !defined(BOARD_NAME)
 #define BOARD_NAME "unknown board"
 #endif
@@ -53,7 +49,7 @@ int main() {
   osKernelStart();
 
   /* Should not be reached, scheduler should now be running. */
-	for(;;) {}
+  for (;;) {}
   return 0;
 }
 
@@ -74,7 +70,7 @@ void initTask(void *parameters) {
   sif::printInfo("Creating objects..\n\r");
 #endif
 
-    ObjectManager* objManager = ObjectManager::instance();
+  ObjectManager *objManager = ObjectManager::instance();
   objManager->setObjectFactoryFunction(ObjectFactory::produce, nullptr);
   objManager->initialize();
 
@@ -84,7 +80,5 @@ void initTask(void *parameters) {
   sif::printInfo("Creating tasks..\n\r");
 #endif
   InitMission::createTasks();
-	TaskFactory::instance()->deleteTask(nullptr);
+  TaskFactory::deleteTask(nullptr);
-
 }
-

bsp_stm32h7_freertos/utility/CMakeLists.txt

@@ -1,4 +1 @@
-target_sources(${TARGET_NAME} PRIVATE
+target_sources(${TARGET_NAME} PRIVATE print.c)
-	print.c
-)
-

bsp_stm32h7_freertos/utility/print.c

@@ -1,7 +1,6 @@
-#include <hardware_init.h>
 #include <boardconfig.h>
+#include <hardware_init.h>
 
-void printChar(const char* character) {
+void printChar(const char *character) {
-	HAL_UART_Transmit(&huart3, (uint8_t *)character, 1 , DEBUG_UART_MS_TIMEOUT);
+  HAL_UART_Transmit(&huart3, (const uint8_t *)character, 1, DEBUG_UART_MS_TIMEOUT);
 }
-

cmake/BuildType.cmake

@@ -1,45 +0,0 @@
-function(set_build_type)
-
-message(STATUS "Used build generator: ${CMAKE_GENERATOR}")
-
-# Set a default build type if none was specified
-set(DEFAULT_BUILD_TYPE "RelWithDebInfo")
-if(EXISTS "${CMAKE_SOURCE_DIR}/.git")
-	set(DEFAULT_BUILD_TYPE "Debug")
-endif()
-
-if(NOT CMAKE_BUILD_TYPE AND NOT CMAKE_CONFIGURATION_TYPES)
-	message(STATUS 
-		"Setting build type to '${DEFAULT_BUILD_TYPE}' as none was specified."
-	)
-	set(CMAKE_BUILD_TYPE "${DEFAULT_BUILD_TYPE}" CACHE
-		STRING "Choose the type of build." FORCE
-	)
-	# Set the possible values of build type for cmake-gui
-	set_property(CACHE CMAKE_BUILD_TYPE PROPERTY STRINGS
-		"Debug" "Release" "MinSizeRel" "RelWithDebInfo"
-	)
-endif()
-
-if(${CMAKE_BUILD_TYPE} MATCHES "Debug")
-	message(STATUS 
-		"Building Debug application with flags: ${CMAKE_C_FLAGS_DEBUG}"
-	)
-elseif(${CMAKE_BUILD_TYPE} MATCHES "RelWithDebInfo")
-	message(STATUS 
-		"Building Release (Debug) application with "
-		"flags: ${CMAKE_C_FLAGS_RELWITHDEBINFO}"
-	)
-elseif(${CMAKE_BUILD_TYPE} MATCHES "MinSizeRel")
-	message(STATUS 
-		"Building Release (Size) application with "
-		"flags: ${CMAKE_C_FLAGS_MINSIZEREL}"
-	)
-else()
-	message(STATUS 
-		"Building Release (Speed) application with "
-		"flags: ${CMAKE_C_FLAGS_RELEASE}"
-	)
-endif()
-
-endfunction()

cmake/scripts/make-debug-cfg.sh

@@ -1,33 +0,0 @@
-#!/bin/sh
-counter=0
-cfg_script_name="cmake-build-cfg.py"
-while [ ${counter} -lt 5 ]
-do
-    if [ -f ${cfg_script_name} ];then
-        break
-    fi
-    counter=$((counter=counter + 1))
-    cd ..
-done
-
-if [ "${counter}" -ge 5 ];then
-    echo "${cfg_script_name} not found in upper directories!"
-    exit 1
-fi
-
-build_generator=""
-os_fsfw="freertos"
-builddir="build-Debug"
-if [ "${OS}" = "Windows_NT" ]; then
-	build_generator="MinGW Makefiles"
-	python="py"
-# Could be other OS but this works for now.
-else
-	build_generator="Unix Makefiles"
-	python="python3"
-fi
-
-echo "Running command (without the leading +):"
-set -x # Print command 
-${python} ${cfg_script_name} -o "${os_fsfw}" -g "${build_generator}" -b "debug" -l "${builddir}"
-# set +x

cmake/scripts/make-release-cfg.sh

@@ -1,33 +0,0 @@
-#!/bin/sh
-counter=0
-cfg_script_name="cmake-build-cfg.py"
-while [ ${counter} -lt 5 ]
-do
-    if [ -f ${cfg_script_name} ];then
-        break
-    fi
-    counter=$((counter=counter + 1))
-    cd ..
-done
-
-if [ "${counter}" -ge 5 ];then
-    echo "${cfg_script_name} not found in upper directories!"
-    exit 1
-fi
-
-build_generator=""
-os_fsfw="freertos"
-builddir="build-Release"
-if [ "${OS}" = "Windows_NT" ]; then
-	build_generator="MinGW Makefiles"
-	python="py"
-# Could be other OS but this works for now.
-else
-	build_generator="Unix Makefiles"
-	python="python3"
-fi
-
-echo "Running command (without the leading +):"
-set -x # Print command 
-${python} ${cfg_script_name} -o "${os_fsfw}" -g "${build_generator}" -b "release" -l "${builddir}"
-# set +x

cmake/scripts/make-size-cfg.sh

@@ -1,33 +0,0 @@
-#!/bin/sh
-counter=0
-cfg_script_name="cmake-build-cfg.py"
-while [ ${counter} -lt 5 ]
-do
-    if [ -f ${cfg_script_name} ];then
-        break
-    fi
-    counter=$((counter=counter + 1))
-    cd ..
-done
-
-if [ "${counter}" -ge 5 ];then
-    echo "${cfg_script_name} not found in upper directories!"
-    exit 1
-fi
-
-build_generator=""
-os_fsfw="freertos"
-builddir="build-Release"
-if [ "${OS}" = "Windows_NT" ]; then
-	build_generator="MinGW Makefiles"
-	python="py"
-# Could be other OS but this works for now.
-else
-	build_generator="Unix Makefiles"
-	python="python3"
-fi
-
-echo "Running command (without the leading +):"
-set -x # Print command 
-${python} ${cfg_script_name} -o "${os_fsfw}" -g "${build_generator}" -b "size" -l "${builddir}"
-# set +x

cmake/scripts/ninja-debug-cfg.sh

@@ -1,32 +0,0 @@
-#!/bin/sh
-counter=0
-cfg_script_name="cmake-build-cfg.py"
-while [ ${counter} -lt 5 ]
-do
-    if [ -f ${cfg_script_name} ];then
-        break
-    fi
-    counter=$((counter=counter + 1))
-    cd ..
-done
-
-if [ "${counter}" -ge 5 ];then
-    echo "${cfg_script_name} not found in upper directories!"
-    exit 1
-fi
-
-build_generator=""
-os_fsfw="freertos"
-builddir="build-Debug"
-build_generator="Ninja"
-if [ "${OS}" = "Windows_NT" ]; then
-	python="py"
-# Could be other OS but this works for now.
-else
-	python="python3"
-fi
-
-echo "Running command (without the leading +):"
-set -x # Print command 
-${python} ${cfg_script_name} -o "${os_fsfw}" -g "${build_generator}" -b "debug" -l "${builddir}"
-# set +x

cmake/scripts/stm32h7-make-debug.sh

@@ -0,0 +1,42 @@
+#!/bin/sh
+counter=0
+init_dir=$(pwd)
+cfg_script_name="cmake-build-cfg.py"
+cfg_script=""
+if [ ! -z ${REPO_ROOT} ]; then
+    cfg_script=${REPO_ROOT}/cmake/${cfg_script_name}
+else
+    while [ ${counter} -lt 5 ]
+    do
+        if [ -f ${cfg_script_name} ];then
+            cfg_script=$(pwd)/${cfg_script_name}
+            break
+        fi
+        counter=$((counter=counter + 1))
+        cd ..
+    done
+
+    if [ "${counter}" -ge 5 ];then
+        echo "${cfg_script_name} not found in upper directories!"
+        cd ${init_dir}
+        exit 1
+    fi
+    cd ${init_dir}
+fi
+
+build_generator=""
+os_fsfw="freertos"
+builddir="cmake-build-debug"
+if [ "${OS}" = "Windows_NT" ]; then
+	build_generator="MinGW Makefiles"
+	python="py"
+# Could be other OS but this works for now.
+else
+	build_generator="Unix Makefiles"
+	python="python3"
+fi
+
+echo "Running command (without the leading +):"
+set -x # Print command 
+${python} ${cfg_script} -o "${os_fsfw}" -g "${build_generator}" -b "debug" -l "${builddir}"
+# set +x

cmake/scripts/stm32h7-make-release.sh

@@ -0,0 +1,42 @@
+#!/bin/sh
+counter=0
+init_dir=$(pwd)
+cfg_script_name="cmake-build-cfg.py"
+cfg_script=""
+if [ ! -z ${REPO_ROOT} ]; then
+    cfg_script=${REPO_ROOT}/cmake/${cfg_script_name}
+else
+    while [ ${counter} -lt 5 ]
+    do
+        if [ -f ${cfg_script_name} ];then
+            cfg_script=$(pwd)/${cfg_script_name}
+            break
+        fi
+        counter=$((counter=counter + 1))
+        cd ..
+    done
+
+    if [ "${counter}" -ge 5 ];then
+        echo "${cfg_script_name} not found in upper directories!"
+        cd ${init_dir}
+        exit 1
+    fi
+    cd ${init_dir}
+fi
+
+build_generator=""
+os_fsfw="freertos"
+builddir="cmake-build-release"
+if [ "${OS}" = "Windows_NT" ]; then
+    build_generator="MinGW Makefiles"
+    python="py"
+# Could be other OS but this works for now.
+else
+    build_generator="Unix Makefiles"
+    python="python3"
+fi
+
+echo "Running command (without the leading +):"
+set -x # Print command 
+${python} ${cfg_script} -o "${os_fsfw}" -g "${build_generator}" -b "release" -l "${builddir}"
+# set +x

cmake/scripts/stm32h7-make-size.sh

@@ -0,0 +1,42 @@
+#!/bin/sh
+counter=0
+init_dir=$(pwd)
+cfg_script_name="cmake-build-cfg.py"
+cfg_script=""
+if [ ! -z ${REPO_ROOT} ]; then
+    cfg_script=${REPO_ROOT}/cmake/${cfg_script_name}
+else
+    while [ ${counter} -lt 5 ]
+    do
+        if [ -f ${cfg_script_name} ];then
+            cfg_script=$(pwd)/${cfg_script_name}
+            break
+        fi
+        counter=$((counter=counter + 1))
+        cd ..
+    done
+
+    if [ "${counter}" -ge 5 ];then
+        echo "${cfg_script_name} not found in upper directories!"
+        cd ${init_dir}
+        exit 1
+    fi
+    cd ${init_dir}
+fi
+
+build_generator=""
+os_fsfw="freertos"
+builddir="cmake-build-size"
+if [ "${OS}" = "Windows_NT" ]; then
+    build_generator="MinGW Makefiles"
+    python="py"
+# Could be other OS but this works for now.
+else
+    build_generator="Unix Makefiles"
+    python="python3"
+fi
+
+echo "Running command (without the leading +):"
+set -x # Print command 
+${python} ${cfg_script} -o "${os_fsfw}" -g "${build_generator}" -b "size" -l "${builddir}"
+# set +x

cmake/scripts/stm32h7-ninja-debug.sh

@@ -0,0 +1,41 @@
+#!/bin/sh
+counter=0
+init_dir=$(pwd)
+cfg_script_name="cmake-build-cfg.py"
+cfg_script=""
+if [ ! -z ${REPO_ROOT} ]; then
+    cfg_script=${REPO_ROOT}/cmake/${cfg_script_name}
+else
+    while [ ${counter} -lt 5 ]
+    do
+        if [ -f ${cfg_script_name} ];then
+            cfg_script=$(pwd)/${cfg_script_name}
+            break
+        fi
+        counter=$((counter=counter + 1))
+        cd ..
+    done
+
+    if [ "${counter}" -ge 5 ];then
+        echo "${cfg_script_name} not found in upper directories!"
+        cd ${init_dir}
+        exit 1
+    fi
+    cd ${init_dir}
+fi
+
+build_generator=""
+os_fsfw="freertos"
+builddir="cmake-build-debug"
+build_generator="Ninja"
+if [ "${OS}" = "Windows_NT" ]; then
+    python="py"
+# Could be other OS but this works for now.
+else
+    python="python3"
+fi
+
+echo "Running command (without the leading +):"
+set -x # Print command 
+${python} ${cfg_script} -o "${os_fsfw}" -g "${build_generator}" -b "debug" -l "${builddir}"
+# set +x

cmake/stm32-cmake

@@ -1 +1 @@
-Subproject commit e3a834d8409c394136af43bdf8c084941ed44b53
+Subproject commit 5ae35e844c2b223f7bdacdf003f61791174848c6

example_common

@@ -1 +1 @@
-Subproject commit 797801ffde3defc48e5f3f69df31f36e5d3879c5
+Subproject commit c564fa37fde467a0216c041bb81eeed7343ac505

fsfw

@@ -1 +1 @@
-Subproject commit 7c2e50b665f515d6234f3d2f070609f8f37efde2
+Subproject commit 678f8710c6656c70c56f4ea6e0ee6c0195052bff

generators/fsfwgen

@@ -1 +1 @@
-Subproject commit e6f5ff18129984b591a6a286968b1ace42c278b2
+Subproject commit 6d423f7106e49f93743fb69e9436e1e652f9e001

load-path.sh

@@ -1,2 +1,10 @@
 #!/bin/bash
+if [ ! -d bsp_stm32h7_freertos ]; then
+	echo "This script should be sourced inside the example repository"
+	exit 1
+fi
+
 export PATH="$(pwd)/xpacks/.bin":$PATH
+export PATH="$(pwd)/cmake/scripts:$PATH"
+
+export REPO_ROOT="$(pwd)"

scripts/auto-formatter.sh

@@ -0,0 +1,32 @@
+#!/bin/bash
+if [[ ! -f README.md ]]; then
+  cd ..
+fi
+
+folder_list=(
+  "./bsp_stm32h7_freertos"
+  "./example_common"
+)
+
+cmake_fmt="cmake-format"
+file_selectors="-iname CMakeLists.txt"
+if command -v ${cmake_fmt} &> /dev/null; then
+  echo "Auto-formatting all CMakeLists.txt files"
+  ${cmake_fmt} -i CMakeLists.txt
+  for dir in ${folder_list[@]}; do
+    find ${dir} ${file_selectors} | xargs ${cmake_fmt} -i
+  done
+else
+  echo "No ${cmake_fmt} tool found, not formatting CMake files"
+fi
+
+cpp_format="clang-format"
+file_selectors="-iname *.h -o -iname *.cpp -o -iname *.c -o -iname *.tpp"
+if command -v ${cpp_format} &> /dev/null; then
+  for dir in ${folder_list[@]}; do
+    echo "Auto-formatting C/C++ files in ${dir} recursively"
+    find ${dir} ${file_selectors} | xargs ${cpp_format} --style=file -i
+  done
+else
+  echo "No ${cpp_format} tool found, not formatting C++/C files"
+fi

tmtc/.gitignore

@@ -0,0 +1 @@
+/tmtc_conf.json

tmtc/.idea/runConfigurations/tmtccli.xml

@@ -0,0 +1,24 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="false" name="tmtccli" type="PythonConfigurationType" factoryName="Python" nameIsGenerated="true">
+    <module name="tmtc" />
+    <option name="INTERPRETER_OPTIONS" value="" />
+    <option name="PARENT_ENVS" value="true" />
+    <envs>
+      <env name="PYTHONUNBUFFERED" value="1" />
+    </envs>
+    <option name="SDK_HOME" value="" />
+    <option name="WORKING_DIRECTORY" value="$PROJECT_DIR$" />
+    <option name="IS_MODULE_SDK" value="true" />
+    <option name="ADD_CONTENT_ROOTS" value="true" />
+    <option name="ADD_SOURCE_ROOTS" value="true" />
+    <EXTENSION ID="PythonCoverageRunConfigurationExtension" runner="coverage.py" />
+    <option name="SCRIPT_NAME" value="$PROJECT_DIR$/tmtccli.py" />
+    <option name="PARAMETERS" value="" />
+    <option name="SHOW_COMMAND_LINE" value="false" />
+    <option name="EMULATE_TERMINAL" value="true" />
+    <option name="MODULE_MODE" value="false" />
+    <option name="REDIRECT_INPUT" value="false" />
+    <option name="INPUT_FILE" value="" />
+    <method v="2" />
+  </configuration>
+</component>

tmtc/common_tmtc

@@ -1 +1 @@
-Subproject commit 99ca187b50ad9cd6ee1ba4bcbf9db981bed653ae
+Subproject commit 3cca54f66fe72f292787499e1e9bf2497a2cfdbf

tmtc/config/hook.py

@@ -0,0 +1,11 @@
+from common_tmtc.config.hook_implementation import CommonFsfwHookBase
+from tmtccmd.tc.definitions import TcQueueT
+
+
+class FsfwHookBase(CommonFsfwHookBase):
+    def pack_service_queue(self, service: int, op_code: str, service_queue: TcQueueT):
+        from common_tmtc.pus_tc.tc_packing import common_service_queue_user
+
+        common_service_queue_user(
+            service=service, op_code=op_code, tc_queue=service_queue
+        )

tmtc/def_tmtc_conf.json

@@ -0,0 +1,4 @@
+{
+    "com_if": "udp",
+    "tcpip_udp_recv_max_size": 1500
+}

tmtc/tmtc_client_cli.py

@@ -1,56 +0,0 @@
-#!/usr/bin/env python3
-"""
-@brief      TMTC Commander entry point for command line mode.
-@details
-This client was developed by KSat for the SOURCE project to test the on-board software but
-has evolved into a more generic tool for satellite developers to perform TMTC (Telemetry and Telecommand)
-handling and testing via different communication interfaces. Currently, only the PUS standard is
-implemented as a packet standard.
-
-Run this file with the -h flag to display options.
-
-@license
-Copyright 2020 KSat e.V. Stuttgart
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-   https://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-
-@author     R. Mueller
-"""
-import sys
-
-from common_tmtc.config.hook_implementation import FsfwHookBase
-from common_tmtc.config.definitions import PUS_APID
-from common_tmtc.pus_tm.factory_hook import ccsds_tm_handler
-try:
-    from tmtccmd.runner import run_tmtc_commander, initialize_tmtc_commander, add_ccsds_handler
-    from tmtccmd.ccsds.handler import CcsdsTmHandler
-except ImportError as error:
-    run_tmtc_commander = None
-    initialize_tmtc_commander = None
-    print(error)
-    print("Python tmtccmd submodule could not be imported")
-    print("Install with \"cd tmtccmd && python3 -m pip install -e .\" for interactive installation")
-    sys.exit(0)
-
-
-def main():
-    hook_obj = FsfwHookBase()
-    initialize_tmtc_commander(hook_object=hook_obj)
-    ccsds_handler = CcsdsTmHandler()
-    ccsds_handler.add_tm_handler(apid=PUS_APID, pus_tm_handler=ccsds_tm_handler, max_queue_len=50)
-    add_ccsds_handler(ccsds_handler)
-    run_tmtc_commander(use_gui=False, app_name="TMTC Commander FSFW")
-
-
-if __name__ == "__main__":
-    main()

tmtc/tmtc_client_gui.py

@@ -1,56 +0,0 @@
-#!/usr/bin/env python3
-"""
-@brief      TMTC Commander entry point for command line mode.
-@details
-This client was developed by KSat for the SOURCE project to test the on-board software but
-has evolved into a more generic tool for satellite developers to perform TMTC (Telemetry and Telecommand)
-handling and testing via different communication interfaces. Currently, only the PUS standard is
-implemented as a packet standard.
-
-Run this file with the -h flag to display options.
-
-@license
-Copyright 2020 KSat e.V. Stuttgart
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-   https://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-
-@author     R. Mueller
-"""
-import sys
-
-from common_tmtc.config.hook_implementation import FsfwHookBase
-from common_tmtc.config.definitions import PUS_APID
-from common_tmtc.pus_tm.factory_hook import ccsds_tm_handler
-try:
-    from tmtccmd.runner import run_tmtc_commander, initialize_tmtc_commander, add_ccsds_handler
-    from tmtccmd.ccsds.handler import CcsdsTmHandler
-except ImportError as error:
-    run_tmtc_commander = None
-    initialize_tmtc_commander = None
-    print(error)
-    print("Python tmtccmd submodule could not be imported")
-    print("Install with \"cd tmtccmd && python3 -m pip install -e .\" for interactive installation")
-    sys.exit(0)
-
-
-def main():
-    hook_obj = FsfwHookBase()
-    initialize_tmtc_commander(hook_object=hook_obj)
-    ccsds_handler = CcsdsTmHandler()
-    ccsds_handler.add_tm_handler(apid=PUS_APID, pus_tm_handler=ccsds_tm_handler, max_queue_len=50)
-    add_ccsds_handler(ccsds_handler)
-    run_tmtc_commander(use_gui=True, app_name="TMTC Commander FSFW")
-
-
-if __name__ == "__main__":
-    main()

tmtc/tmtccli.py

@@ -0,0 +1,23 @@
+#!/usr/bin/env python3
+"""TMTC commander for FSFW Example"""
+from common_tmtc.tmtcc import (
+    tmtcc_post_args,
+    tmtcc_pre_args,
+    create_default_args_parser,
+    add_default_tmtccmd_args,
+    parse_default_input_arguments,
+)
+from config.hook import FsfwHookBase
+
+
+def main():
+    tmtcc_pre_args()
+    hook_obj = FsfwHookBase(json_cfg_path="tmtc_conf.json")
+    arg_parser = create_default_args_parser()
+    add_default_tmtccmd_args(arg_parser)
+    args = parse_default_input_arguments(arg_parser, hook_obj)
+    tmtcc_post_args(hook_obj=hook_obj, use_gui=False, args=args)
+
+
+if __name__ == "__main__":
+    main()

tmtc/tmtccmd

@@ -1 +1 @@
-Subproject commit 4d71822fa1f1cfc16ccd4d0fb750eb2900a263b2
+Subproject commit 2354f5d2778c7681ddf6602766dd4e68b943e1c0

tmtc/tmtcgui.py

@@ -0,0 +1,12 @@
+#!/usr/bin/env python3
+"""TMTC commander for the FSFW Example"""
+from common_tmtc.tmtcc import tmtcc_post_args, tmtcc_pre_args
+
+
+def main():
+    hook_obj = tmtcc_pre_args()
+    tmtcc_post_args(hook_obj=hook_obj, use_gui=True, args=None)
+
+
+if __name__ == "__main__":
+    main()