Update Miosix Kernel #239
Open
Update Miosix Kernel #239
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I tried flashing my RT3s radio (with a copycat MCU). It compiles and flashes alright, but it doesn't boot. I explored a bit, and I find that there probably is a problem with miosix. In particular, the function For now I don't have many clues on why it could be. |
|
I wrote a preliminary implementation for MK22 MCUs. It is untested as I don't have a GD-77 or MD-9600. |
De-clutters the architecture
Accounting for new arch structure and new kernel files
Fixed a bad cast warning in kernel.cpp Fixed a warning with variadic macros in logging.h Fixed invalid type warning in util.cpp
This fixes a problem where the time would pass 11.904 times too slow. It turns out that the OS used an uninitialized TimeConversion instance. This commit fixes that, though I don't really know why this fix is even needed.
Preliminary implementation for NXP MK22 MCU series. Tested.
|
I rebased the branch on the current Master branch. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While implementing user functions (functions that user code can register in OpenRTX to be called periodically or asynchronously to avoid creating a thread for minor functionalities) I realized that the function
pthread_cond_timedwaitis needed.It looks like that function is available in recent versions of miosix. Updating the kernel would allow to implement user functions. It is also probably gonna be needed for OpenHT.
This PR updates the miosix kernel, then fixes the code for Module17. This PR does not work for all targets yet.
One of the main problem that was found in the update process is that the default miosix implementation for the system timer (TIM5) uses an uninitialized TimeConversion instance. Commit 86fce68 fixes it.
As @silseva noted, a system timer implementation needs to be implemented for the MK22 processor.