Using app_with_dfu jump to bootloader.

RE: Using app_with_dfu jump to bootloader.

Coldson — Fri, 01 Jul 2016 16:05:34 GMT

Add comment:

    else
    {
        // Keys were not available, thus we have a non-encrypted connection.
        err_code = dm_peer_addr_get(&m_dm_handle, &m_peer_data.addr);
        APP_ERROR_CHECK(err_code);

        err_code = sd_softdevice_vector_table_base_set(NRF_UICR->NRFFW[0]);
        APP_ERROR_CHECK(err_code);
        
        err_code = dfu_ble_svc_peer_data_set(&m_peer_data);
        APP_ERROR_CHECK(err_code);
    }

don't forget to add to this line also.

Fixed!!!!!!!!!

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Fri, 01 Jul 2016 15:19:14 GMT

@Coldson: yes you can do that also, the root cause is that we use UART when we already forwarded the interrupt vector table, using sd_softdevice_vector_table_base_set(). After that call, we should not use any peripheral that can trigger interrupt.

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Fri, 01 Jul 2016 15:18:02 GMT

Just updated my comment above.

RE: Using app_with_dfu jump to bootloader.

Coldson — Fri, 01 Jul 2016 15:17:23 GMT

I provide my work around solution:

Mark all DM_TRC() in dm_peer_addr_get()function.

Then it work fine.

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Fri, 01 Jul 2016 15:04:33 GMT

Hi Coldson,

Good news, I found a bug. It was because we redirect the vector table to the bootloader too early in the bootloader_start() function. So if you cut the 2 lines :

err_code = sd_softdevice_vector_table_base_set(NRF_UICR->NRFFW[0]);
APP_ERROR_CHECK(err_code);

In that function in dfu_app_handler.c.

And paste them after the call to dm_peer_addr_get in the same file, in dfu_app_peer_data_set. It should be ok. It should look like this:

  err_code = dm_handle_get(conn_handle, &m_dm_handle);
    if (err_code == NRF_SUCCESS)
    {
        err_code = dm_distributed_keys_get(&m_dm_handle, &key_set);
        if (err_code == NRF_SUCCESS)
        {
            APP_ERROR_CHECK(err_code);

            m_peer_data.addr              = key_set.keys_central.p_id_key->id_addr_info;
            m_peer_data.irk               = key_set.keys_central.p_id_key->id_info;
            m_peer_data.enc_key.enc_info  = key_set.keys_periph.enc_key.p_enc_key->enc_info;
            m_peer_data.enc_key.master_id = key_set.keys_periph.enc_key.p_enc_key->master_id;


            app_context_data   = (DFU_APP_ATT_TABLE_CHANGED << DFU_APP_ATT_TABLE_POS);
            app_context.len    = sizeof(app_context_data);
            app_context.p_data = (uint8_t *)&app_context_data;
            app_context.flags  = 0;

            err_code = dm_application_context_set(&m_dm_handle, &app_context);
            APP_ERROR_CHECK(err_code);
		err_code = sd_softdevice_vector_table_base_set(NRF_UICR->NRFFW[0]);
		APP_ERROR_CHECK(err_code);
            err_code = dfu_ble_svc_peer_data_set(&m_peer_data);
            APP_ERROR_CHECK(err_code);
        }
        else
        {
            // Keys were not available, thus we have a non-encrypted connection.
            err_code = dm_peer_addr_get(&m_dm_handle, &m_peer_data.addr);
            APP_ERROR_CHECK(err_code);
		err_code = sd_softdevice_vector_table_base_set(NRF_UICR->NRFFW[0]);
		APP_ERROR_CHECK(err_code);

            err_code = dfu_ble_svc_peer_data_set(&m_peer_data);
            APP_ERROR_CHECK(err_code);
        }
    }

RE: Using app_with_dfu jump to bootloader.

Coldson — Fri, 01 Jul 2016 13:15:41 GMT

@Hung Bui, the example hrm_with_dfu s132. I want to let log print on my console, so in this example need to modify log_uart_init() line 175, change APP_UART_FLOW_CONTROL_ENABLED to APP_UART_FLOW_CONTROL_DISABLED, then I can get the log on console. But it will let APPLICATION not jump to bootloader.

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Fri, 01 Jul 2016 13:07:44 GMT

