search

mysensors / MySensors

MySensors library and examples
https://www.mysensors.org
1.32k stars 895 forks source link

Teensy with RFM95 does not go to sleep #891

Open micropet opened 7 years ago

micropet commented 7 years ago

The command sleep (xx) does not work with the Teensy LC 3.2 / 3.6. It is simply ignored.

trlafleur commented 7 years ago

This is the same issues with the M0 processors.
Sleep, WDT are not yet implemented.....

See issue 715

micropet commented 7 years ago

OK thanks. So that means wait ...

The Arduino Pro has too little memory and it would be nice to upgrade to more powerful CPUs (ESP32, SAMD21, Teensy).

trlafleur commented 7 years ago

No, you can just implement the low power mode in your code.....

See code from rocketscream and others.... do a quick search....

~~ /) ~~~~ /) ~~ _/) ~~ _/) ~~

Tom Lafleur

On Jul 11, 2017, at 12:31 AM, micropet notifications@github.com wrote:

OK thanks. So that means wait ...

The Arduino Pro has too little memory and it would be nice to upgrade to more powerful CPUs (ESP32, SAMD21, Teensy).

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

micropet commented 7 years ago

Yes Tom , I did. And the Teensys goes to sleep with Snooze.h.

But it would be better if the basic functions are in the MySensors lib.

Peter

trlafleur commented 7 years ago

Yes I agree , but the primary developer are still in the 8 bit world. So it's do it yourself or wait. Zero, M0 was added well over a year ago and still no direct support for sleep or WDT support.

~~ /) ~~~~ /) ~~ _/) ~~ _/) ~~

On Jul 11, 2017, at 10:00 PM, micropet notifications@github.com wrote:

Yes Tom , I did. At the Teensys it goes with Snooze.h.

But it would be better if the basic functions are in the MySensors lib.

Peter

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

scalz commented 7 years ago

@trlafleur said "the primary developer are still in the 8 bit world. So it's do it yourself or wait. Zero, M0 was added well over a year ago and still no direct support for sleep or WDT support."

Anthological! Hello !??

Do you consider yourself as a 32bits dev? because of using some libs with atsam?? why don't you help, instead of complains?? I don't understand this kind of frustration.. We have not all same priorities, simple ;)

This is not kind to behave like this whereas Core Team is using lot of his free time to feed users for free..

@micropet Regarding this issue, i'll (or someone else in Core Team) take a look later, as it's not in my top priorities for the moment, unfortunately, i'm sorry.

micropet commented 7 years ago

@scalz OK, thank you for your efforts.

trlafleur commented 7 years ago

It was NOT my intent to complain, but to clarified that the support for some processors are missing some features. Nothing more.

I do understand that the developers do have priority in their life's and on this project... again, it was NOT my intent to criticise their effort... I greatly appreciate the effort of the developers, its a great product and has a great future.

Thank you for your effort.

d00616 commented 7 years ago

The sleep mode and WDT are implemented for nRF51(M0) and nRF52(M4) MCUs. Adding these functionality for all Cortex CPUs isn't simple because there are a lot of differences between Cortex-M MCUs.

Often the Arduino Ports are at an early stage like the arduino-nRF5 missing some functionality like WDT, but things becoming more and more stable.