rfm69 network issue

[alexsh1]

This is a node scheme:

Pretty much standard.
I wonder if anyone has used a constructor for rf69 library?

By the way I tried to load the default sketches (gateway and node) from lowpower lab and gateway does not hear the node. Again, I'm not sure wiring on the node is done in a standard moteino way - I'm using the pins which MySensors are using

D9 - not used
D10 - NSS
D11 - MOSI
D12 - MISO
D13 - SCK
IRQ - D2

Obviously 3.3V and GND and ANT

EDIT: yes the pins used my Moteino Mega are different.

So the question is how do I used a constructor for rf69 and MySensors???

[scalz]

i guess you have soldered the jumper...hmm, with lowpowerlab for instance, if your gw send a packet, does the node receive something? Or vice versa ?

For debugging, bad practice in prod but :
You could put a serial message in the isr of your radio module lib , and check if it reachs it at least. Then if ack flag is triggered etc. But at least, if it doesn't enter in your isr, then it's sure there is a problem with your irq pin...
You can debug this in interruptHandler() of rfm69.cpp
i have looked briefly in mysensors libs, but i don't see why that couldn't work for your moteino mega. i have one too but not tried with mysensors and no radio on it. But i use rfm69 libs with mysensors on Atsam (arduino zero similar) so this is working.

What do you mean by rfm69 constructor??

[alexsh1]

@scalz

RFM69 radio = RFM69(RFM69_CS, RFM69_IRQ, IS_RFM69HCW, RFM69_IRQN);

This is what I mean. I need to understand what it means for 328p and 1284p

[alexsh1]

After fiddling with the radios I manage to make them talk, but the following I got from the node:

Starting sensor (RRNNA-, 2.0.0-beta)
Radio init successful.
send: 1-1-0-0 s=255,c=3,t=15,pt=0,l=2,sg=0,st=fail:

send: 1-1-0-0 s=255,c=0,t=17,pt=0,l=10,sg=0,st=fail:2.0.0-beta
send: 1-1-0-0 s=255,c=3,t=6,pt=1,l=1,sg=0,st=fail:0
send: 1-1-0-0 s=255,c=3,t=11,pt=0,l=16,sg=0,st=fail:Light Lux Sensor
send: 1-1-0-0 s=255,c=3,t=12,pt=0,l=3,sg=0,st=fail:1.0
send: 1-1-0-0 s=0,c=0,t=16,pt=0,l=0,sg=0,st=fail:
find parent
send: 1-1-255-255 s=255,c=3,t=7,pt=0,l=0,sg=0,st=bc:
Init complete, id=1, parent=0, distance=255
54612
send: 1-1-0-0 s=0,c=1,t=23,pt=3,l=2,sg=0,st=fail:54612

Meantime, I can see that the same message is transferred three times. From the Domoticz log:

 2016-06-29 20:03:18.057 Hardware Monitor: Fetching data (System sensors)
2016-06-29 20:03:22.299 MySensors: Node: 1, Sketch Name: Light Lux Sensor
2016-06-29 20:03:22.347 MySensors: Node: 1, Sketch Name: Light Lux Sensor
2016-06-29 20:03:22.393 MySensors: Node: 1, Sketch Name: Light Lux Sensor
2016-06-29 20:03:22.442 MySensors: Node: 1, Sketch Version: 1.0
2016-06-29 20:03:22.485 MySensors: Node: 1, Sketch Version: 1.0
2016-06-29 20:03:22.529 MySensors: Node: 1, Sketch Version: 1.0
2016-06-29 20:03:24.856 (Moteino) Lux (Light Level)
2016-06-29 20:03:24.899 (Moteino) Lux (Light Level)
2016-06-29 20:03:24.944 (Moteino) Lux (Light Level) 

[alexsh1]

These test sketches work just fine including ACK

Node sketch (rfm69cw)

/* RFM69 library and code by Felix Rusu - felix@lowpowerlab.com
// Get libraries at: https://github.com/LowPowerLab/
// Make sure you adjust the settings in the configuration section below !!!
// **********************************************************************************
// Copyright Felix Rusu, LowPowerLab.com
// Library and code by Felix Rusu - felix@lowpowerlab.com
// **********************************************************************************
// License
// **********************************************************************************
// This program is free software; you can redistribute it 
// and/or modify it under the terms of the GNU General    
// Public License as published by the Free Software       
// Foundation; either version 3 of the License, or        
// (at your option) any later version.                    
//                                                        
// This program is distributed in the hope that it will   
// be useful, but WITHOUT ANY WARRANTY; without even the  
// implied warranty of MERCHANTABILITY or FITNESS FOR A   
// PARTICULAR PURPOSE. See the GNU General Public        
// License for more details.                              
//                                                        
// You should have received a copy of the GNU General    
// Public License along with this program.
// If not, see <http://www.gnu.org/licenses></http:>.
//                                                        
// Licence can be viewed at                               
// http://www.gnu.org/licenses/gpl-3.0.txt
//
// Please maintain this license information along with authorship
// and copyright notices in any redistribution of this code
// **********************************************************************************/

#include <RFM69.h>    //get it here: https://www.github.com/lowpowerlab/rfm69
#include <SPI.h>

//*********************************************************************************************
// *********** IMPORTANT SETTINGS - YOU MUST CHANGE/ONFIGURE TO FIT YOUR HARDWARE *************
//*********************************************************************************************
#define NETWORKID     100  // The same on all nodes that talk to each other
#define NODEID        2    // The unique identifier of this node
#define RECEIVER      1    // The recipient of packets

//Match frequency to the hardware version of the radio on your Feather
#define FREQUENCY     RF69_433MHZ
//#define FREQUENCY     RF69_868MHZ
//#define FREQUENCY     RF69_915MHZ
#define ENCRYPTKEY    "sampleEncryptKey" //exactly the same 16 characters/bytes on all nodes!
#define IS_RFM69HCW   false // set to 'true' if you are using an RFM69HCW module

//*********************************************************************************************
#define SERIAL_BAUD   115200

#define RFM69_CS      10
#define RFM69_IRQ     2
#define RFM69_IRQN    0  // Pin 2 is IRQ 0!
#define RFM69_RST     9

#define LED           8  // onboard blinky


int16_t packetnum = 0;  // packet counter, we increment per xmission

RFM69 radio;

//RFM69 radio = RFM69(RFM69_CS, RFM69_IRQ, IS_RFM69HCW, RFM69_IRQN);

void setup() {
  while (!Serial); // wait until serial console is open, remove if not tethered to computer
  Serial.begin(SERIAL_BAUD);

  Serial.println("Arduino RFM69HCW Transmitter");
  
  // Hard Reset the RFM module
  pinMode(RFM69_RST, OUTPUT);
  digitalWrite(RFM69_RST, HIGH);
  delay(100);
  digitalWrite(RFM69_RST, LOW);
  delay(100);

  // Initialize radio
  radio.initialize(FREQUENCY,NODEID,NETWORKID);
  if (IS_RFM69HCW) {
    radio.setHighPower();    // Only for RFM69HCW & HW!
  }
  radio.setPowerLevel(31); // power output ranges from 0 (5dBm) to 31 (20dBm)
  
  radio.encrypt(ENCRYPTKEY);
  
  pinMode(LED, OUTPUT);
  Serial.print("\nTransmitting at ");
  Serial.print(FREQUENCY==RF69_433MHZ ? 433 : FREQUENCY==RF69_868MHZ ? 868 : 915);
  Serial.println(" MHz");
}


void loop() {
  delay(1000);  // Wait 1 second between transmits, could also 'sleep' here!
    
  char radiopacket[20] = "Hello World #";
  itoa(packetnum++, radiopacket+13, 10);
  Serial.print("Sending "); Serial.println(radiopacket);
    
  if (radio.sendWithRetry(RECEIVER, radiopacket, strlen(radiopacket))) { //target node Id, message as string or byte array, message length
    Serial.println("OK");
    Blink(LED, 50, 3); //blink LED 3 times, 50ms between blinks
  }

  radio.receiveDone(); //put radio in RX mode
  Serial.flush(); //make sure all serial data is clocked out before sleeping the MCU
}

void Blink(byte PIN, byte DELAY_MS, byte loops)
{
  for (byte i=0; i<loops; i++)
  {
    digitalWrite(PIN,HIGH);
    delay(DELAY_MS);
    digitalWrite(PIN,LOW);
    delay(DELAY_MS);
  }
}

Gateway sketch (Moteino Mega, rfm69hcw):

/* RFM69 library and code by Felix Rusu - felix@lowpowerlab.com
// Get libraries at: https://github.com/LowPowerLab/
// Make sure you adjust the settings in the configuration section below !!!
// **********************************************************************************
// Copyright Felix Rusu, LowPowerLab.com
// Library and code by Felix Rusu - felix@lowpowerlab.com
// **********************************************************************************
// License
// **********************************************************************************
// This program is free software; you can redistribute it 
// and/or modify it under the terms of the GNU General    
// Public License as published by the Free Software       
// Foundation; either version 3 of the License, or        
// (at your option) any later version.                    
//                                                        
// This program is distributed in the hope that it will   
// be useful, but WITHOUT ANY WARRANTY; without even the  
// implied warranty of MERCHANTABILITY or FITNESS FOR A   
// PARTICULAR PURPOSE. See the GNU General Public        
// License for more details.                              
//                                                        
// You should have received a copy of the GNU General    
// Public License along with this program.
// If not, see <http://www.gnu.org/licenses></http:>.
//                                                        
// Licence can be viewed at                               
// http://www.gnu.org/licenses/gpl-3.0.txt
//
// Please maintain this license information along with authorship
// and copyright notices in any redistribution of this code
// **********************************************************************************/

#include <RFM69.h>    //get it here: https://www.github.com/lowpowerlab/rfm69
#include <SPI.h>

//*********************************************************************************************
// *********** IMPORTANT SETTINGS - YOU MUST CHANGE/ONFIGURE TO FIT YOUR HARDWARE *************
//*********************************************************************************************
#define NETWORKID     100  //the same on all nodes that talk to each other
#define NODEID        1  

//Match frequency to the hardware version of the radio on your Feather
#define FREQUENCY     RF69_433MHZ
//#define FREQUENCY     RF69_868MHZ
//#define FREQUENCY      RF69_915MHZ
#define ENCRYPTKEY     "sampleEncryptKey" //exactly the same 16 characters/bytes on all nodes!
#define IS_RFM69HCW    true // set to 'true' if you are using an RFM69HCW module

//*********************************************************************************************
#define SERIAL_BAUD   115200

#define RFM69_CS      10
#define RFM69_IRQ     2
#define RFM69_IRQN    0  // Pin 2 is IRQ 0!
#define RFM69_RST     9

#define LED           15  // onboard blinky

int16_t packetnum = 0;  // packet counter, we increment per xmission

RFM69 radio;

void setup() {
  while (!Serial); // wait until serial console is open, remove if not tethered to computer
  Serial.begin(SERIAL_BAUD);

  Serial.println("Feather RFM69HCW Receiver");
  
  // Hard Reset the RFM module
  pinMode(RFM69_RST, OUTPUT);
  digitalWrite(RFM69_RST, HIGH);
  delay(100);
  digitalWrite(RFM69_RST, LOW);
  delay(100);
  
  // Initialize radio
  radio.initialize(FREQUENCY,NODEID,NETWORKID);
  if (IS_RFM69HCW) {
    radio.setHighPower();    // Only for RFM69HCW & HW!
  }
  radio.setPowerLevel(31); // power output ranges from 0 (5dBm) to 31 (20dBm)
  
  radio.encrypt(ENCRYPTKEY);
  
  pinMode(LED, OUTPUT);

  Serial.print("\nListening at ");
  Serial.print(FREQUENCY==RF69_433MHZ ? 433 : FREQUENCY==RF69_868MHZ ? 868 : 915);
  Serial.println(" MHz");
}

void loop() {
  //check if something was received (could be an interrupt from the radio)
  if (radio.receiveDone())
  {
    //print message received to serial
    Serial.print('[');Serial.print(radio.SENDERID);Serial.print("] ");
    Serial.print((char*)radio.DATA);
    Serial.print("   [RX_RSSI:");Serial.print(radio.RSSI);Serial.print("]");

    //check if received message contains Hello World
    if (strstr((char *)radio.DATA, "Hello World"))
    {
      //check if sender wanted an ACK
      if (radio.ACKRequested())
      {
        radio.sendACK();
        Serial.println(" - ACK sent");
      }
      Blink(LED, 40, 3); //blink LED 3 times, 40ms between blinks
    }  
  }

  radio.receiveDone(); //put radio in RX mode
  Serial.flush(); //make sure all serial data is clocked out before sleeping the MCU
}

void Blink(byte PIN, byte DELAY_MS, byte loops)
{
  for (byte i=0; i<loops; i++)
  {
    digitalWrite(PIN,HIGH);
    delay(DELAY_MS);
    digitalWrite(PIN,LOW);
    delay(DELAY_MS);
  }
}

[scalz]

@alexsh1 ok. i thought you were talking about something else. So to answer, it works the same for 328p, 1284p if irq and pin is right. for st fail, perhaps the adapter..it's a lot better when soldered directly on node pcb.

[alexsh1]

@scalz Sorted. One radio on the GW is with amplification so I had to add the following to the code:

#define MY_IS_RFM69HW
#define MY_RF69_IRQ_PIN 2 // not sure if this line makes any difference

All works perfectly now.

[alexsh1]

Anyone knows how I can change the rfm69 speed within the sketch?
Something like MY_RF69_DATARATE or something?

Additionally, I'd like to use encryption, but no idea where to start. This is very easy with rf69 library, but not sure how to apply in MySensors.

[alexsh1]

This is the final product - I am @GertSanders would appreciate it

[scalz]

@alexsh1 cool :) for encryption this is done at radio init. You can enable/disable it by using MY_RFM69_ENABLE_ENCRYPTION
And for the datarate, there is no var for this. it's done by using register in radio init. I could look at adding this as I have locally updated mys with the latest rfm69 features.

[alexsh1]

@scalz Thanks! Encryption - but how do I define the key?

#define ENCRYPTKEY     "sampleEncryptKey"

or something.

Looking forward to your updates on rfm69!

[scalz]

@alexsh1 For the encrypt key...I'm not sure how this works in mysensors. I will look into this later today ;)
Actually, it works like this : during radio init, encryption is enabled or not depending of the above define. For the key (which is stored in eeprom) it uses EEPROM_RF_ENCRYPTION_AES_KEY_ADDRESS which is related to signing defines..you can have more info on this in MyEepromAddresses.h . you will see how it is constructed.

[alexsh1]

@scalz So they key is taken from EEPROM and I cannot define it in the sketch? I think this is similar how soft signing works on non-ATSHA204A nodes - they HMAC/AES key is stored in EEPROM

[alexsh1]

@scalz said:

And for the datarate, there is no var for this. it's done by using register in radio init. I could look at adding this as I have locally updated mys with the latest rfm69 features.

This is not working in MySensors sketch:

radio.writeReg(0x03,0x0D);
radio.writeReg(0x04,0x05);

Where do I set the bitrate?

[scalz]

@alexsh1 i don't know if it's possible on the fly, just for knowledge i will look later if it's possible as rfm69 already works well for me. And for longer range and as i have some devices with RFM95, i've started to port radiohead rfm95 for mysensors but not finished yet
So, about your question, it's done at init. in rfm69 driver, initialize(), line 52-53. you can try with some presets defined in rfm69registers.h if you want.

[alexsh1]

@scalz thanks - as I said I am new to RFM69 and have to start from scratch (looking into the driver library) so apologies for asking noobs questions.
I did go through the driver and I see what you mean. Unfortunately those things like bidrate are not configurable from MySensors.h

I'm very much interested in RFM95 though the cost of a transmitter is prohibitively expensive (£7.5 for one on aliexpress vs £0.7 for nrf24l01+). I think I'll use it for some sensors outside of the house. Where do you stand with porting?

[jpaulin]

This post is deleted!

[scalz]

@alexsh1 : just to say i have added the possibility to set datarate ;) you're right, that could be interesting to test.
that will be a simple define like
MY_RFM69_BITRATE_CUSTOM 4800
where 4800 in kbits. if not defined it takes default params.
And here few interesting comments about the datarate : https://openenergymonitor.org/emon/node/6178

[frencho]

@chrille I'm facing similar behavior with my setup, I get the st:fail but if I touch the antenna on the gateway side, it's working fine.

My setup is arduino 2.0, on the same node as @alexsh1 (the @GertSanders one) with the NRF2RFM69 adapter from @tbowmo. Antenna is wire lenght of 82.2 mm for the 868Mhz rfm69HW.

on the gateway side I did :
// Enable and select radio type attached
//#define MY_RADIO_NRF24
#define MY_RADIO_RFM69

I added a small capacitor on the VCC-GND of the radio, and I soldered the jumper 2 to connect IRQ

the inclusion mode is also fully commented (I don't have inclusion button on board)

and the config file also #define MY_IS_RFM69HW is uncommented as well as #define MY_RADIO_RFM69

Any idea where my st:fail could come from ?

[chrille]

@frencho said:

Any idea where my st:fail could come from ?

No, I have only seen the issue with the combination of ESP866 and RFM69. I suggest you get the RFM69 library from lowpowerlabs and load the "Node" and "Gateway" sketches from https://github.com/LowPowerLab/RFM69/tree/master/Examples/

The sketches will send packets in both directions and it's very easy to see packet loss and also the RSSI (signal strenght) of the traffic on both sides. This is what I used to diagnose my issue. You can install the RFM69 library directly in your libraries folder without conflicting with mysensors.