IngmarEckhardt / IngmarEckhardt/dwarfOS
setup with options
- Dominant language
- C
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
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.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at the setupMcu function and inspect the CMakeLists definitions controlling the low-frequency watch quartz, along with the uart_helper used for stdout setup. Decide how the configuration options and future options should be represented, then verify that MCU initialization, optional quartz behavior, and stdout setup work as intended.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c, cmake
- Domain
- build-system, embedded-iot
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100