thank You @manutremo

My problem was caused by the fact that the radio was inserted into a socket. The 4.7uF stabilizing capacitor was put on the main board side of the socket. Moving the capacitor to the radio board or just soldering the radio to the main board (without using a socket) solved the problem. So, don't use a socket between the radio and the stabilizing capacitor.

Another interesting thing is that by putting a finger onto the antenna also solved the problem. I could use that method for testing all my radios (using the socket) before permanently soldering them onto the main board.

And here is a 5th similar issue I believe: https://forum.mysensors.org/topic/6147/tsm-fpar-no-reply

@gloob
Thanks, I will follow Your thread. The problem looks similar.

Hi there!

My new NRF24L01+ modules arrived today and I'm trying to test them.

I had an unknown device listed in my Domoticz MySensors Gateway with USB so I deleted it in the Domoticz GUI. I thought it would be able to re-register with the Domoticz gateway but it fails. At least that's what I think. Anyway, I'm not able to get it back in the Domoticz device listing. I've restarted the gateway and tried the inclusion button etc but nothing seems to help. Below is the serial monitor debugging info. Any ideas?

0 MCO:BGN:INIT NODE,CP=RNNNA--,VER=2.1.1
32 TSM:INIT
49 TSF:WUR:MS=0
81 TSM:INIT:TSP OK
98 TSF:SID:OK,ID=10
114 TSM:FPAR
180 TSF:MSG:SEND,10-10-255-255,s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=OK:
2277 !TSM:FPAR:NO REPLY
2293 TSM:FPAR
2359 TSF:MSG:SEND,10-10-255-255,s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=OK:
4456 !TSM:FPAR:NO REPLY
4472 TSM:FPAR
4538 TSF:MSG:SEND,10-10-255-255,s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=OK:
6635 !TSM:FPAR:NO REPLY
6651 TSM:FPAR
6717 TSF:MSG:SEND,10-10-255-255,s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=OK:
8814 !TSM:FPAR:FAIL
8830 TSM:FAIL:CNT=1
8847 TSM:FAIL:PDT

@Nca78

I checked the board. Pin 8 IRQ of the NRF24L01+ module is not connected.

Hi guys!

In the example above, it would be more appropriate using millis() in a loop than using sleep(50). It's also kind of wrong to use delay(50) I believe since it's taking up 100% of the CPU.

Never mind. I'm just interested to understand how the sleep(50) inside the main loop would cause the "endless loop of interrupts" in my example. Actually I suspect it's a bug so it might be worth digging into it a bit.

I've been experimenting with both strong and weak external pull-ups and I've also tried internal pull ups but it made no difference actually. I'm currently using an external resistor of 4,7M Ohm and it's working great (unless using sleep(50) inside the main loop of course).

Hardware used is a Slim Node I never tried using a "pull down" instead of a "pull up" resistor.

Hi there!

I feel I need to share this and I invite You to reflect upon this and share your ideas here.

I've been working on a battery powered node with 2 push buttons. Normally it sleeps but it will wake up using two interrupts when one of the push buttons gets pressed.
interruptReturn = sleep(PRIMARY_BUTTON_PIN-2, CHANGE, SECONDARY_BUTTON_PIN-2, CHANGE, ONE_DAY_SLEEP_TIME);

I've spent many hours trying to figure out how to make this script work. I had problem with that once a push button was pressed it was generating and endless loop of interrupts. It never slept.

It sounds weird but using the following line

sleep(50); // Short delay to allow switch to properly settle

instead of

delay(50); // Short delay to allow switch to properly settle

caused the never ending interrupt loop.

Below is my final script.

#define MY_BAUD_RATE 9600
#define MY_RADIO_NRF24

#include <SPI.h>
#include <MySensors.h>
#include <Vcc.h>
const float VccMin   = 1.8;           // Minimum expected Vcc level, in Volts.
const float VccMax   = 3.2;           // Maximum expected Vcc level, in Volts.
const float VccCorrection = 1.0/1.0;  // Measured Vcc by multimeter divided by reported Vcc
Vcc vcc(VccCorrection);

#define NODE_ID 17
#define SKETCH_NAME "2x Binary Switch Sleep"
#define SKETCH_VERSION "3.2"

#define CHILD_ID_1 0
#define CHILD_ID_2 1
#define PRIMARY_BUTTON_PIN 2 // Arduino Digital I/O pin for button/reed switch
#define SECONDARY_BUTTON_PIN 3 // Arduino Digital I/O pin for button/reed switch

#define BATTERY_REPORT_DAY 2
#define BATTERY_REPORT_BY_IRT_CYCLE 10 // Adjust this according to usage frequency.
#define ONE_DAY_SLEEP_TIME 86400000 // No wakeup timer at all

int oldValue1=-1;
int oldValue2=-1;
int dayCounter = BATTERY_REPORT_DAY;
int irtCounter = 0;
int interruptReturn = MY_WAKE_UP_BY_TIMER;

MyMessage msg1(CHILD_ID_1,V_TRIPPED);
MyMessage msg2(CHILD_ID_2,V_TRIPPED);

void setup()	
{	
	pinMode(PRIMARY_BUTTON_PIN,INPUT);
	pinMode(SECONDARY_BUTTON_PIN,INPUT);
	// We use external and very weak pull ups to save power
	//digitalWrite(PRIMARY_BUTTON_PIN, HIGH);    // Enable internal pull-up
	//digitalWrite(SECONDARY_BUTTON_PIN, HIGH);    // Enable internal pull-up
}

void presentation() {
	sendSketchInfo(SKETCH_NAME, SKETCH_VERSION);
	present(CHILD_ID_1, S_DOOR);
	present(CHILD_ID_2, S_DOOR);
}

void loop() 
{
	if (interruptReturn == MY_WAKE_UP_BY_TIMER) { // Woke up by timer (or first run)
		dayCounter++;
		#ifdef MY_DEBUG
		Serial.println("Woke up by timer (or first run)");
		#endif
		if (dayCounter >= BATTERY_REPORT_DAY) {
			dayCounter = 0;
			sendBatteryReport();
		}
	} else {    // Woke up by pin change
		irtCounter++;
		#ifdef MY_DEBUG
		Serial.print("Woke up by pin change on pin D");Serial.println(interruptReturn+2);
		#endif
		if (interruptReturn == 0) {
			delay(50); // Short delay to allow switch to properly settle
			int value1 = digitalRead(PRIMARY_BUTTON_PIN);
			if (value1 != oldValue1) {
				send(msg1.set(value1==HIGH ? 0 : 1));
				oldValue1 = value1;
				Serial.print("Sent value: '"); Serial.print(value1==HIGH ? 0 : 1); Serial.println("'.");
			}
		} else {
			delay(50); // Short delay to allow switch to properly settle
			int value2 = digitalRead(SECONDARY_BUTTON_PIN);
			if (value2 != oldValue2) {
				send(msg2.set(value2==HIGH ? 0 : 1));
				oldValue2 = value2;
				Serial.print("Sent value: '"); Serial.print(value2==HIGH ? 0 : 1); Serial.println("'.");
			}
		}
	}
	if (irtCounter>=BATTERY_REPORT_BY_IRT_CYCLE) {
		irtCounter = 0;
		sendBatteryReport();
	}
	#ifdef MY_DEBUG
	Serial.println("Zzzzzz");
	#endif
	delay(500); //delay to allow serial to fully print before sleep
	// Sleep until something happens with the sensor, or one sleep_time has passed since last awake.
	interruptReturn = sleep(PRIMARY_BUTTON_PIN-2, CHANGE, SECONDARY_BUTTON_PIN-2, CHANGE, ONE_DAY_SLEEP_TIME);
}

void sendBatteryReport() {
	float v = vcc.Read_Volts();
	float p = vcc.Read_Perc(VccMin, VccMax);
	#ifdef MY_DEBUG
	Serial.print("VCC = "); Serial.print(v); Serial.println(" Volts");
	Serial.print("VCC = "); Serial.print(p); Serial.println(" %");
	#endif
	int batteryPcnt = static_cast<int>(p);
	sendBatteryLevel(batteryPcnt);
}

The script above works now. (Arduino 1.6.13 and MySensors 2.1.0). It would be nice though to know why I can't use sleep(50) instead of delay(50) in my code. Any ideas?

Cheers!

@Nca78 said:

@รอเรือ said:

I decided to test all my 20 sensors and found out that 3 out of 20 actually could power down. Starting the tedious work to un-solder the radios of the remaining sensors now.

Using the link in the store, I ordered 30 new from Ebay http://www.ebay.com/itm/400594940658?rmvSB=true

This vendor state that their radios will consume 1 uA in sleep mode. I will test all of them at arrival. I believe it's a bit more safe to buy radios on Ebay because it's easy to complain if something shouldn't work as expected.

Cheers!

Is it too late for you to open a dispute ? With pictures of your multimeter showing current for "normal" module and current for "failing" module + text explanations, it should be possible to get most of your money back.
Also, put an evaluation about the radio. If like many people you put many stars before testing the product, you can still add additionnal feedback, that will be useful for the next users who will be tempted to buy from this seller.

Hi! Unfortunately I did my review before testing them in sleep mode. I won't do that next time for sure. I will see if I can make an additional review. I have no hope to regain any money from them though. Thanks for the tip.

I decided to test all my 20 sensors and found out that 3 out of 20 actually could power down. Starting the tedious work to un-solder the radios of the remaining sensors now.

Using the link in the store, I ordered 30 new from Ebay http://www.ebay.com/itm/400594940658?rmvSB=true

This vendor state that their radios will consume 1 uA in sleep mode. I will test all of them at arrival. I believe it's a bit more safe to buy radios on Ebay because it's easy to complain if something shouldn't work as expected.

Cheers!

10pcs NRF24L01+ 2.4GHz Antenna Wireless Transceiver Module for Arduino New

$9.99

2252 available