NRF5 Hardware module crash after while

[Didou]

@ncollins Will try that just have to understand how to just add the fix to the latest stable release

[Didou]

@evb You do undertstand it very well, it was also my first understanding and to avoid any issue relating to EMC I did also tried outside the cabinet, not using my own power supply adding extra capacitors and lately power up the CPU module with the NFC on batteries .... In all cases the crash issue is still there !

I was pretty surprises as there is quiet a lot of different modules been built on openhardwareio and I have seen no report about this issue, so i was conviced the issue was mine !

[evb]

@Didou let's then keep our fingers crossed that it's just a software problem :-)

[waspie]

@Didou @evb
I download the dev version AND replaced the two files with the ones from the proposed fix.

I think that's the right path - it seems to be showing positive results.

[ncollins]

@Didou for my testing, I pulled the entire branch referenced in the merge request. I believe that is MySensors-development + nrf_esb_redesign changes. But you're probably safe to just replace the two files touched in the commits.

[Didou]

@ncollins OK thanks, I did try that pulling diretly from github, so branch + files (Alpha-1)

I did upgrade 3 modules and keep many others with previous to have a comparison basis

Let's cross fingers :)

It is about 12 hours for now, none of the updated one did crash and one for the unpatched did crash. I need to wait at lest 24 to 36 hours to confirm

[ncollins]

@Didou Which NRF5 module are you using exactly?

I also believe this code from your sketch is unnecessary, given MySensors handles most of it:

void nRF_Init() {
  NRF_POWER->DCDCEN = 0; // Not reducing the radio current   
  NRF_PWM0  ->ENABLE = 0;
  NRF_PWM1  ->ENABLE = 0;
  NRF_PWM2  ->ENABLE = 0;
  NRF_TWIM1 ->ENABLE = 0;
  NRF_TWIS1 ->ENABLE = 0;
  //NRF_RADIO->TXPOWER = 8;
}

void disableNfc() {
  NRF_NFCT->TASKS_DISABLE = 1;
  NRF_NVMC->CONFIG = 1;
  NRF_UICR->NFCPINS = 0;
  NRF_NVMC->CONFIG = 0;
}

void turnOffAdc() {
  if (NRF_SAADC->ENABLE) {
    NRF_SAADC->TASKS_STOP = 1;
    while (NRF_SAADC->EVENTS_STOPPED) {}
    NRF_SAADC->ENABLE = 0;
    while (NRF_SAADC->ENABLE) {}
  }
}

[Didou]

@ncollins I'm using NRF52832 Ebyte modules

[Didou]

@Didou After a few days of test of your NRF fix, I can say there is a huge improuvement either for USB, AC supply or battery powered nodes, they just do not failed ! this is great. thanks for your efforts improving the mysensor code

It's a while since the mysensor master has been updated, any ideas when it's goanna happend, it will also be good for others to benefits

[waspie]

This post is deleted!

[waspie]

@ncollins

Are you currently working with any NRF5x? Do you have a working motion/contact sketch for the NRF52?