-
-
Notifications
You must be signed in to change notification settings - Fork 348
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RTOS SDK based Sming #495
Comments
@hreintke i will look it. where is task implementation |
Task implementation is "just plain" rtos. As it is now Sming is running in main task and doesn't use any other. Please focus first on porting current sming functionality before introducing new. |
@hreintke second question is can we use app/framework things like main sming with nonos sdk? i think its important for adding new examples |
@alonewolfx2 : |
Maybe this is stupid question, but what do you mean by this? |
In sming you build framework -> libsming.a -> building application using lib -> flash |
On right, that doesn't sound like a big problem. I'm away at the moment, then will be working all over Christmas :-( but I'll have a look when I get chance. Sounds interesting! |
No, not a big problem but just one of the things (makefiles etc) where I do not have much experience. |
@hreintke why not just create another RTOS branch and then implement selector which sdk to use? |
@kireevco : I have been thinking of that but So if we want to move/support RTOS SDK. Let's take the step now in a one shot. If someone really wants to stick to NONOS SDK, he/she will keep current functionality but no further upgrades. |
+1 2015-12-22 8:48 GMT+01:00 hreintke notifications@github.com:
|
I'm not sure if it is a safe step. I would keep both sdks until rtos will be confirmed stable. That's my opinion. Sent from my iPhone
|
so this is only for windows right? On Tue, Dec 22, 2015 at 10:12 AM, Dmitry Kireev notifications@github.com
|
What is necesary is a "working development environment" for Sming. That is available for mac.
And you should be up and running |
@kireevco : My statement is : When Sming RTOS is stable, we should cease NONOS SDK support |
i think too early for cease nonos sdk. espressif will support nonos sdk until last quaerter on 2016. |
@alonewolfx2 I'm with @hreintke. |
@hreintke let's use https://github.com/SmingHub/SmingRTOS if you will be using a separate repo. But bear in mind, it's splitting efforts and we still need to find one to focus one, and if that would be RTOS, then ceasing support for NONOS would be the only way to continue with good quality code. |
@kireevco : Yes sure that is the idea but... |
Work in progress in https://github.com/hreintke/Sming_RTOS_POC.git |
Related to #553 I think we should continue this discussion. Many changes in RTOS is a breaking changes. Will be right to make decisions about new arctitecture and choose optimal solutions. Some most actual questions:
RTOS threads provide many benifits and simplification for core and user code. When I worked on Smig core development, I have made vary many solutions which can be omitted in multithread environment. Now we should find the best compromise between new features with large rewriting and backwards compatibility. |
i think we should. But step by step. firstly stable code
every delegates can work on new thread. so any delegates doesnt break any wifi or other operations
Old libraries and codes almost full compatible with old nonos sdk. it needs a couple adding or deleting. so we can continue on rtos sdk. espressif will continue with rtos sdk and leave nonos sdk in 2016. so if we want to keep allive sming on esp32 (i want to it) we cant use nonos sdk anymore. |
Work in progress in Sming_RTOS_POC |
@ALL :
I have been working on a Proof of Concept of the RTOS SDK based Sming.
Reason is the move from Espressif towards RTOS SDK, also for the upcoming ESP32.
Result is in my https://github.com/hreintke/Sming_RTOS_POC.git repository.
More details in the "README"
Have a look, test, feedback and start helping to get to an Alpha version.
Disclaimer :
The text was updated successfully, but these errors were encountered: