You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Description:
The current setup of the MCU in the setupMcu function could be enhanced by providing more readability, flexibility and configurability. Instead of using defines in the CMakeLists to enable or disable the low-frequency watch quartz functionalities, these could be passed as function parameters to setupMcu. This would allow for easier and more dynamic configuration changes. Additionally, the implementation should be designed to easily add other options in the future. For clarity and readability, numerical values should be represented using defines or enums.
Furthermore, setting up stdout is a common requirement, often used for unit testing or debugging. Even if it's not utilized in the final version of the software, it's beneficial during the development phase. Therefore, integrating the setup of stdout with the uart_helper into the setupMcu function as an option or a separate function would be advantageous.
User Story:
As a developer, I need the setupMcu function to accept configuration options such as enabling or disabling low-frequency watch quartz functionalities, and to set up stdout using the uart_helper during MCU setup for simpler debugging and testing.
The text was updated successfully, but these errors were encountered:
Description:
The current setup of the MCU in the setupMcu function could be enhanced by providing more readability, flexibility and configurability. Instead of using defines in the CMakeLists to enable or disable the low-frequency watch quartz functionalities, these could be passed as function parameters to setupMcu. This would allow for easier and more dynamic configuration changes. Additionally, the implementation should be designed to easily add other options in the future. For clarity and readability, numerical values should be represented using defines or enums.
Furthermore, setting up stdout is a common requirement, often used for unit testing or debugging. Even if it's not utilized in the final version of the software, it's beneficial during the development phase. Therefore, integrating the setup of stdout with the uart_helper into the setupMcu function as an option or a separate function would be advantageous.
User Story:
As a developer, I need the setupMcu function to accept configuration options such as enabling or disabling low-frequency watch quartz functionalities, and to set up stdout using the uart_helper during MCU setup for simpler debugging and testing.
The text was updated successfully, but these errors were encountered: