Issues with Buttonless DFU added to existing project

RE: Issues with Buttonless DFU added to existing project

Vidar Berg — Thu, 24 May 2018 07:44:33 GMT

The bootloader must be present when you initialize the DFU service because it sets up the SVCI interface to the bootloader. This might explain the error in step 2 . However, I don't think it's related to the invalid state errror from nrf_sdh_enable_request(). I'd suggest to step through the nrf_sdh_enable_request function and see if NRF_ERROR_INVALID_STATE is returned because m_nrf_sdh_enabled is 'true', or if the error gets returned from sd_softdevice_enable().

RE: Issues with Buttonless DFU added to existing project

Eduardo Garcia — Wed, 23 May 2018 18:45:12 GMT

Thanks Vidar! That are some good points.I fixed my issue with the WDT on the bootloader and I believe that was the main issue-however we are getting the following error if we debug our device without the bootloader in place:

I have noticed that when I am debugging and restart the device I am getting failures at "nrf_sdh_enable_request();" with error code 0x08. I get this until I do a power cycle. I believe I was getting errors unrelated to this, and when I reset I was getting this issue, which is why I may have confused it with the bootloader. But I tried without the bootloader, got an unrelated error, and after reset kept getting this error!

To clarify:

1.) Erased device, loaded SD S132 and Application via nRF GO and J-LINK (note NO bootloader).

2.) Got an unrelated error (bug in my code, or just reset device with debugger)

3.) Reset debugger to understand what happened

4.) After this reset I keep getting "nrf_sdh_enable_request();" errors on the main application until I power cycle.

This does not happen if I have the bootloader present on step 1.

The code is the same as the one I states some posts before, but I recopy it here:

Application main.c

  int main(void)
{
int32_t err_code = 0;
    int32_t task_error = 0;
   
    
    nrf_delay_ms(1000); //wait after power up
    
    err_code  = NRF_LOG_INIT(NULL);
    NRF_LOG_DEFAULT_BACKENDS_INIT();
    //nrf_dfu_svci_vector_table_unset();
    NRF_LOG_INFO("Hello!");
    NRF_LOG_FLUSH();
    SEGGER_RTT_WriteString(0, "RTT initialized\n");
     
     BLE_init(); //Chrashes inside here
     ...
     }

Inside BLE_Init

int32_t BLE_init(void)
{
    bool erase_bonds;
    uint32_t err_code;

    // Initialize
    ble_stack_init(); //Crashes inside here
    err_code = app_timer_init();
    //...
    }

Thanks!

RE: Issues with Buttonless DFU added to existing project

Vidar Berg — Wed, 23 May 2018 07:39:37 GMT

it was a bit misleading of me to say that the bootloader disables the softdevice before branching to the main app. Actually, the bootloader does not enable the softdevice on normal boot (power on reset, etc), only when it enters DFU mode and initialize the BT transport. After DFU complete, the bootloader will perform a system reset to ensure that the application always boots into the same state as it would after a normal boot.

To debug the bootloader you can step through the nrf_dfu.c::nrf_dfu_init() and check if nrf_bootloader_app_start() is being called. Then try to disable the softdevice after nrf_dfu_mbr_init_sd(); to see if that helps. Although it shouldn't be necessary as the softdevice should be disabled at this point (Master Boot Record (MBR) and SoftDevice initialization procedure)

Also, you should remove nrf_drv_clock_uninit() since you're using the WDT, please refer to this thread for more details: https://devzone.nordicsemi.com/f/nordic-q-a/30677/bootloader-doesn-t-load-application-with-watchdog (fixed in SDK 15 bootloader)

"The unit always seems to be stuck in PC 0x13d6e of SoftDevice."

The program counter will stay at this address most of the time as this is where the __WFE instruction is being called from the softdevice to enter System ON mode (sleep).

RE: Issues with Buttonless DFU added to existing project

Eduardo Garcia — Tue, 22 May 2018 14:34:52 GMT

I forgot to say, the SDK is nRF5_SDK_14.0.0

RE: Issues with Buttonless DFU added to existing project

Eduardo Garcia — Tue, 22 May 2018 14:34:11 GMT

Thank you for your response!

I have not made any changes to the code of the BLE DFU (Other than WDT calls to feed the WDT on the bootloader!). As you can see with the code above the main application only does NRF_LOG and RTT - related code before calling BLE_init() which in turn calls ble_stack_init(). Could you please point me to where the DFU code disables the SD so I can debug and see if it hits that place? I could not find it!