@Coldson: which example I should test ? And I don't understand this "because I need to open UART and let it print on my teraterm.So I need to let APP_UART_FLOW_CONTROL_ENABLED->APP_UART_FLOW_CONTROL_DISABLED, or I will get nothing on my console." We are talking about the issue to jump from application to bootloader when UART is used in the application.

RE: Using app_with_dfu jump to bootloader.

Coldson — Fri, 01 Jul 2016 13:02:44 GMT

nRF5_SDK_11.0.0_89a8197.7z

@Hung Bui, because I need to open UART and let it print on my teraterm.

So I need to let APP_UART_FLOW_CONTROL_ENABLED->APP_UART_FLOW_CONTROL_DISABLED, or I will get nothing on my console.

Thx

RE: Using app_with_dfu jump to bootloader.

Coldson — Fri, 01 Jul 2016 08:53:19 GMT

And for disable peripheral. Now I work around disable RTC while starting DFU, and it work. So the last problem is the UART.

RE: Using app_with_dfu jump to bootloader.

Coldson — Fri, 01 Jul 2016 08:48:47 GMT

Hi Hung Bui, u can reproduce this simply using the 11.0.0 sdk hrs_with_dfu and add define "ENABLE_DEBUG_LOG_SUPPORT" and try to upgrade the nrf52-DK, this is the problem I found.

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Fri, 01 Jul 2016 07:52:05 GMT

Hi Coldson,

It could be the issue that one of the applications (your application or the bootloader) was using the peripheral (UART or RTC) without configure it properly. When switching from application to bootloader and vise versa, we don't reset all the registers to default state. So it's up to the application to make sure all the register are configured before using it. Also I would suggest to disable any timer, peripheral before branching to bootloader.

Again, if you can provide an demo code that can reproduce the issue it's would be easier for us to track down the problem and find a solution.

RE: Using app_with_dfu jump to bootloader.

Coldson — Thu, 30 Jun 2016 16:21:30 GMT

@Hung Bui, I integrate dfu example to my application again, and it must not enable UART and not init RTC then DFU work. But why? Are there something related with DFU flow?

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Tue, 28 Jun 2016 15:11:46 GMT

@Coldson: I mean you can try to integrate the DFU service in any example in the SDK (not the ready made hrm with dfu) , and test jumping to bootloader, if it doesn't work you can send us that example.

RE: Using app_with_dfu jump to bootloader.

Coldson — Tue, 28 Jun 2016 14:49:18 GMT

It work on hrm with DFU example but fail on my own application, I will keep check my application. Thx.

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Tue, 28 Jun 2016 14:06:25 GMT

@Coldson: I would suggest to test with an example (any example) in the SDK. If it doesn't work, please send the code here and we can check what could be wrong.

RE: Using app_with_dfu jump to bootloader.

Coldson — Tue, 28 Jun 2016 12:55:45 GMT

Hi, Hung Bui, if I set break point at line 111, it will never enter that line(111).

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Tue, 28 Jun 2016 07:52:50 GMT

Hi Coldson, I assume that you have flashed the bootloader (the \examples\dfu\bootloader project) in addition to your application.

The dfu_ble_svc_peer_data_set(&m_peer_data); is an SVC call, and the code is in the bootloader not in the application, you won't be able to step through it, please add a break point after that call.

Please let me know what happen if you add breakpoint at line 111 instead of line 110.

RE: Using app_with_dfu jump to bootloader.

Coldson — Mon, 27 Jun 2016 16:33:13 GMT

@Hung Bui, I add break point there because the whole flow will not end at the function :"bootloader_util_app_start(NRF_UICR->NRFFW[0]);", and jump to bootloader, so I add breakpoint to find which code are going wrong, I will keep find the solution. I will share the root cause in this topic if there're any progress.

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Mon, 27 Jun 2016 15:36:25 GMT

@Coldson: That what I suggested, don't add breakpoint on that function and the step over it. Add the breakpoint after the function instead. Or do you get hardfault when calling that ?

RE: Using app_with_dfu jump to bootloader.

Coldson — Mon, 27 Jun 2016 15:13:30 GMT

Hi, Hung Bui, there're no return after that break point. And, Yes, I use the device manager in my application, I will check what I feed to that function as m_peer_data.

RE: Using app_with_dfu jump to bootloader.

Hung Bui — Mon, 27 Jun 2016 13:36:24 GMT

Hi Coldson,

Which error code you got at step #4 ? you may want to add the break point after the call (one line after )

Did you use device manager in your application ? What did you feed to the function as m_peer_data ?