Conversation
…head_flight_sw added the include in the cmakelists.txt
| @@ -0,0 +1,115 @@ | |||
| /* STM32H753ZI Memory Layout */ | |||
There was a problem hiding this comment.
Was this generated from cubeMX? it should have some setup for the board so we can use all the flash
There was a problem hiding this comment.
Nah this one was not
| endif() | ||
| target_link_libraries(stm_hal | ||
| PRIVATE cmsis | ||
| PUBLIC cmsis |
There was a problem hiding this comment.
No clue I think something else uses cmsis thats why, probably should clean up the build more
| To flash, (only available for MCU) run: | ||
| ```bash | ||
| cmake -B buuld -DTARGET=<TARGET> | ||
| cmake --build build --target flash |
There was a problem hiding this comment.
i think we should separate build and flash.
we can cmake --build build as usual, then ./flash.sh <build output> @CBL17 thoughts?
| add_subdirectory(drivers) | ||
| add_subdirectory(hal) | ||
| else() | ||
| add_subdirectory(simulations) |
There was a problem hiding this comment.
By simulations it means simulated drivers
| add_subdirectory(simulations) | ||
| endif() | ||
|
|
||
| find_program(OPENOCD openocd) |
There was a problem hiding this comment.
in the else case, can you print a warning that openocd is not available?
the reason for my below comment on making build and flash separate is that if openocd is not found, there is a silent failure and flash is not available. then, it's unclear for users why the flash fails.
| vTaskDelayUntil(&xLastWakeTime, pdMS_TO_TICKS(1000)); | ||
|
|
||
| auto time = static_cast<long long>(std::chrono::system_clock::now().time_since_epoch().count()); | ||
| printf("[%lld] Hello world!\n", time); |
There was a problem hiding this comment.
wouldn't be opposed to leaving printf in, but i think we'll have to setup uart hal separately.
2 participants
No description provided.