Thanks!

RE: Issues with Buttonless DFU added to existing project

Vidar Berg — Tue, 22 May 2018 11:14:53 GMT

Hello,

The bootloader disables the softdevice before branching to the main application, so it should not return invalid state when you try to re-enable it from the app. Have you made any changes to the default bootloader code that may have changed this behavior? Also, do you have a LF crystal in your design, or are you using the internal RC oscillator?

*EDIT*:

Related to nrf_sdh_enable_request() returning invalid state: are you doing any fstorage operations prior to ble_stack_init()? Looks like the fstorage softdevice backend may enable the softdevice through nrf_sdh_request_continue();

RE: Issues with Buttonless DFU added to existing project

Eduardo Garcia — Thu, 17 May 2018 18:26:02 GMT

Another test, it seems that if I do the following I can get it to work:

*Disable WDT on main application (Don't know as of now how to handle WDT feeding on Bootloader, will search around...)

*Assume that SD can be already initialized: Added this code to "ble_stack_init()"

static void ble_stack_init(void)
{
    ret_code_t err_code;
    ble_cfg_t ble_cfg;
    if(nrf_sdh_is_enabled() == false){
      err_code = nrf_sdh_enable_request();
      APP_ERROR_CHECK(err_code);
    }else
    {
      NRF_LOG_INFO("SD is already enabled!");
    }
    ...
    }

*Assume LF clock is already initialized (on a function after BLE initialization that sets our App's HW):

->Note protection of "nrf_clock_lf_src_set(NRF_CLOCK_LFCLK_Xtal);"

int32_t CLK_lfclk_config(void)
{
  int32_t status = 0;
  status = nrf_drv_clock_init();
  if(status != NRF_ERROR_MODULE_ALREADY_INITIALIZED) //Assumption
  {
    nrf_clock_lf_src_set(NRF_CLOCK_LFCLK_Xtal);
    //APP_ERROR_CHECK(err_code);

    nrf_drv_clock_lfclk_request(NULL);
  }else
  {
    status = 0;
  }
  
  if (status !=0)
    return ((int32_t)CLOCK_ERROR);
  else
    return (0);
}

I kind of understand that the BLE bootloader may initialize the SD and the LF clock, but I am worried to not understand why the "secure_dfu_ble" did not do this. I also worry that I am missing something that might come back to byte me! Is there a list of components to take care of when adding buttonless DFU? Or, am I doing something wrong and because of that I need the changes I implemented?

Thanks!

RE: Issues with Buttonless DFU added to existing project

Eduardo Garcia — Thu, 17 May 2018 17:10:53 GMT

I solved a minor issue, the setting for the bootloader button on the "secure_dfu_ble" example was set up incorrectly, so maybe that was the issue with the need to reset multiple times until it worked. The main issue, however, is still present!

I tested using the "ble_app_buttonless_dfu" application to upload my application and the upload works. However it still crashes on sd_softdevice_enable() with an NRF_ERROR_INVALID_STATE error.

Can it be that the stack is already initialized by the bootloader and when it enters my code and initializes again I get this error?I am not sure what the issue is; I see that the "ble_app_buttonless_dfu" example initializes the BLE stack very early on and it does not de-initialize it. I am doing the same thing.

My code in main.c (needs some cleanup...):

  int main(void)
{
int32_t err_code = 0;
    int32_t task_error = 0;
   
    
    nrf_delay_ms(1000); //wait after power up
    
    err_code  = NRF_LOG_INIT(NULL);
    NRF_LOG_DEFAULT_BACKENDS_INIT();
    //nrf_dfu_svci_vector_table_unset();
    NRF_LOG_INFO("Hello!");
    NRF_LOG_FLUSH();
    SEGGER_RTT_WriteString(0, "RTT initialized\n");
     
     BLE_init(); //Chrashes inside here
     ...
     }

Code inside BLE_Init

int32_t BLE_init(void)
{
    bool erase_bonds;
    uint32_t err_code;

    // Initialize
    ble_stack_init(); //Crashes inside here
    err_code = app_timer_init();
    //...
    }

Crashes on "err_code = nrf_sdh_enable_request();" with a return error of NRF_ERROR_INVALID_STATE.

Thanks