Re: [riot-devel] LPM idle_thread

2015-05-10 Thread Oleg Hahm
Hi! DEVELHELP should not be used to change the semantics of things. Can you elaborate? The system should behave the same whether develhelp is used or not. I would claim that this is impossible. Even a simple puts() already affects the timings, calls peripheral drivers, and may cause

Re: [riot-devel] LPM idle_thread

2015-05-10 Thread Oleg Hahm
Hi Kaspar! I would claim that this is impossible. Even a simple puts() already affects the timings, calls peripheral drivers, and may cause interrupts. the same was bad wording. I meant semantically, like Martine initially stated. I guess you mean Ludwig. I just wanted to point out that

Re: [riot-devel] LPM idle_thread

2015-05-10 Thread Kaspar Schleiser
Hi, On 05/10/15 09:34, Oleg Hahm wrote: DEVELHELP should not be used to change the semantics of things. Can you elaborate? The system should behave the same whether develhelp is used or not. I would claim that this is impossible. Even a simple puts() already affects the timings, calls

Re: [riot-devel] LPM idle_thread

2015-05-10 Thread Kaspar Schleiser
Hi, On 05/10/15 10:38, Oleg Hahm wrote: I would claim that this is impossible. Even a simple puts() already affects the timings, calls peripheral drivers, and may cause interrupts. the same was bad wording. I meant semantically, like Martine initially stated. I guess you mean Ludwig. Yes,

Re: [riot-devel] LPM idle_thread

2015-05-10 Thread Kaspar Schleiser
Hi, On 05/09/15 20:24, Oleg Hahm wrote: DEVELHELP should not be used to change the semantics of things. Can you elaborate? The system should behave the same whether develhelp is used or not. If you do something like #ifdef DEVELHELP if(input_not_correct) { print(error message);

[riot-devel] LPM idle_thread

2015-05-09 Thread Frank Holtz
Hello, why is LPM_SLEEP or LPM_POWERDOWN is commented out? kernel_init.c: --- static void *idle_thread(void *arg) { (void) arg; while (1) { if (lpm_prevent_sleep) { lpm_set(LPM_IDLE); } else {

Re: [riot-devel] LPM idle_thread

2015-05-09 Thread Oleg Hahm
Hi Frank! why is LPM_SLEEP or LPM_POWERDOWN is commented out? I think it's mostly commented out for debugging. Debugging with power save modes can become really nasty (even printf() debugging). However, I agree this shouldn't be solved like this. Another case for DEVELHELP, maybe? Cheers,

Re: [riot-devel] LPM idle_thread

2015-05-09 Thread Ludwig Ortmann
Hi, DEVELHELP should not be used to change the semantics of things. Cheers, Ludwig Am 9. Mai 2015 18:43:36 MESZ, schrieb Oleg Hahm oliver.h...@inria.fr: Hi Frank! why is LPM_SLEEP or LPM_POWERDOWN is commented out? I think it's mostly commented out for debugging. Debugging with power save