nRF5 action!


  • Hardware Contributor

    @Jokgi said in nRF5 Bluetooth action!:

    @NeverDie I am not sure about all the boards for sell on that site. but I would have to say that any J-link programmer that is packaged as such for $12-$40 dollars is probably counterfeit. Updating these items with newer J-link firmware would more then likely disable them. (On purpose)

    Yes they are of course counterfeit, and using old firmware. You have a message when you use them with Segger software asking you to upgrade to newer firmware and if you accept your programmer is disabled (web is full of solutions to reflash a firmware).
    But if you don't upgrade the firmware it works fine as a SWD programmer, at least for basic use (programming). I have not tried any debugging.



  • @Nca78 Corrrect. It seems to me that a < 50 dollar Nordic nRF52 based Dev kit (or similar) which can be updated would be much less hassle.


  • Hero Member

    Are we limited to using RTC0? I've tried switching to RTC1 and RTC2, and I get the sense there are conflicts with both of them and the MySensors code if they're used.


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Are we limited to using RTC0? I've tried switching to RTC1 and RTC2, and I get the sense there are conflicts with both of them and the MySensors code if they're used.

    RTC1 is blocked by arduino (nRF5/delay.c) and RTC0 (nRF51) and RTC2 (nRF52) is used in MySensors (hal/architecture/MyHwNRF5.cpp)


  • Hero Member

    Success. I now have a "listen mode" for the nRF52832 radio that's completely controlled by the PPI while the MCU sleeps. Presently, it wakes up the radio every 100ms and listens for 1ms. This means no wasted power from oversight by the MCU. Using just the PPI, I can control to within about 30us over how long to make the cycle period and/or the listening duration.
    0_1505768284420_NewFile2.jpg
    0_1505768305717_NewFile1.jpg

    The scope shots show the current drawn. Scale: 1mv=1ma. As you can see, the DCDC regulator is engaged.

    Next step will be to have packet receipt wake up the MCU via an ISR, so that the packet can be processed. After that, I'll see how narrow I can make the receive window and still receive packets reliably. I think under 100us will be possible. Maybe even less than 60us if the bitrate is 2mbps. 🙂



  • @NeverDie Great Job!!!! Suggestion... If you are going to use the BTLE Softdevice you will have a smaller receive window if you use the external 32khz crystal option rather then the internal 32khz RC one. (+/- 20ppm with the crystal vs +/- 250 or 500ppm with the RC.


  • Hero Member

    At the extreme, one of the questions that will need answering is: what's the minimum number of bytes in a transmitted frame that you really do need before the receiver starts receiving garbage packets? For instance, a 10 byte frame, which should be more than adequate, would take 40us of airtime to either transmit or receive at 2mbps bitrate. One could get to a lower number by maybe sending a null packet as a wake-up packet, in which case maybe you also don't need CRC. So, that leaves you with some preamble, a network ID, and maybe a destination ID--or about 5 bytes. So, that would be around 20us of airtime. One could shrink that further by reducing the number of network ID bytes, but too much of that and possibly one starts to receive garbage packets.

    The RFM69 doesn't try to decode packets whose RSSI is below a specific programmable threshhold. I don't believe the nRF52 radio uses RSSI as a filter in that way though. It seems that the nRF52832 radio tries to decode whatever it's receiving, regardless of the RSSI. Or, at least, that's how I remember it. Anyone else played around with it?


  • Hero Member

    @d00616

    I adapted your RTC0 code for handling IRQ's, and it looks like this:

    
      NRF_RADIO->INTENSET = B100;  //interrupt MCU if a payload is received.
      // Enable interrupt
      NVIC_SetPriority(RADIO_IRQn, 15);
      NVIC_ClearPendingIRQ(RADIO_IRQn);
      NVIC_EnableIRQ(RADIO_IRQn);
    
    #if __CORTEX_M == 0x04
    #define NRF5_RESET_EVENT(event)                                                 \
            event = 0;                                                                   \
            (void)event
    #else
    #define NRF5_RESET_EVENT(event) event = 0
    #endif
    
    
    // This must be in one line
    extern "C" { void RADIO_IRQHandler(void) {packetCounter++; NRF5_RESET_EVENT(NRF_RADIO->EVENTS_PAYLOAD); NRF_RADIO->EVENTS_PAYLOAD=0; }}
    

    Question: Is

    NRF5_RESET_EVENT(NRF_RADIO->EVENTS_PAYLOAD); 
    

    doing anything more than

    NRF_RADIO->EVENTS_PAYLOAD=0;
    

    is?


  • Hero Member

    The good news is that I can now literally "see" that the listen-mode is receiving packets, because I've programmed the PPI to toggle an LED every time a packet is received (and it only toggles the LED if and only if I know that I'm sending the node packets from a different node). The bad news is that--so far, anyway--it doesn't appear to trigger an IRQ event that wakes up the MCU and runs the ISR.


  • Mod

    @NeverDie it seems like the preamble can be configured to be either 8 or 16 bits. Not sure if that's of any use, but I've been thinking about adjusting the LoRa preamble to get better sleep times. Maybe that could be applicable for nrf as well.


  • Hero Member

    It turns out that using just the standard radiohead packet/frame structure, I'm able to 100% reliably receive a two byte string (in this case, the letter "H" followed by a zero as the termination character) using the above PPI solution with a receive window of about 200ms. That includes preamble, CRC, and a long network ID number.

    I'm pleased with that result. It's vastly better than without the PPI solution.


  • Hero Member

    OK, I'm going to start with this, which works:

    //A simplified re-mix of code mostly written by d00616
    
    #include <nrf.h>
    #include <MySensors.h>
    
    int interrupt = 0;
    uint32_t theCounter;
    
    
    int8_t myHwSleep(unsigned long ms)
    {
      hwSleepPrepare(ms);
      //while (nrf5_rtc_event_triggered == false) {
        hwSleep();
      //}
      hwSleepEnd(ms);
      return MY_WAKE_UP_BY_TIMER;
    }
    
    
    void setup() {
      // put your setup code here, to run once:
      Serial.begin(250000);
      Serial.println("Start");
    
      // Configure RTC
      NRF_RTC0->TASKS_STOP = 1;
      NRF_RTC0->PRESCALER = 32;  
      NRF_RTC0->CC[0] = NRF_RTC0->COUNTER + (655);  //comparison for when to turn off the Rx.
      NRF_RTC0->EVTENSET = RTC_EVTENSET_COMPARE0_Msk;
      NRF_RTC0->INTENSET = RTC_INTENSET_COMPARE0_Msk;
      NRF_RTC0->TASKS_START = 1;
      NRF_RTC0->EVENTS_COMPARE[0] = 0;
    
      // Enable interrupt
      NVIC_SetPriority(RTC0_IRQn, 15);
      NVIC_ClearPendingIRQ(RTC0_IRQn);
      NVIC_EnableIRQ(RTC0_IRQn);
      Serial.println();
      Serial.println();
      Serial.println("Starting...");
    }
    
    
    void loop() {
    
      Serial.print(millis());
      Serial.print(" ");
      Serial.print(theCounter);
      Serial.print(" ");
      Serial.println(interrupt);
      myHwSleep(5000000);
    }
    
    /**
     * Reset events and read back on nRF52
     * http://infocenter.nordicsemi.com/pdf/nRF52_Series_Migration_v1.0.pdf
     */
    #if __CORTEX_M == 0x04
    #define NRF5_RESET_EVENT(event)                                                 \
            event = 0;                                                                   \
            (void)event
    #else
    #define NRF5_RESET_EVENT(event) event = 0
    #endif
    
    // This must be in one line
    extern "C" { void RTC0_IRQHandler(void) {   theCounter=NRF_RTC0->COUNTER; NRF5_RESET_EVENT(NRF_RTC0->EVENTS_COMPARE[0]); interrupt++; NRF_RTC0->TASKS_CLEAR = 1; }}
    

    and see if I can generate equivalent code which triggers on a pin change instead of an RTC event. If I can get that to work, then I'll take another stab at getting it to work based on the radio receiving a packet.


  • Hero Member

    I was only partially done with the code, but I decided to run it anyway. Oddly enough, even though there is no reference to " RTC0_IRQHandler(void)" as being the event handler, it gets fired off anyway whenever I press the button (pin P0.16):

    
    #include <nrf.h>
    #include <MySensors.h>
    
    int interrupt = 0;
    uint32_t theCounter;
    
    
    int8_t myHwSleep(unsigned long ms)
    {
      hwSleepPrepare(ms);
       hwSleep();
      hwSleepEnd(ms);
      return MY_WAKE_UP_BY_TIMER;
    }
    
    
    void setup() {
      // put your setup code here, to run once:
      Serial.begin(250000);
      Serial.println("Start");
    
      //Configure GPIOTE 
      NRF_GPIOTE->CONFIG[0]=0x31001;  // Toggle Event; Pin P0.16; Event Mode 
                                      //So, any pin change on P0.16 will trigger an event.
                                       //On the Nordic nRF52 DK, Pin 0.16 is a button.
                                       
      NRF_GPIOTE->CONFIG[1]=0x131203;  //Initial setting: HIGH; Toggle Task; Pin P0.18; Task Mode
                                       //On the Nordic nRF52 DK, Pin 0.18 is an LED.
                                       //Note: initial setting of HIGH means that the LED will be initially OFF.
    
      NRF_GPIOTE->INTENSET=1;  //Enable an event on Pin P0.16 to trigger an interrupt.
                                       
      NRF_PPI->CH[0].EEP = (uint32_t)&NRF_GPIOTE->EVENTS_IN[0];  //P0.16 pin change occured.
      NRF_PPI->CH[0].TEP = (uint32_t)&NRF_GPIOTE->TASKS_OUT[1]; //Toggle LED on pin P0.18.
      
      NRF_PPI->CHENSET=B1; //enable Channel 0.
    
      // Enable interrupt
      NVIC_SetPriority(GPIOTE_IRQn, 15);
      NVIC_ClearPendingIRQ(GPIOTE_IRQn);
      NVIC_EnableIRQ(GPIOTE_IRQn);
      Serial.println();
      Serial.println();
      Serial.println("Starting...");
    }
    
    
    void loop() {
    
      Serial.print(millis());
      Serial.print(" ");
      Serial.print(theCounter);
      Serial.print(" ");
      Serial.println(interrupt);
      myHwSleep(5000000);
    }
    
    /**
     * Reset events and read back on nRF52
     * http://infocenter.nordicsemi.com/pdf/nRF52_Series_Migration_v1.0.pdf
     */
    #if __CORTEX_M == 0x04
    #define NRF5_RESET_EVENT(event)                                                 \
            event = 0;                                                                   \
            (void)event
    #else
    #define NRF5_RESET_EVENT(event) event = 0
    #endif
    
    // This must be in one line
    extern "C" { void RTC0_IRQHandler(void) {   theCounter=NRF_RTC0->COUNTER; NRF5_RESET_EVENT(NRF_RTC0->EVENTS_COMPARE[0]); interrupt++; NRF_RTC0->TASKS_CLEAR = 1; }}
    

    So, it looks as though, at present, there can be only a single ISR for an entire sketch?


  • Hero Member

    Interesting result. I eliminated most of the sample code, and it actually manages to work even without an ISR! Instead of running an ISR, it appears to simply wake up from exactly where it last fell asleep. Then it continues running until it's put to sleep again:

    #include <nrf.h>
    #include <MySensors.h>
    
    uint32_t buttonPressCounter=0;
    
    
    int8_t myHwSleep(unsigned long ms)
    {
      hwSleepPrepare(ms);
      hwSleep();
      hwSleepEnd(ms);
      return MY_WAKE_UP_BY_TIMER;
    }
    
    
    void setup() {
      // put your setup code here, to run once:
      Serial.begin(250000);
      Serial.println("Start");
    
      //Configure GPIOTE 
      NRF_GPIOTE->CONFIG[0]=0x31001;  // Toggle Event; Pin P0.16; Event Mode 
                                      //So, any pin change on P0.16 will trigger an event.
                                       //On the Nordic nRF52 DK, Pin 0.16 is a button.
                                       
      NRF_GPIOTE->CONFIG[1]=0x131203;  //Initial setting: HIGH; Toggle Task; Pin P0.18; Task Mode
                                       //On the Nordic nRF52 DK, Pin 0.18 is an LED.
                                       //Note: initial setting of HIGH means that the LED will be initially OFF.
    
      NRF_GPIOTE->INTENSET=1;  //Enable an event on Pin P0.16 to trigger an interrupt.
                                       
      NRF_PPI->CH[0].EEP = (uint32_t)&NRF_GPIOTE->EVENTS_IN[0];  //P0.16 pin change occured.
      NRF_PPI->CH[0].TEP = (uint32_t)&NRF_GPIOTE->TASKS_OUT[1]; //Toggle LED on pin P0.18.
      
      NRF_PPI->CHENSET=B1; //enable Channel 0.
    
      // Enable interrupt
      NVIC_SetPriority(GPIOTE_IRQn, 15);
      NVIC_ClearPendingIRQ(GPIOTE_IRQn);
      NVIC_EnableIRQ(GPIOTE_IRQn);
      Serial.println();
      Serial.println();
      Serial.println("Starting...");
    }
    
    
    void loop() {
      Serial.print("time=");
      Serial.print(millis());
      Serial.print(", buttonPressCounter=");
      Serial.println(buttonPressCounter++);
      myHwSleep(5000000);
    }
    

  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Question: Is
    NRF5_RESET_EVENT(NRF_RADIO->EVENTS_PAYLOAD);

    doing anything more than
    NRF_RADIO->EVENTS_PAYLOAD=0;

    is?

    Yes. It reads back the register. http://infocenter.nordicsemi.com/topic/com.nordic.infocenter.nrf52/dita/nrf52/migration/functional.html?cp=2_4_0


  • Hero Member

    Whenever I use:

    #include <MySensors.h>
    

    on the nRF52832, there is around a 10 second delay between the end of "void startup()" and the beginning of "loop()". Why is that, and what is the MySensors library doing during that interval?


  • Mod

    @NeverDie it depends on the value of MY_TRANSPORT_WAIT_READY_MS
    See https://github.com/mysensors/MySensors/issues/927 for a discussion on what happens when - we're trying to clarify the documentation.


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    P.S.: you can reduce the RX/TX time by enabling fast ramp up in MODECNF0 if you haven't to care about nRF51 compatibility.

    Thanks for the tip. I tried it both ways. The first scope capture below is taken without MODECNF0 enabled on bit 0, and the second is with it enabled on bit 0.
    0_1505865330798_NewFile5.png
    0_1505865340675_NewFile6.png

    I don't really see any difference. Do you?
    Scale: 1mv=1ma. Scope captures of current drawn.

    Maybe @jokgi can comment? His bio says he's a Senior Field Application Engineer at Nordic Semiconductor.
    Maybe he sees (or knows) something that's not apparent about the fast ramp enable bit?


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Whenever I use:
    #include <MySensors.h>

    on the nRF52832, there is around a 10 second delay between the end of "void startup()" and the beginning of "loop()". Why is that, and what is the MySensors library doing during that interval?

    As an alternative, you can define '#define MY_CORE_ONLY' and use ' transportInit(); transportSetAddress(MY_NODE_ID);' to initialize the radio. This dosn't work at the moment with the nRF5. I'm currently looking what the reason is. All radio registers are equal when it's initialized after MySensors normal and my setup(). It's not able to send or receive packages.

    Maybe someone has an idea about the reason. Here is my code for nRF5 and other with nRF24.

    // Undefine to work in gateway mode
    #define MY_CORE_ONLY
    
    
    #define MY_NODE_ID (0)
    
    // Enable debug
    #define MY_DEBUG
    #define MY_DEBUG_VERBOSE_RF24
    #define MY_DEBUG_VERBOSE_NRF5_ESB
    
    // Enable and select radio type attached
    #ifndef ARDUINO_ARCH_NRF5
    #define MY_RADIO_NRF24
    #else
    #define MY_RADIO_NRF5_ESB
    #include <nrf.h>
    #endif
    //#define MY_RADIO_RFM69
    //#define MY_RADIO_RFM95
    
    //#define MY_OTA_LOG_SENDER_FEATURE
    //#define MY_OTA_LOG_RECEIVER_FEATURE
    
    #ifndef MY_CORE_ONLY
    #define MY_GATEWAY_SERIAL
    #endif
    #include <MySensors.h>
    
    void state() {
    #ifdef ARDUINO_ARCH_NRF5
     Serial.println("----------------------------------");
     Serial.print("NRF_RADIO->STATE  ");
     Serial.println(NRF_RADIO->STATE, HEX);
     Serial.print("NRF_RADIO->EVENTS_READY  ");
     Serial.println(NRF_RADIO->EVENTS_READY, HEX);
     Serial.print("NRF_RADIO->EVENTS_ADDRESS  ");
     Serial.println(NRF_RADIO->EVENTS_ADDRESS, HEX);
     Serial.print("NRF_RADIO->EVENTS_PAYLOAD  ");
     Serial.println(NRF_RADIO->EVENTS_PAYLOAD, HEX);
     Serial.print("NRF_RADIO->EVENTS_END  ");
     Serial.println(NRF_RADIO->EVENTS_END, HEX);
     Serial.print("NRF_RADIO->EVENTS_DISABLED  ");
     Serial.println(NRF_RADIO->EVENTS_DISABLED, HEX);
     Serial.print("NRF_RADIO->EVENTS_DEVMATCH  ");
     Serial.println(NRF_RADIO->EVENTS_DEVMATCH, HEX);
     Serial.print("NRF_RADIO->EVENTS_DEVMISS  ");
     Serial.println(NRF_RADIO->EVENTS_DEVMISS, HEX);
     Serial.print("NRF_RADIO->EVENTS_RSSIEND  ");
     Serial.println(NRF_RADIO->EVENTS_RSSIEND, HEX);
     Serial.print("NRF_RADIO->EVENTS_BCMATCH  ");
     Serial.println(NRF_RADIO->EVENTS_BCMATCH, HEX);
     Serial.print("NRF_RADIO->CRCSTATUS  ");
     Serial.println(NRF_RADIO->CRCSTATUS, HEX);
     Serial.print("NRF_RADIO->RXMATCH  ");
     Serial.println(NRF_RADIO->RXMATCH, HEX);
     Serial.print("NRF_RADIO->RXCRC  ");
     Serial.println(NRF_RADIO->RXCRC, HEX);
     Serial.print("NRF_RADIO->DAI  ");
     Serial.println(NRF_RADIO->DAI, HEX);
     Serial.print("NRF_RADIO->PACKETPTR  ");
     Serial.println(NRF_RADIO->PACKETPTR, HEX);
     Serial.print("NRF_RADIO->FREQUENCY  ");
     Serial.println(NRF_RADIO->FREQUENCY, HEX);
     Serial.print("NRF_RADIO->TXPOWER  ");
     Serial.println(NRF_RADIO->TXPOWER, HEX);
     Serial.print("NRF_RADIO->MODE  ");
     Serial.println(NRF_RADIO->MODE, HEX);
     Serial.print("NRF_RADIO->PCNF0  ");
     Serial.println(NRF_RADIO->PCNF0, HEX);
     Serial.print("NRF_RADIO->PCNF1  ");
     Serial.println(NRF_RADIO->PCNF1, HEX);
     Serial.print("NRF_RADIO->BASE0  ");
     Serial.println(NRF_RADIO->BASE0, HEX);
     Serial.print("NRF_RADIO->BASE1  ");
     Serial.println(NRF_RADIO->BASE1, HEX);
     Serial.print("NRF_RADIO->PREFIX0  ");
     Serial.println(NRF_RADIO->PREFIX0, HEX);
     Serial.print("NRF_RADIO->PREFIX1  ");
     Serial.println(NRF_RADIO->PREFIX1, HEX);
     Serial.print("NRF_RADIO->TXADDRESS  ");
     Serial.println(NRF_RADIO->TXADDRESS, HEX);
     Serial.print("NRF_RADIO->RXADDRESSES  ");
     Serial.println(NRF_RADIO->RXADDRESSES, HEX);
     Serial.print("NRF_RADIO->CRCCNF  ");
     Serial.println(NRF_RADIO->CRCCNF, HEX);
     Serial.print("NRF_RADIO->SHORTS  ");
     Serial.println(NRF_RADIO->SHORTS, HEX);
     Serial.print("NRF5_RADIO_TIMER->MODE ");
     Serial.println(NRF5_RADIO_TIMER->MODE);
     Serial.print("NRF5_RADIO_TIMER->BITMODE ");
     Serial.println(NRF5_RADIO_TIMER->BITMODE);
     Serial.print("NRF5_RADIO_TIMER->SHORTS ");
     Serial.println(NRF5_RADIO_TIMER->SHORTS);
     Serial.print("NRF5_RADIO_TIMER->PRESCALER ");
     Serial.println(NRF5_RADIO_TIMER->PRESCALER);
      // Reset compare events
    #ifdef NRF51
      for (uint8_t i=0;i<4;i++) {
    #else
      for (uint8_t i=0;i<6;i++) {
    #endif
     Serial.print("NRF5_RADIO_TIMER->EVENTS_COMPARE[");
     Serial.print(i);
     Serial.print("] ");
     Serial.println(NRF5_RADIO_TIMER->EVENTS_COMPARE[i]);
    }
    Serial.println("----------------------------------");
    #endif
    }
    
    void setup() {
      Serial.begin(115200);
      
      #ifdef MY_CORE_ONLY
      transportInit();
      delay(1000);
      transportSetAddress(MY_NODE_ID);
      #endif
    }
    
    void loop() {
      state();
    
      #ifdef MY_CORE_ONLY
      // Check for packages
      if (transportAvailable()) {
        uint8_t buffer[256];
        uint8_t num = transportReceive(&buffer);
        for (int i=0;i<num;i++) {
          if (buffer[i]<0x10) Serial.print("0");
          Serial.print(buffer[i], HEX);
          Serial.print(" ");
        }
        Serial.println();
      }
      #endif
    
      // Pause
      //sleep(1000); don't use this on SAMD. The device must be restored with reset doubleclick
      delay(1000);
      
      // Send data
      //transportSend(MY_NODE_ID, "abcd", 4, false);
    }
    

    @NeverDie said in nRF5 Bluetooth action!:

    @d00616 said in nRF5 Bluetooth action!:

    P.S.: you can reduce the RX/TX time by enabling fast ramp up in MODECNF0 if you haven't to care about nRF51 compatibility.

    Thanks for the tip. I tried it both ways. The first scope capture below is taken without MODECNF0 enabled on bit 0, and the second is with it enabled on bit 0.

    Have you checked the MODECNF0 register after ramp up? Maybe the register must be changed in a specific state?


  • Hero Member

    Not sure why, but so far I haven't been able to get the radio to generate an interrupt (after it receives a packet) that directly wakes the MCU. So, as a workaround, I'm using PPI to have the radio toggle a GPIO output pin, which is directly shorted to a GPIO input pin. Changes in that input pin are able to trigger an interrupt which wakes the MCU. So, in this circuitous way, I'm able to get the radio to wake the MCU. It works, but with a propagation delay, and obviously I shouldn't have to be doing it so indirectly.

    Has anyone else had any success yet in waking the MCU from sleep upon packet receipt by the radio?

    Which interrupt register in the MCU is the interrupt generated by the radio tied to? Perhaps it needs to be premptively cleared. I think the next step is to check whether the radio's interrupt is even being received by the MCU.


  • Contest Winner

    @d00616 said in nRF5 Bluetooth action!:

    Maybe someone has an idea about the reason. Here is my code for nRF5 and other with nRF24.

    Now, I have the NRF5_ESB working under MY_CORE_ONLY condition. The HFCLK is not initialized. I do some code changes to allow using the radio in core only mode.


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    @d00616 said in nRF5 Bluetooth action!:

    Maybe someone has an idea about the reason. Here is my code for nRF5 and other with nRF24.

    Now, I have the NRF5_ESB working under MY_CORE_ONLY condition. The HFCLK is not initialized. I do some code changes to allow using the radio in core only mode.

    I'm not sure what that means. Can you clarify what is working and what isn't, especially with regards to interrupts? If the code isn't yet ready, I'll just stick with the GPIO pins solution (above) and revisit this topic again at some future date when its further along.


  • Hero Member

    Actually, there's another reason for wanting to avoid using the GPIO pins to intermediate waking up the MCU: earlier measurements in this thread showed that using the GPIO's in anything other than a "disconnected" state noticeably increases the current draw while sleeping.


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    The HFCLK is not initialized.

    Not sure if it helps you at all, but to save extra energy I'm using the PPI to turn-on HFCLK before the Rx "listen-mode" begins, and then turn it off after the receiver is subsequently put to sleep:

      //Note: radio is assumed to be sleeping by this point, and with high frequency crystal oscillator turned off.
      NRF_PPI->CH[0].EEP = (uint32_t)&NRF_RTC0->EVENTS_OVRFLW;  //when COUNTER overflows.
      NRF_PPI->CH[0].TEP = (uint32_t)&NRF_CLOCK->TASKS_HFCLKSTART;  //turn-on the HF crystal oscillator
    
      NRF_PPI->CH[1].EEP = (uint32_t)&NRF_CLOCK->EVENTS_HFCLKSTARTED;  //After HF Clock started.
      NRF_PPI->CH[1].TEP = (uint32_t)&NRF_RADIO->TASKS_RXEN;  //turn on the radio receiver
    
      NRF_PPI->CH[2].EEP = (uint32_t)&NRF_RADIO->EVENTS_READY;  //After event READY, radio shall be in state RXIDLE.
      NRF_PPI->CH[2].TEP = (uint32_t)&NRF_RADIO->TASKS_START;  //Move from RXIDLE mode into RX mode.
    
      NRF_PPI->CH[3].EEP = (uint32_t)&NRF_RTC0->EVENTS_COMPARE[0];  // If time to turn off the radio receiver
      NRF_PPI->CH[3].TEP = (uint32_t)&NRF_RADIO->TASKS_STOP; //Move radio from RX mode back into RXIDLE mode
      
      NRF_PPI->CH[4].EEP = (uint32_t)&NRF_RTC0->EVENTS_COMPARE[1];  // If enough time has passed
      NRF_PPI->CH[4].TEP = (uint32_t)&NRF_RADIO->TASKS_DISABLE; //Sleep the radio
      NRF_PPI->FORK[4].TEP = (uint32_t)&NRF_CLOCK->TASKS_HFCLKSTOP; //Turn-off the high frequency crystal oscillator 
      
      NRF_PPI->CH[5].EEP = (uint32_t)&NRF_RTC0->EVENTS_COMPARE[2];  // If 100ms has passed
      NRF_PPI->CH[5].TEP = (uint32_t)&NRF_RTC0->TASKS_TRIGOVRFLW; //Set COUNTER so that it will overflow in 16 ticks.
    
      NRF_PPI->CH[6].EEP = (uint32_t)&NRF_RADIO->EVENTS_END;  //packet received.
      NRF_PPI->CH[6].TEP = (uint32_t)&NRF_GPIOTE->TASKS_OUT[1]; //Make pin P0.18. be LOW (turn on the LED).
    
      NRF_PPI->CH[7].EEP = (uint32_t)&NRF_GPIOTE->EVENTS_IN[0];  //P0.16 pin change occured.
      NRF_PPI->CH[7].TEP = (uint32_t)&NRF_GPIOTE->TASKS_OUT[2]; //Make pin P0.17. be LOW (turn on the LED).
      
      NRF_PPI->CHENSET=B11111111; //enable Channels 7,6,5,4,3,2,1,and 0.
    

    It works. 🙂


  • Hero Member

    @d00616

    Can you see any reason as to why the radio isn't waking the MCU after it receives a packet? Here's the entire sketch:

    #include <nrf.h>
    //#include <MySensors.h>
    #include <RH_NRF51.h>
    
    // Singleton instance of the radio driver
    RH_NRF51 nrf51;
    
    bool toggle=true;
    uint32_t packetCounter=0;
    uint8_t myBuffer[20];  //required buffer for transmitting packet
    uint8_t my_default_network_address[] = {0xE7, 0xE7, 0xE7, 0xE7, 0xE7};
    
    bool mySetNetworkAddress(uint8_t* address, uint8_t len)
    {
        if (len < 3 || len > 5)
      return false;
    
        // First byte is the prefix, remainder are base
        NRF_RADIO->PREFIX0    = ((address[0] << RADIO_PREFIX0_AP0_Pos) & RADIO_PREFIX0_AP0_Msk);
        uint32_t base;
        memcpy(&base, address+1, len-1);
        NRF_RADIO->BASE0 = base;
    
        NRF_RADIO->PCNF1 =  (
      (((sizeof(myBuffer)) << RADIO_PCNF1_MAXLEN_Pos)  & RADIO_PCNF1_MAXLEN_Msk)  // maximum length of payload
      | (((0UL)        << RADIO_PCNF1_STATLEN_Pos) & RADIO_PCNF1_STATLEN_Msk) // expand the payload with 0 bytes
      | (((len-1)      << RADIO_PCNF1_BALEN_Pos)   & RADIO_PCNF1_BALEN_Msk)); // base address length in number of bytes.
    
        return true;
    }
    
    void myHwSleepPrepare(unsigned long ms)
    {
      // Idle serial device
      NRF_UART0->TASKS_STOPRX = 1;
      NRF_UART0->TASKS_STOPTX = 1;
      NRF_UART0->TASKS_SUSPEND = 1;
    
      //NRF_CLOCK->TASKS_HFCLKSTOP = 1;
      
      // Enable low power sleep mode
      NRF_POWER->TASKS_LOWPWR = 1;
    }
    
    // Sleep in System ON mode
    inline void doTheSleep()
    {
      __WFE();
      __SEV();
      __WFE();
    }
    
    void myHwSleepEnd(unsigned long ms)
    {
      // Start HFCLK
      //if (nrf5_pwr_hfclk) {
      if (false) {
        NRF_CLOCK->EVENTS_HFCLKSTARTED = 0;
        NRF_CLOCK->TASKS_HFCLKSTART = 1;
        while (NRF_CLOCK->EVENTS_HFCLKSTARTED == 0)
          ;
        // Enable low latency sleep mode
        NRF_POWER->TASKS_CONSTLAT = 1;
      }
    
       // Start serial device
    //#ifndef MY_DISABLED_SERIAL
      NRF_UART0->TASKS_STARTRX = 1;
      NRF_UART0->TASKS_STARTTX = 1;
    //#endif
    }
    
    void myHwSleep(unsigned long ms)
    {
      myHwSleepPrepare(ms);
      doTheSleep();
      //now sleeping
      myHwSleepEnd(ms);
    
    }
    
    
    
    void setup() {
      Serial.begin(250000);
      Serial.println();
      Serial.println("Starting...");
      Serial.flush();
    
      NRF_POWER->DCDCEN=1;  //enable the DCDC voltage regulator as the default.
      
      // Configure RTC
      NRF_RTC0->TASKS_STOP = 1;  //stop the RTC counter so that it can be configured without incident
    
      // Enable interrupt
      //NVIC_SetPriority(GPIOTE_IRQn, 15);
      //NVIC_ClearPendingIRQ(GPIOTE_IRQn);
      //NVIC_EnableIRQ(GPIOTE_IRQn);
      
      NVIC_SetPriority(RADIO_IRQn, 15);
      NVIC_ClearPendingIRQ(RADIO_IRQn);
      NVIC_EnableIRQ(RADIO_IRQn);
      NRF_RADIO->INTENSET = B1000;  //interrupt MCU if a packet is received.
      while (NRF_RADIO->INTENSET != B1000) {}  //wait until confirmed
          
      //NRF_GPIOTE->INTENSET = 1;  //interrupt MCU if change detected on pin P0.16.
      //while (NRF_GPIOTE->INTENSET != 1) {}  //wait until confirmed
      
      Serial.print("LFCLKSTAT=0X");
      Serial.print(NRF_CLOCK->LFCLKSTAT,HEX);
      Serial.print("=B");
      Serial.println(NRF_CLOCK->LFCLKSTAT,BIN);
      Serial.flush();
    
      NRF_RADIO->FREQUENCY=123;
      NRF_RADIO->MODE=1;  //set 2Mbps datarate.
      NRF_RADIO->MODECNF0=1;  //enable fast ramp-up of radio from DISABLED state.
    
      if (!nrf51.init())
        Serial.println("init failed");
      mySetNetworkAddress(my_default_network_address, sizeof(my_default_network_address));   
      
      if ((NRF_CLOCK->LFCLKSTAT)>>2) {//if LF clock is running
        NRF_CLOCK->TASKS_LFCLKSTOP=1;  //stop the clock
        while ((NRF_CLOCK->LFCLKSTAT)& 1)  {}  //busy-wait until LF clock has stopped running
        Serial.println("LF clock is stopped.");
      }
    
      Serial.print("LFCLKSTAT=0x");
      Serial.print(NRF_CLOCK->LFCLKSTAT,HEX);
      Serial.print("=B");
      Serial.println(NRF_CLOCK->LFCLKSTAT,BIN);
      Serial.flush();
      
      NRF_CLOCK->LFCLKSRC=1;  //use the crystal oscillator.
      while (!(NRF_CLOCK->LFCLKSRC=1)) {}  //
    
      Serial.println("Crystal oscillator is now the LF choice.");
      Serial.print("LFCLKSTAT=0x");
      Serial.print(NRF_CLOCK->LFCLKSTAT,HEX);
      Serial.print("=B");
      Serial.println(NRF_CLOCK->LFCLKSTAT,BIN);
      Serial.flush();
     
      NRF_CLOCK->TASKS_LFCLKSTART=1;  //start the crystal oscillator clock
    
    
      
      while (!(NRF_CLOCK->EVENTS_LFCLKSTARTED)) {}  //busy-wait until the clock is confirmed started.
      Serial.println("Low Frequency crystal oscillator started.");
      Serial.flush();
    
    
      Serial.print("LFCLKSTAT=0x");
      Serial.print(NRF_CLOCK->LFCLKSTAT,HEX);
      Serial.print("=B");
      Serial.println(NRF_CLOCK->LFCLKSTAT,BIN);
      
      Serial.print("LFCLKSRC=0x");
      Serial.print(NRF_CLOCK->LFCLKSRC,HEX);
      Serial.print("=B");
      Serial.println(NRF_CLOCK->LFCLKSRC,BIN);
      
      Serial.print("Present PRESCALER=");
      Serial.println(NRF_RTC0->PRESCALER);
    
      Serial.print("LFCLKSTAT=0x");
      Serial.print(NRF_CLOCK->LFCLKSTAT,HEX);
      Serial.print("=B");
      Serial.println(NRF_CLOCK->LFCLKSTAT,BIN);
      NRF_RTC0->PRESCALER=0;  //32768 frequency
      while (NRF_RTC0->PRESCALER!=0)  {}  //busy-wait until pre-scaler changes
      Serial.print("New PRESCALER=");
      Serial.println(NRF_RTC0->PRESCALER);
    
      Serial.println();
      Serial.println("Finished setup.");
      Serial.flush();
    
      //Radio enters RXIDLE soon after COUNTER overflows.
      NRF_RTC0->CC[0] = 24;  //the time to exit RX state.
      NRF_RTC0->CC[1] = 25;  //the time to put radio to sleep from RXIDLE
      NRF_RTC0->CC[2] = 3300;  //the time to restart the cycle
    
      NRF_RTC0->EVENTS_COMPARE[0] = 0;  //Clear the event flag.
      NRF_RTC0->EVENTS_COMPARE[1] = 0;  //Clear the event flag.
      NRF_RTC0->EVENTS_COMPARE[2] = 0;  //Clear the event flag.
        
      NRF_RADIO->TASKS_DISABLE=1;  //sleep the radio
      while (NRF_RADIO->STATE) {}; //wait until radio is DISABLED (i.e. STATE=0);
    
      //Configure GPIOTE 
      NRF_GPIOTE->CONFIG[0]=0x31001;  // Toggle Event; Pin P0.16; Event Mode 
                                      //So, any pin change on P0.16 will trigger an event.
                                       //On the Nordic nRF52 DK, Pin 0.16 is a button.
                                       
      NRF_GPIOTE->CONFIG[1]=0x131203;  //Initial setting: HIGH; Toggle Task; Pin P0.18; Task Mode
                                       //On the Nordic nRF52 DK, Pin 0.18 is an LED.
                                       //Note: initial setting of HIGH means that the LED will be initially OFF.
    
      NRF_GPIOTE->CONFIG[2]=0x131103;  //Initial setting: HIGH; Toggle Task; Pin P0.17; Task Mode
                                       //On the Nordic nRF52 DK, Pin P0.17 is an LED.
                                       //Note: initial setting of HIGH means that the LED will be initially OFF.          
    
      
      //Note: radio is assumed to be sleeping by this point, and with high frequency crystal oscillator turned off.
      NRF_PPI->CH[0].EEP = (uint32_t)&NRF_RTC0->EVENTS_OVRFLW;  //when COUNTER overflows.
      NRF_PPI->CH[0].TEP = (uint32_t)&NRF_CLOCK->TASKS_HFCLKSTART;  //turn on the HF crystal oscillator
    
      NRF_PPI->CH[1].EEP = (uint32_t)&NRF_CLOCK->EVENTS_HFCLKSTARTED;  //After HF Clock started.
      NRF_PPI->CH[1].TEP = (uint32_t)&NRF_RADIO->TASKS_RXEN;  //turn on the radio receiver
      //NRF_PPI->FORK[1].TEP = (uint32_t)&NRF_GPIOTE->TASKS_OUT[0]; //Make pin P0.18. be HIGH
    
    
      NRF_PPI->CH[2].EEP = (uint32_t)&NRF_RADIO->EVENTS_READY;  //After event READY, radio shall be in state RXIDLE.
      NRF_PPI->CH[2].TEP = (uint32_t)&NRF_RADIO->TASKS_START;  //Move from RXIDLE mode into RX mode.
    
      NRF_PPI->CH[3].EEP = (uint32_t)&NRF_RTC0->EVENTS_COMPARE[0];  // If time to turn off the radio receiver
      NRF_PPI->CH[3].TEP = (uint32_t)&NRF_RADIO->TASKS_STOP; //Move radio from RX mode back into RXIDLE mode
      
      NRF_PPI->CH[4].EEP = (uint32_t)&NRF_RTC0->EVENTS_COMPARE[1];  // If enough time has passed
      NRF_PPI->CH[4].TEP = (uint32_t)&NRF_RADIO->TASKS_DISABLE; //Sleep the radio
      NRF_PPI->FORK[4].TEP = (uint32_t)&NRF_CLOCK->TASKS_HFCLKSTOP; //Turn off the high frequency crystal oscillator 
      
      NRF_PPI->CH[5].EEP = (uint32_t)&NRF_RTC0->EVENTS_COMPARE[2];  // If 100ms has passed
      NRF_PPI->CH[5].TEP = (uint32_t)&NRF_RTC0->TASKS_TRIGOVRFLW; //Set COUNTER so that it will overflow in 16 ticks.
    
      NRF_PPI->CH[6].EEP = (uint32_t)&NRF_RADIO->EVENTS_END;  //packet received.
      NRF_PPI->CH[6].TEP = (uint32_t)&NRF_GPIOTE->TASKS_OUT[1]; //Make pin P0.18. be LOW (turn on the LED).
    
      NRF_PPI->CH[7].EEP = (uint32_t)&NRF_GPIOTE->EVENTS_IN[0];  //P0.16 pin change occured.
      NRF_PPI->CH[7].TEP = (uint32_t)&NRF_GPIOTE->TASKS_OUT[2]; //Make pin P0.17. be LOW (turn on the LED).
      
      NRF_PPI->CHENSET=B11111111; //enable Channels 7,6,5,4,3,2,1,and 0.
      
      NRF_CLOCK->TASKS_LFCLKSTART=1;  //start the crystal oscillator clock
      
      while (!(NRF_CLOCK->EVENTS_LFCLKSTARTED)) {}  //busy-wait until the clock is confirmed started.
      Serial.println("Crystal oscillator started.");
      Serial.flush();
    
      NRF_RTC0->EVTENSET=0x70003;  //enable routing of RTC TICK, OVRFLW, and comparison events to PPI for the comparisons
      while (NRF_RTC0->EVTEN!= (0x70003)) {};  //wait until EVTENSET setting is confirmed.
      
      NRF_RTC0->EVENTS_TICK=0;  //clear TICKS flag
      NRF_RTC0->EVENTS_OVRFLW=0;  //clear overflow flag
      NRF_RADIO->EVENTS_END=0;  //clear payload received flag.
    
      NRF_CLOCK->TASKS_HFCLKSTOP=1;  //Turn off the high frequency crystal oscillator.
      NRF_RTC0->TASKS_TRIGOVRFLW=1;  //prepare COUNTER so that an overflow will ensue in 16 ticks.
      while (NRF_RTC0->COUNTER!=0xFFFFF0) {} //wait until COUNTER is primed to overflow.
      NRF_RTC0->TASKS_START=1;  //  Resume the RTC, which had been paused.
      while ((NRF_RTC0->EVENTS_TICK==0)) {}  //wait until the radio is confirmed to be started.
    }
    uint32_t loopCounter=0;
    void loop() {
    
      
      Serial.print("time=");
      Serial.print(millis());
      Serial.print(", packetCounter=");
      Serial.println(packetCounter);
      
      Serial.flush();
      
      myHwSleep(500000000);  //sleep 100ms.  Already offset in setup by 1ms from wake-up of PPI.
    }
    
    
    // * Reset events and read back on nRF52
    //* http://infocenter.nordicsemi.com/pdf/nRF52_Series_Migration_v1.0.pdf
     
    #if __CORTEX_M == 0x04
    #define NRF5_RESET_EVENT(event)                                                 \
            event = 0;                                                                   \
            (void)event
    #else
    #define NRF5_RESET_EVENT(event) event = 0
    #endif
    
    
    // This must be in one line
    extern "C" { void RADIO_IRQHandler(void) {packetCounter++; NRF5_RESET_EVENT(NRF_RADIO->EVENTS_END); NRF_RADIO->EVENTS_END=0; }}
    
    
    

    The PPI does toggle an LED each time it receives a packet, but unfortunately the CPU remains asleep. Nonetheless, it does seem to follow your prescription for the ISR.



  • @NeverDie
    What if you activate a pin change interrupt/wake at the pin the PPI toggles?
    Or connect it to another pin with pin change wake?


  • Hero Member

    @Uhrheber said in nRF5 Bluetooth action!:

    @NeverDie
    What if you activate a pin change interrupt/wake at the pin the PPI toggles?
    Or connect it to another pin with pin change wake?

    Yes, I have tested that as a workaround. It "works", but using GPIO pins increases the current drain.


  • Hero Member

    Well, I guess it's almost moot now, because I found and tested a better workaround. I re-wrote the PPI code so that after packet receipt, the PPI triggers an RTC0 overflow. It is that which is then used to wake-up the MCU. No GPIO pins need be involved, so no propagation delays and no increase in current drawn. It works.

    I still think the radio ISR code (above) should have worked, but fortunately that's no longer holding me back now that I have a good enough workaround. 🙂


  • Hero Member

    However, there's one fly in the ointment remaining. It turns out that some other timer is sometimes waking up the CPU:

    time=15798, Radio STATE=0, COUNTER=0x49, packetCounter=22
    time=15900, Radio STATE=0, COUNTER=0x49, packetCounter=23
    time=16001, Radio STATE=0, COUNTER=0x49, packetCounter=24
    time=16103, Radio STATE=0, COUNTER=0x49, packetCounter=25
    time=16204, Radio STATE=0, COUNTER=0x49, packetCounter=26
    time=16306, Radio STATE=0, COUNTER=0x49, packetCounter=27
    time=512000, Radio STATE=0, COUNTER=0x2035, packetCounter=27
    time=1024000, Radio STATE=0, COUNTER=0x269, packetCounter=27
    time=1536000, Radio STATE=0, COUNTER=0x1803, packetCounter=27
    time=2048000, Radio STATE=3, COUNTER=0x36, packetCounter=27
    time=2560000, Radio STATE=0, COUNTER=0x1570, packetCounter=27
    time=3072000, Radio STATE=0, COUNTER=0x3104, packetCounter=27
    time=3584000, Radio STATE=0, COUNTER=0x1337, packetCounter=27
    time=4096000, Radio STATE=0, COUNTER=0x2871, packetCounter=27
    time=4608000, Radio STATE=0, COUNTER=0x1104, packetCounter=27
    time=5120000, Radio STATE=0, COUNTER=0x2638, packetCounter=27
    

    All the lines labelled packetCounter=27 (after the first one that is) are a result of this. Looking at the time, they appear to happen on the rollover of some other timer (?)--apparently the one that is responsible for keeping track of millis(). I can filter them out after-the-fact, but I'd rather they not be waking up the CPU for no reason, as that is just a waste of energy.


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    @d00616
    Can you see any reason as to why the radio isn't waking the MCU after it receives a packet? Here's the entire sketch:

    I have no Idea why. The code is looking fine.


  • Hero Member

    FWIW, I noticed on the oscilliscope that turning on-and-off the HFCLK ten times a second produces a fair amount of ringing. If I simply leave HFCLK turned on, most of the ringing is eliminated.

    [Edit: So, if doing this as part of an aggressive energy saving approach (for instance, turning OFF HFCLK after RX mode and later turning it on again before initiating a new RX), what sort of extra circuitry beyond the two inductors for the DCDC might be needed? I don't know that the ringing is causing any actual problems, but it doesn't look proper on a scope. For now, I'm just flagging it so that folks are aware of it as a possible issue. ]


  • Hero Member

    To better quantify the issue, I measured sleep currents (now using sleep routines that are a fork from what's in mysensors.h), and with the High Frequency clock turned OFF, the sleep current is measured at 2.2ua using a uCurrent Gold. However, the same setup, but with the High Frequency clock left ON, the sleep current is measured at 596ua using the same a uCurrent Gold.

    So, clearly, for a battery/supercap application, leaving the High Frequency clock running all the time is not an especially good option.



  • @NeverDie
    I am puzzled with your 596ua?
    I thought you were under 10ua with the mysensors sleep some time ago?
    Mine only measures 4-5ua when in sleep?


  • Hero Member

    This post is deleted!

  • Hero Member

    @rmtucker said in nRF5 Bluetooth action!:

    I am puzzled with your 596ua?
    I thought you were under 10ua with the mysensors sleep some time ago?

    There's no contradiction. It's a different scenario. The MySensors hwSleep function turns off the High Frequency oscillator when sleeping and turns it back on when it wakes up. So, it's perfectly fine for sleeping your device, having it wake up to send something, and then go back to sleep.

    The present scenario that I'm working on though is where the MCU sleeps and the PPI manages a "listen mode" where the PPI wakes up the radio once every 100ms for a roughly 200us window of time to listen for an incoming packet. Then it goes back to sleep if nothing is received. On the other hand, if a packet is received, it wakes up the MCU so that the packet can be read and dealt with. Presently I have the PPI turn off the high-frequency oscillator each time after it has finished RX in the listen-mode cycle. Before entering RX again to listen for a new packet, it first ramps up the high frequency oscillator. According to the datasheet, the high frequency crystal oscillator must be operating in order for the radio to either transmit or receive. i.e. it can't simply run off the high frequency RC oscillator the way the MCU can.

    My measurements show that while the High Frequency oscillator is running, it consumes about 596ua.



  • Did anyone managed to get two NRF52832 to connect to each other with the arduino IDE and communicate?

    Also why does I2C initializes only after an SWD programmed gets connected?

    Wierd phenomenon when I use an I2C oled display with that chip and program it with an st link V2 after it displays alright when it's booted if the SWD programmer is connected but as soon as you disconnect it everything else works but the I2C display...


  • Hero Member

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Did anyone managed to get two NRF52832 to connect to each other with the arduino IDE and communicate?

    Yes. @d00616's demo code will do this.

    I don't know the answers to the rest of your questions, because I don't use the ST.


  • Hero Member

    Here's a scopeshot of how the revised current draw looks:
    0_1506116792416_NewFile7.png
    As you can see, there is now about a 370us warm-up time at the beginning for the High Frequency oscillator to come up to speed before the RX cycle can be started. Then it takes about 100us for the receiver to warm-up to RXIDLE. From there it finally achieves about 200us of actual productive RX time. Then everything powers down until the end of the 100ms cycle, after which it all repeats again. To conserve energy, all this is managed by the PPI while the MCU sleeps.
    Scale: 1mv=1ma

    I think this is about as energy efficient as it's ever going to get, short of chipping away at the number of bits in the frame/packet size, as I indicated earlier.



  • @NeverDie said in nRF5 Bluetooth action!:

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Did anyone managed to get two NRF52832 to connect to each other with the arduino IDE and communicate?
    

    Yes. @d00616's demo code will do this.

    I don't know the answers to the rest of your questions, because I don't use the ST.

    Any idea how I reach to this code?

    Also you say you don't experience any issues with I2C like that?


  • Hero Member

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Any idea how I reach to this code?

    Yes, it's all explained in detail by @d00616 here: https://www.openhardware.io/view/376/MySensors-NRF5-Platform

    Also you say you don't experience any issues with I2C like that?

    Haven't tried I2C on this platform yet. I'd be very surprised if it didn't work though, as that's ARM Cortex M4 stuff, which is well vetted. i.e. no real dependency on anything Nordic per se.



  • @NeverDie said in nRF5 Bluetooth action!:

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Any idea how I reach to this code?

    Yes, it's all explained in detail by @d00616 here: https://www.openhardware.io/view/376/MySensors-NRF5-Platform

    Also you say you don't experience any issues with I2C like that?

    Haven't tried I2C on this platform yet. I'd be very surprised if it didn't work though, as that's ARM Cortex M4 stuff, which is well vetted. i.e. no real dependency on anything Nordic per se.

    The link you attached links me to a getting started page not wireing two nrf's together


  • Hero Member

    @Mike_Lemo said in nRF5 Bluetooth action!:

    The link you attached links me to a getting started page not wireing two nrf's together

    Your question was ambiguous. When you said "connect" I just assumed you meant wirelessly connect.

    Sorry, I can't help you. Seems like @scalz has gotten I2C to work with it though, but more likely for reading a TH sensor than for the purpose of wiring two nRF52832's. Still, that would prove that it works. If you know the I2C protocol, it shouldn't be hard to go from that to wiring two nRF52832's together.


  • Hero Member

    @NeverDie said in nRF5 Bluetooth action!:

    I think this is about as energy efficient as it's ever going to get, short of chipping away at the number of bits in the frame/packet size, as I indicated earlier.

    I also measured the current between the peaks shown in the scopeshot. By increasing the period between listens, I was able to do the measurement using a uCurrent Gold. Doing so, I found that the current drawn was 10.7ua using the Low Frequency crystal oscillator, and 11.2ua using the Low Frequency RC oscillator. I'm now sure how to square that with some of the earlier measurements I had taken with the nRF52832 sleeping using the MySensors sleep routine, as those measurements came out to about 6ua. Perhaps the difference is the extra current required to run the PPI in this configuration? With neither oscillator configured, and no PPI, it measures, as I said earlier, at 2.2ua, which is close to what the datasheet predicts.



  • @Mike_Lemo said in nRF5 Bluetooth action!:

    @NeverDie said in nRF5 Bluetooth action!:

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Any idea how I reach to this code?

    Yes, it's all explained in detail by @d00616 here: https://www.openhardware.io/view/376/MySensors-NRF5-Platform

    Also you say you don't experience any issues with I2C like that?

    Haven't tried I2C on this platform yet. I'd be very surprised if it didn't work though, as that's ARM Cortex M4 stuff, which is well vetted. i.e. no real dependency on anything Nordic per se.

    The link you attached links me to a getting started page not wireing two nrf's together

    Yes I did mean wirelessly like central and peripherial connection... Is that supported?


  • Hero Member

    I'm sorry, but I can't help you any more than I already have.


  • Hero Member

    @NeverDie said in nRF5 Bluetooth action!:

    I think this is about as energy efficient as it's ever going to get

    Epilog: I ran it overnight on a 10F capacitor, and it dropped only 0.011v per hour. I'm very happy with that, considering it's listening every 100ms as to whether or not it has received a packet. 🙂


  • Hero Member

    @d00616 Your posting says,

    At the moment on Arduino, there is no definition of various OUTPUT modes. If you want to access all nRF5 output modes, you have to use hwPinMode and the OUTPUT_... macro.

    Exactly which macro would that be? It looks to me as though what most users will want is the function nrf5_pinmode(..,..), which appears to do all the actual work. Is that right? It is defined in the file nrf5_wiring_digital.c.

    Meanwhile, hwPinMode appears to be merely a straight pass-through for pinMode:

    
    void hwPinMode(uint8_t pin, uint8_t mode)
    {
      pinMode(pin, mode);
    }
    


  • Where is it possible to find a reference schematic for using the NTF52832 E73-2G4M04S module with NFC?

    not much is being given in the datasheet not even where the NFC pins go.


  • Contest Winner

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Where is it possible to find a reference schematic for using the NTF52832 E73-2G4M04S module with NFC?
    not much is being given in the datasheet not even where the NFC pins go.

    Please look into the product documentation:
    http://infocenter.nordicsemi.com/topic/com.nordic.infocenter.nrf52832.ps.v1.1/nfc.html?cp=2_1_0_41_8#concept_ryw_4hk_1s

    @NeverDie said in nRF5 Bluetooth action!:

    At the moment on Arduino, there is no definition of various OUTPUT modes. If you want to access all nRF5 output modes, you have to use hwPinMode and the OUTPUT_... macro.

    Exactly which macro would that be? It looks to me as though what most users will want is the function nrf5_pinmode(..,..), which appears to do all the actual work. Is that right? It is defined in the file nrf5_wiring_digital.c.

    hwPinMode allows to define platform specific PinMode replacements. Code may be portable. This is the reason pointing to nrf5_pinmode().

    nrf5_pinmode() has a little bit more functionality than the original pinmode function.

    Meanwhile, hwPinMode appears to be merely a straight pass-through for pinMode:

    void hwPinMode(uint8_t pin, uint8_t mode)
    {
    pinMode(pin, mode);
    }

    This disables the capability using nRF5 specific pin modes with the MySensors API.



  • @d00616 said in nRF5 Bluetooth action!:

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Where is it possible to find a reference schematic for using the NTF52832 E73-2G4M04S module with NFC?
    not much is being given in the datasheet not even where the NFC pins go.

    Please look into the product documentation:
    http://infocenter.nordicsemi.com/topic/com.nordic.infocenter.nrf52832.ps.v1.1/nfc.html?cp=2_1_0_41_8#concept_ryw_4hk_1s

    @NeverDie said in nRF5 Bluetooth action!:

    At the moment on Arduino, there is no definition of various OUTPUT modes. If you want to access all nRF5 output modes, you have to use hwPinMode and the OUTPUT_... macro.

    Exactly which macro would that be? It looks to me as though what most users will want is the function nrf5_pinmode(..,..), which appears to do all the actual work. Is that right? It is defined in the file nrf5_wiring_digital.c.

    hwPinMode allows to define platform specific PinMode replacements. Code may be portable. This is the reason pointing to nrf5_pinmode().

    nrf5_pinmode() has a little bit more functionality than the original pinmode function.

    Meanwhile, hwPinMode appears to be merely a straight pass-through for pinMode:

    void hwPinMode(uint8_t pin, uint8_t mode)
    {
    pinMode(pin, mode);
    }

    This disables the capability using nRF5 specific pin modes with the MySensors API.

    I'm talking about the module it's self isn't there a reference schematic for that? I see there are some component in there but else do I have to add to make this work?


  • Hardware Contributor

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Yes I did mean wirelessly like central and peripherial connection... Is that supported?

    You're mixing things maybe. You're talking about bluetooth. that's not Mysensors 😉
    But if you want to get a connection between two nrf52832 or nrf52832/nrf24, take a look at d00616 docs.

    a nrf52 is a nrf52, no matter the module.

    So nfc pins (which are fixed) will be the same on every nrf52 you'll find. It can just happen that you get a board where those pins are used for other purpose (then you can't use nfc without little hack).

    But regarding the cdebyte modules, these are simply nrf52 with pinout. So no problem here. Just take a look at the nordic link d00616 showed about using nfc.

    In case.. pins are P0.09 and P0.10. But you'll need to tune your nfc antenna, and add capacitors. Sparkfun, adafruit have some infos on this as they're selling boards.


  • Hero Member

    Also, if you're interested in NFC, the Nordic nRF52832 DK comes with an antenna for it. It would probably be the easiest to use, because a connector for the antenna is already on the board.



  • @scalz

    Yeah but my question was about what is connected in the module and what components I have to use....

    Also about the NFC I'm planning to use it with the arduino IDE so just wanted to ask if there is a library for it because the SDK is quite useless in this case as well as the Central peripheral connection.


  • Hero Member

    I was able to reduce the active listen period to about 100us:
    0_1506303847635_NewFile3.png
    Now listening every 100ms yields a 10F supercap voltage measured decline of just 9mv per hour. i.e. a decline of 0.108v by the end of 12 hours.


  • Contest Winner

    @Mike_Lemo said in nRF5 Bluetooth action!:

    Also about the NFC I'm planning to use it with the arduino IDE so just wanted to ask if there is a library for it because the SDK is quite useless in this case as well as the Central peripheral connection.

    Now there is a second port of arduino to nRF52. This includes some libraries, like NFC, but they using the SDK. MySensors is currently not ready for this arduino-port.


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Now listening every 100ms yields a 10F supercap voltage measured decline of just 9mv per hour. i.e. a decline of 0.108v by the end of 12 hours.

    Great job. If I'm not wrong the method allows nearly 1 year of listening time with a CR2032.



  • @Mike_Lemo your best bet is to "convert" the module to Arduino Primo and use the NFC libraries developed for it.
    d00616 gave you the link to the arduino org github. Pls. note that Primo core generates a merged softdevice+sketch hex so you should locate it in the Temp folder and upload.


  • Hero Member

    Here's a very simple OPEN/CLOSE remote control I was able to quickly throw together using my small prototyping board:
    0_1506355423385_open_close.jpg
    It required only two buttons, a diode, a resistor, and (obviously) some wire. When not in use, everything is powered 100% OFF to save the most energy possible. So, pushing either button powers it ON, at which point it rapidly determines which button was pushed and then sends the corresponding packet to the receiver. From the standpoint of human perception, it all appears to happen instantly.


  • Hero Member

    This is how the next version of the protoboard will look:
    0_1506377541668_proto1.png 0_1506377552245_proto2.png


  • Hero Member

    Interestingly, it looks as though Arduino is suggesting/recommending users to use the regular Arduino Primo to program the Arduino Primo Core (i.e. the wearable).
    alt text



  • @NeverDie is this SWD?


  • Hero Member

    @Toyman said in nRF5 Bluetooth action!:

    @NeverDie is this SWD?

    I don't have either the Primo or the Primo Core, so I can't say for sure. However, I presume so.



  • @Mike_Lemo
    This sounds more like an EMC issue.
    The connected ST Link filters some disturbance at the i2c lines, and when it's not connected, the disturbance corrupts the signals.
    Try connecting capacitors from the i2c lines to ground. I'd start with 100pF each.
    May also be a power supply issue. Did you connect the st-link's 3.3V line to the board?


  • Hero Member

    Here's the small budget nRF51 soldered to the breakout board that I had linked earlier above:
    0_1506460203841_nrf51bob.jpg


  • Hero Member

    I have the RTC running off the low frequency internal RC, because I don't see a crystal oscillator on the module. I have it blinking an LED now and sending text to the serial port, which I'm able to read on the serial console.

    At least so far, all is good. 🙂


  • Hardware Contributor

    @NeverDie said in nRF5 Bluetooth action!:

    Here's a very simple OPEN/CLOSE remote control I was able to quickly throw together using my small prototyping board:
    0_1506355423385_open_close.jpg
    It required only two buttons, a diode, a resistor, and (obviously) some wire. When not in use, everything is powered 100% OFF to save the most energy possible. So, pushing either button powers it ON, at which point it rapidly determines which button was pushed and then sends the corresponding packet to the receiver. From the standpoint of human perception, it all appears to happen instantly.

    Hello @NeverDie, could you please share the code for this ?
    I'd like to adapt and test it on a reprogrammed nrf51822 beacon that I just received.


  • Hero Member

    @Nca78
    I can, but it wouldn't be proper "mysensors" code, because mysensors code has a 5-10 second power-up delay. So, for that reason, it's using radiohead instead. Do you still want it anyway?


  • Hardware Contributor

    @NeverDie said in nRF5 Bluetooth action!:

    @Nca78
    I can, but it wouldn't be proper "mysensors" code, because mysensors code has a 5-10 second power-up delay. So, for that reason, it's using radiohead instead. Do you still want it anyway?

    Ah I see, probably still interesting to see, please share anyway 🙂



  • I am trying to reprogram a commercial nrf51 module. Under "commerical" I mean "not from aliexpress", but installed in "smart" bluetooth socket.
    I can connect to it with Black Magic Probe just fine, but after mass_erase, BMP reports 0xfffffffe instead of usual 0xffffffff, meaning something is lef behind. That prohibits reflashing of softdevice BUT I can load "plain" skethes that do not require softdevice
    I ASSUME the module has some kind of write protection and/or UICR registers set that are not erased with BMP mass+erase command.
    I've tried to load d0016 uuicr clearing sketch, It loads fine, but I've still got 0xfffffffe after masserase.
    So what's the proper way to really completely erase the module?
    I have nrf52 dk that I tried to use as a programmer but it didn't worked.


  • Hero Member

    @Nca78 said in nRF5 Bluetooth action!:

    @NeverDie said in nRF5 Bluetooth action!:

    @Nca78
    I can, but it wouldn't be proper "mysensors" code, because mysensors code has a 5-10 second power-up delay. So, for that reason, it's using radiohead instead. Do you still want it anyway?

    Ah I see, probably still interesting to see, please share anyway 🙂

    Here it is, warts and all:
    0_1506603795796_remote_control_v031.zip
    It works, but it's just hastily improvised throw-away code, so I made no attempt to polish it. i.e. For anyone reading this: It is very definitely not demo code.



  • @NeverDie This may be old news but I will throw it out anyway. The Reset pin on the nRF52 series is both a GPIO and RESET line (one or the other) . This is not a dedicated Reset however and the application must define that pin as RESET. This is the only pin that can be defined as reset. The Datasheet describes this as well.



  • @Toyman What is the make and model of the module. You can use the nRF52-DK to erase the device. P20 is a ease access to the programming pins on that board. You need to power up the module, run the VDD from the module back to the dev kit , P20, Pin2. Hook up the SWDIO lines to Pin3 and the SWDCLK to Pin4. and Ground, Pin8. As reset is not hardwired you don't have to hook that up unless you want to (Pin7. . ![alt text](image url) You can use nRFJPROG to exersise the J-Link (Segger). This is found in the downloadable tools on the Nordic site. BTW - I have no problem using J-Link Commander version 6.20c with the nRF52-DK with the nRF52832 mounted. I hope this helped. 0_1506616486979_Programming using P20 on the dev kit.JPG


  • Hero Member

    @Jokgi said in nRF5 Bluetooth action!:

    @NeverDie This may be old news but I will throw it out anyway. The Reset pin on the nRF52 series is both a GPIO and RESET line (one or the other) . This is not a dedicated Reset however and the application must define that pin as RESET. This is the only pin that can be defined as reset. The Datasheet describes this as well.

    Yes, and thanks to @d00616's work, RESET can also be enabled/disabled from the Tools menu of the arduino IDE if using myNRF5Board as the board definition.


  • Hero Member

    I just now compared the transmission range and packet loss of the cheap nRF51822 to the Ebyte nRF52832. I expected that the Ebyte, with its bigger and fancier antenna, to mop the floor with the nRF51822, but I was surprised to find that they actually seem fairly comparable (at least in my informal, off-the-cuff testing). So, if you have a sensor that's mostly transmission oriented, like, say, a TH sensor, the smaller and cheaper nRF51822 is maybe worth considering.



  • @NeverDie said in nRF5 Bluetooth action!:

    the smaller and cheaper nRF51822 is maybe worth considering.

    That is interesting. What sort of range where you getting?

    Do you have the 840 dev kit? If so, can you measure range on that guy? Should be 10x I would think.


  • Hero Member

    @Terrence

    Around 30 feet, through some walls, but at 2mbps. I realize that's not very far, but even so the measured packet loss (informal testing) is pretty high at that speed. I can see why most people default to 250kbps instead.

    I don't have the 840 dev kit yet, mostly because I don't see anyother 840 modules on the market right now.


  • Hero Member

    What's a good initial value to use for DATAWHITEIV, if doing data whitening?



  • @Jokgi Thanks. That's exactly what I did. Flashed successfully but ideally I'd like to find a way to power the module from the DK (I had to use the external power)
    Unfortunately, this was the first time when BMP let me down. It just failed to load either softdevice or the sketch hex while Jlink@DK handled it without any problem.


  • Hero Member

    @d00616
    Which settings are key for ensuring that packets sent by the nRF24 are correctly received by the nRF5? It would be nice to leverage the versions of the nRF24 that have amplified Tx. For sending wake-up packets, I shoudln't need shockburst, and so maybe this will work fairly easily.


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Which settings are key for ensuring that packets sent by the nRF24 are correctly received by the nRF5? It would be nice to leverage the versions of the nRF24 that have amplified Tx.

    These are more than settings. You have to set the Radio configuration like in Radio_ESB.cpp, reverse the addresses and handle sending ACK packages by software.

    This is a good resource to understand the OTA protocol: https://hackaday.io/project/11942-antenna-diversity-receive-and-transmit/log/39510-shockburst-vs-enhanced-shockburst-nrf24-vs-nrf5x

    I can provide a simple example how to use the MySensors transport code without Radio Head. This works for nRF5 and nRF24 modules, but its outside of that what the MY_CORE_ONLY mode is designed for.

    #define MY_CORE_ONLY
    
    #ifndef ARDUINO_ARCH_NRF5
    #define MY_NODE_ID (1)
    #define SND_TO (2)
    #else
    #define MY_NODE_ID (2)
    #define SND_TO (1)
    #endif
    
    // Enable debug
    #define MY_DEBUG
    //#define MY_DEBUG_VERBOSE_RF24
    //#define MY_DEBUG_VERBOSE_NRF5_ESB
    
    
    // RF24_250KBPS RF24_1MBPS RF24_2MBPS
    #define MY_RF24_DATARATE (RF24_1MBPS)
    // NRF5_250KBPS NRF5_1MBPS NRF5_2MBPS
    #define MY_NRF5_ESB_MODE (NRF5_1MBPS)
    
    
    // Enable and select radio type attached
    #ifndef NRF5
    #define MY_RADIO_NRF24
    #else
    #define MY_RADIO_NRF5_ESB
    #endif
    
    #include <MySensors.h>
    
    void setup() {
      Serial.begin(115200);
      
      hwInit();
      transportInit();
      transportSetAddress(MY_NODE_ID);
    }
    
    void loop() {
      // Check for packages
      while (transportAvailable()) {
        uint8_t buffer[256];
        uint8_t num = transportReceive(&buffer);
        Serial.print("Data=");
        for (int i=0;i<num;i++) {
          if (buffer[i]<0x10) Serial.print("0");
          Serial.print(buffer[i], HEX);
          Serial.print(" ");
        }
        Serial.println();
      }
    
      wait(1000);
    
      // Send data
      transportSend(SND_TO, "abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz",32, false);
    }
    

  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Yes, and thanks to @d00616's work, RESET can also be enabled/disabled from the Tools menu of the arduino IDE if using myNRF5Board as the board definition.

    The reset support isn't complete at the moment. The menu is active, but it has no effect at the moment. I fix this with release 0.2.0.



  • I just received 2 of those little boards.
    Ideal for small sensor nodes, I'd say, but not very breadboard friendly.
    So I dug out the verowire, and did a little soldering.

    0_1506700720203_IMG_20170929_174130.jpg


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    @NeverDie said in nRF5 Bluetooth action!:

    Which settings are key for ensuring that packets sent by the nRF24 are correctly received by the nRF5? It would be nice to leverage the versions of the nRF24 that have amplified Tx.

    These are more than settings. You have to set the Radio configuration like in Radio_ESB.cpp, reverse the addresses and handle sending ACK packages by software.

    This is a good resource to understand the OTA protocol: https://hackaday.io/project/11942-antenna-diversity-receive-and-transmit/log/39510-shockburst-vs-enhanced-shockburst-nrf24-vs-nrf5x

    I can provide a simple example how to use the MySensors transport code without Radio Head. This works for nRF5 and nRF24 modules, but its outside of that what the MY_CORE_ONLY mode is designed for.

    #define MY_CORE_ONLY
    
    #ifndef ARDUINO_ARCH_NRF5
    #define MY_NODE_ID (1)
    #define SND_TO (2)
    #else
    #define MY_NODE_ID (2)
    #define SND_TO (1)
    #endif
    
    // Enable debug
    #define MY_DEBUG
    //#define MY_DEBUG_VERBOSE_RF24
    //#define MY_DEBUG_VERBOSE_NRF5_ESB
    
    
    // RF24_250KBPS RF24_1MBPS RF24_2MBPS
    #define MY_RF24_DATARATE (RF24_1MBPS)
    // NRF5_250KBPS NRF5_1MBPS NRF5_2MBPS
    #define MY_NRF5_ESB_MODE (NRF5_1MBPS)
    
    
    // Enable and select radio type attached
    #ifndef NRF5
    #define MY_RADIO_NRF24
    #else
    #define MY_RADIO_NRF5_ESB
    #endif
    
    #include <MySensors.h>
    
    void setup() {
      Serial.begin(115200);
      
      hwInit();
      transportInit();
      transportSetAddress(MY_NODE_ID);
    }
    
    void loop() {
      // Check for packages
      while (transportAvailable()) {
        uint8_t buffer[256];
        uint8_t num = transportReceive(&buffer);
        Serial.print("Data=");
        for (int i=0;i<num;i++) {
          if (buffer[i]<0x10) Serial.print("0");
          Serial.print(buffer[i], HEX);
          Serial.print(" ");
        }
        Serial.println();
      }
    
      wait(1000);
    
      // Send data
      transportSend(SND_TO, "abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz",32, false);
    }
    

    I tried compiling it using the Arduino Windows IDE, but for some reason it complains about not finding a whole litany of .h files: socket.h, w5100.h, netdb.h.... Does mysensors.h really need to drag in all of those .h files, even if only indirectly? Or, if there's an easy way to make it find them, what is it?


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    I tried compiling it using the Arduino Windows IDE, but for some reason it complains about not finding a whole litany of .h files: socket.h, w5100.h, netdb.h.... Does mysensors.h really need to drag in all of those .h files, even if only indirectly? Or, if there's an easy way to make it find them, what is it?

    I can compile it with Linux. What are missing is part of the Ethernet library. I don't know why it's included in you build.


  • Hero Member

    I just now upgraded to the current release of the mysensors development library, and the compile problem went away. 🙂 So, if anyone else encounters the same problem, I recommend doing that.


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    I just now upgraded to the current release of the mysensors development library, and the compile problem went away. So, if anyone else encounters the same problem, I recommend doing that.

    Sorry. I have forgotten my uncommited changes. At the moment, you have to start the HFCLK in the CORE_ONLY mode. This is done in hwInit() later.


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    @NeverDie said in nRF5 Bluetooth action!:

    I just now upgraded to the current release of the mysensors development library, and the compile problem went away. So, if anyone else encounters the same problem, I recommend doing that.

    ... At the moment, you have to start the HFCLK in the CORE_ONLY mode. ...

    Do you have a revised sketch which does that? I tried running the sketch you gave above on an nRF52 DK, but it immediately goes into a boot-loop, wherein in keeps rebooting itself, over and over and over again.


  • Hero Member

    @NeverDie said in nRF5 Bluetooth action!:

    I tried running the sketch you gave above on an nRF52 DK, but it immediately goes into a boot-loop, wherein in keeps rebooting itself, over and over and over again.

    I just now tried running it on a pro-mini using a nRF24, and it also gets into a boot-loop.


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    I just now tried running it on a pro-mini using a nRF24, and it also gets into a boot-loop.

    Please replace wait() with delay(). This is an issue in the transport code, which is triggered while sleep() or wait() is executed.


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    @NeverDie said in nRF5 Bluetooth action!:

    I just now tried running it on a pro-mini using a nRF24, and it also gets into a boot-loop.

    Please replace wait() with delay(). This is an issue in the transport code, which is triggered while sleep() or wait() is executed.

    OK, made that change, and it no longer boot-loops.

    However, neither node appears to be receiving anything from the other.

    Please advise.


  • Hero Member

    @d00616
    Since the original code didn't work, I upgraded it somewhat to give a larger Rx window. However, it still doesn't work:

    #define MY_CORE_ONLY
    
    #ifndef ARDUINO_ARCH_NRF5
    #define MY_NODE_ID (1)
    #define SND_TO (2)
    #else
    #define MY_NODE_ID (2)
    #define SND_TO (1)
    #endif
    
    // Enable debug
    #define MY_DEBUG
    //#define MY_DEBUG_VERBOSE_RF24
    //#define MY_DEBUG_VERBOSE_NRF5_ESB
    
    
    // RF24_250KBPS RF24_1MBPS RF24_2MBPS
    #define MY_RF24_DATARATE (RF24_1MBPS)
    // NRF5_250KBPS NRF5_1MBPS NRF5_2MBPS
    #define MY_NRF5_ESB_MODE (NRF5_1MBPS)
    
    
    // Enable and select radio type attached
    #ifndef NRF5
    #define MY_RADIO_NRF24
    #else
    #define MY_RADIO_NRF5_ESB
    #endif
    
    #include <MySensors.h>
    
    void setup() {
      Serial.begin(115200);
      Serial.println("Starting....");
      Serial.print("MY_NODE_ID=");
      Serial.println(MY_NODE_ID);
      Serial.print("SND_TO=");
      Serial.println(SND_TO);
      
      hwInit();
      transportInit();
      transportSetAddress(MY_NODE_ID);
    }
    
    uint32_t theTime=0;
    uint32_t loopCounter=0;
    void loop() {
      // Check for packages
      while ((millis()-theTime)<1000) {
        while (transportAvailable()) {
          uint8_t buffer[256];
          uint8_t num = transportReceive(&buffer);
          Serial.print("Data=");
          for (int i=0;i<num;i++) {
            if (buffer[i]<0x10) Serial.print("0");
            Serial.print(buffer[i], HEX);
            Serial.print(" ");
          }
          Serial.println();
        }
      }
      theTime=millis();
    
      //delay(1000);
      Serial.print(loopCounter++);
      Serial.println(", SENDING:  abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz");
    
      // Send data
      transportSend(SND_TO, "abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz",32, false);
    }
    

    One node is an nRF24 on a pro mini, and the other is an nRF52832.


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Since the original code didn't work, I upgraded it somewhat to give a larger Rx window. However, it still doesn't work:

    Please add this to your setup() function (https://forum.mysensors.org/topic/6961/nrf5-bluetooth-action/985😞

    	// Clock is manged by sleep modes. Radio depends on HFCLK.
    	// Force to start HFCLK
    	NRF_CLOCK->EVENTS_HFCLKSTARTED = 0;
    	NRF_CLOCK->TASKS_HFCLKSTART = 1;
    	while (NRF_CLOCK->EVENTS_HFCLKSTARTED == 0)
    		;
    
    	// Enable low latency sleep mode
    	NRF_POWER->TASKS_CONSTLAT = 1;
    
    	// Enable cache on >= NRF52
    #ifndef NRF51
    	NRF_NVMC->ICACHECNF = NVMC_ICACHECNF_CACHEEN_Msk;
    #endif
    

    At the moment I prepare a new Pull Request fixing this including some small fixes and improvements for nRF5 MCUs. When it's integrated the HFCLK is startet in hwInit().



  • @d00616 is this universal recommendation?


  • Contest Winner

    @Toyman said in nRF5 Bluetooth action!:

    @d00616 is this universal recommendation?

    I think you mean to start the HFCLK. This isn't a universal recommendation. At the moment the HFCLK is started in MyMainNRF5.cpp. This file is ignored in CORE_ONLY mode. I moved any initialization code into hwInit().


  • Contest Winner

    @d00616 said in nRF5 Bluetooth action!:

    At the moment I prepare a new Pull Request fixing this including some small fixes and improvements for nRF5 MCUs. When it's integrated the HFCLK is startet in hwInit().

    The Pull Request is available: https://github.com/mysensors/MySensors/pull/938


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    @d00616 said in nRF5 Bluetooth action!:

    At the moment I prepare a new Pull Request fixing this including some small fixes and improvements for nRF5 MCUs. When it's integrated the HFCLK is startet in hwInit().

    The Pull Request is available: https://github.com/mysensors/MySensors/pull/938

    That link lists changes to the files, but it doesn't seem to provide the new files. Or else I'm overlooking where it does?


  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    That link lists changes to the files, but it doesn't seem to provide the new files. Or else I'm overlooking where it does?

    You have to checkout this pull request: https://help.github.com/articles/checking-out-pull-requests-locally/


  • Hero Member

    @d00616 said in nRF5 Bluetooth action!:

    @NeverDie said in nRF5 Bluetooth action!:

    That link lists changes to the files, but it doesn't seem to provide the new files. Or else I'm overlooking where it does?

    You have to checkout this pull request: https://help.github.com/articles/checking-out-pull-requests-locally/

    Maybe I need write-access or something? Those instructions refer to a command line, and I just don't see one anywhere.

    0_1506801237481_pull.png


  • Hero Member

    Well, anyway, I added code to start the high frequency clock, and now it seems to work:

    #define MY_CORE_ONLY
    
    #ifndef ARDUINO_ARCH_NRF5
    #define MY_NODE_ID (1)
    #define SND_TO (2)
    #else
    #define MY_NODE_ID (2)
    #define SND_TO (1)
    #endif
    
    // Enable debug
    #define MY_DEBUG
    //#define MY_DEBUG_VERBOSE_RF24
    //#define MY_DEBUG_VERBOSE_NRF5_ESB
    
    
    // RF24_250KBPS RF24_1MBPS RF24_2MBPS
    #define MY_RF24_DATARATE (RF24_1MBPS)
    // NRF5_250KBPS NRF5_1MBPS NRF5_2MBPS
    #define MY_NRF5_ESB_MODE (NRF5_1MBPS)
    
    
    // Enable and select radio type attached
    #ifndef NRF5
    #define MY_RADIO_NRF24
    #else
    #define MY_RADIO_NRF5_ESB
    #endif
    
    #include <MySensors.h>
    #include <nrf.h>
    
    void setup() {
      Serial.begin(115200);
      Serial.println("Starting....");
      Serial.print("MY_NODE_ID=");
      Serial.println(MY_NODE_ID);
      Serial.print("SND_TO=");
      Serial.println(SND_TO);
    
      if (MY_NODE_ID==2) {
        NRF_CLOCK->TASKS_HFCLKSTART=1;  //activate the high frequency crystal oscillator
        while ((NRF_CLOCK->EVENTS_HFCLKSTARTED==0)) {};  //wait until high frequency clock start is confirmed
      }
    
    
      hwInit();
      transportInit();
      transportSetAddress(MY_NODE_ID);
    }
    
    uint32_t theTime=0;
    uint32_t loopCounter=0;
    void loop() {
      // Check for packages
      while ((millis()-theTime)<1000) {
        while (transportAvailable()) {
          uint8_t buffer[256];
          uint8_t num = transportReceive(&buffer);
          Serial.print("Data=");
          for (int i=0;i<num;i++) {
            if (buffer[i]<0x10) Serial.print("0");
            Serial.print(buffer[i], HEX);
            Serial.print(" ");
          }
          Serial.println();
        }
      }
      theTime=millis();
    
      //delay(1000);
      Serial.print(loopCounter++);
      Serial.println(", SENDING:  abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz");
    
      // Send data
      transportSend(SND_TO, "abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz",32, false);
    }
    

  • Contest Winner

    @NeverDie said in nRF5 Bluetooth action!:

    Maybe I need write-access or something? Those instructions refer to a command line, and I just don't see one anywhere.

    You can do this with git on your local machine:

    git clone https://github.com/mysensors/MySensors.git
    cd MySensors
    git fetch origin pull/938/head:pr938
    git checkout pr938
    

Log in to reply
 

Suggested Topics

0
Online

11.2k
Users

11.1k
Topics

112.5k
Posts