NRF51822 DFU Problem

RE: NRF51822 DFU Problem

Mohamed O.Abouzeid — Fri, 04 Dec 2015 11:52:21 GMT

@Simon I'm facing the same problem now, I tryn to solve & I need your help, how to set that member in the dfu_app_handler.c ? ?

RE: NRF51822 DFU Problem

Simon Ampleman — Fri, 17 Apr 2015 17:56:33 GMT

I found the problem.

The device manager is used in dfu_app_handler.c and it requires the member m_dm_handle_valid to be set in order to call dfu_app_set_peer_data.

I had a problem in the initialization of the device manager, so the dfu_app_set_dm_handle never gets called so the peer data is not set.

Thank you again.

Simon

RE: NRF51822 DFU Problem

Simon Ampleman — Fri, 17 Apr 2015 15:59:37 GMT

Thank you for your detailed explanation.

I updated the linker file and verified the dfu_ble_svc.c file according to your answer. I also ajusted the memory layout for 16 kB RAM version.

However, I still have the same result.

Here are the new files :

Bootloader Linker file :

/* Linker script to configure memory regions. */

SEARCH_DIR(.)
GROUP(-lgcc -lc -lnosys)

MEMORY
{
  FLASH (rx) : ORIGIN = 0x00035000, LENGTH = 0xB000
  RAM (rwx) :  ORIGIN = 0x20002000, LENGTH = 0x1F80
  NOINIT (rwx) :  ORIGIN = 0x20003F80, LENGTH = 0x80
  BOOTLOADER_SETTINGS (rwx) : ORIGIN = 0x0003FC00, LENGTH = 0x400
  UICR_BOOTLOADER (r) : ORIGIN = 0x10001014, LENGTH = 0x04
}
SECTIONS
{
  .bootloaderSettings :
  {

  } > BOOTLOADER_SETTINGS

  /* Ensures the Bootloader start address in flash is written to UICR when flashing the image. */
  .uicrBootStartAddress : 
  {
    KEEP(*(.uicrBootStartAddress))
  } > UICR_BOOTLOADER

  /* No init RAM section in bootloader. Used for bond information exchange. */
  .NoInit :
  {

  } > NOINIT
}	

INCLUDE "gcc_nrf51_common.ld"

Application linker file

/* Linker script to configure memory regions. */

SEARCH_DIR(.)
GROUP(-lgcc -lc -lnosys)

MEMORY
{
  FLASH (rx) : ORIGIN = 0x00016000, LENGTH = 0x2A000 
  RAM (rwx) :  ORIGIN = 0x20002000, LENGTH = 0x2000
}

INCLUDE "gcc_nrf51_common.ld"

dfu_ble_svc.c

#if defined   ( __GNUC__ )
static dfu_ble_peer_data_t m_peer_data __attribute__((section(".NoInit")))__attribute__((used));     /**< This variable should be placed in a non initialized RAM section in order to be valid upon soft reset from application into bootloader. */
static uint16_t            m_peer_data_crc __attribute__((section(".NoInit")))__attribute__((used)); /**< CRC variable to ensure the integrity of the peer data provided. */

#else
static dfu_ble_peer_data_t m_peer_data __attribute__((section("NoInit"), zero_init));     /**< This variable should be placed in a non initialized RAM section in order to be valid upon soft reset from application into bootloader. */
static uint16_t            m_peer_data_crc __attribute__((section("NoInit"), zero_init)); /**< CRC variable to ensure the integrity of the peer data provided. */
#endif

Bootloader map file

.bootloaderSettings
                0x0003fc00      0x400
 .bootloaderSettings
                0x0003fc00      0x400 _build/bootloader_util_gcc.o
                0x0003fc00                m_boot_settings

.uicrBootStartAddress
                0x10001014        0x4
 *(.uicrBootStartAddress)
 .uicrBootStartAddress
                0x10001014        0x4 _build/bootloader_util_gcc.o
                0x10001014                m_uicr_bootloader_start_address

.NoInit         0x20003f80       0x30
 .NoInit        0x20003f80       0x30 _build/dfu_ble_svc.o

Last part of Application Map file in the ending region of the ram allocation

                0x20002700                m_ble_evt_handler.lto_priv.86
                0x20002704                m_ble_evt_buffer_size.lto_priv.88
                0x20002708                m_evt_buffer.lto_priv.87
 *(COMMON)
                0x2000270c                . = ALIGN (0x4)
                0x2000270c                __bss_end__ = .

.heap           0x20002710      0x800
                0x20002710                __end__ = .
                0x20002710                end = __end__
 *(.heap*)
 .heap          0x20002710      0x800 _build/gcc_startup_nrf51.o
                0x20002710                __HeapBase
                0x20002f10                __HeapLimit = .

.stack_dummy    0x20002710      0x800
 *(.stack*)
 .stack         0x20002710      0x800 _build/gcc_startup_nrf51.o
                0x20004000                __StackTop = (ORIGIN (RAM) + 0x2000)
                0x20003800                __StackLimit = (__StackTop - SIZEOF (.stack_dummy))
                0x20004000                PROVIDE (__stack, __StackTop)
                0x00000001                ASSERT ((__StackLimit >= __HeapLimit), region RAM overflowed with stack)

Anybody see something wrong in these files ?

Thank you,

Simon

RE: NRF51822 DFU Problem

Torsten Rasmussen — Fri, 17 Apr 2015 09:38:04 GMT

Hi,

Based on your reporting I assume you are trying out the buttonless update procedure, where you jump from application into the bootloader. For this to work, the bootloader needs to know the LTK, IRK and Address of the device which triggered the DFU mode. Those are distributed from the app to bootloader through an SVC as described in: developer.nordicsemi.com/.../a00079.html

This means that the SVC must be implemented in the bootloader, which I assume that you already do. However, the data will be located in RAM and it is therefore critical that this data is not zero initialized upon reset into bootloader.

Look in the file: dfu_ble_svc.c and find the definition of m_peer_data and m_peer_data_crc. Ensure those are placed in no init section, for gcc:

__attribute__((section(".noinit"))) static dfu_ble_peer_data_t m_peer_data; 
__attribute__((section(".noinit"))) static uint16_t            m_peer_data_crc;

And then ensure to define this section in the linker script:

As example:

MEMORY
{
  /** Flash start address for the bootloader. This setting will also be stored in UICR to allow the
   *  MBR to init the bootloader when starting the system. This value must correspond to 
   *  BOOTLOADER_REGION_START found in dfu_types.h. The system is prevented from starting up if 
   *  those values do not match. The check is performed in main.c, see
   *  APP_ERROR_CHECK_BOOL(*((uint32_t *)NRF_UICR_BOOT_START_ADDRESS) == BOOTLOADER_REGION_START);
   */
  FLASH (rx) : ORIGIN = 0x3C000, LENGTH = 0x3C00

  /** RAM Region for bootloader. This setting is suitable when used with s110, s120, s130, s310. */
  RAM (rwx) :  ORIGIN = 0x20002c00, LENGTH = 0x5380

  /** Location of non initialized RAM. Non initialized RAM is used for exchanging bond information
   *  from application to bootloader when using buttonluss DFU OTA. 
   */
  NOINIT (rwx) :  ORIGIN = 0x20007f80, LENGTH = 0x80

  /** Location of bootloader setting in at the last flash page. */
  BOOTLOADER_SETTINGS (rw) : ORIGIN = 0x0003FC00, LENGTH = 0x0400

  /** Location in UICR where bootloader start address is stored. */
  UICR_BOOTLOADER (r) : ORIGIN = 0x10001014, LENGTH = 0x04
}

SECTIONS
{
  /* Ensures the bootloader settings are placed at the last flash page. */
  .bootloaderSettings :
  {
	
  } > BOOTLOADER_SETTINGS

  /* Ensures the Bootloader start address in flash is written to UICR when flashing the image. */
  .uicrBootStartAddress : 
  {
    KEEP(*(.uicrBootStartAddress))
  } > UICR_BOOTLOADER

  /* No init RAM section in bootloader. Used for bond information exchange. */
  .noinit :
  {

  } > NOINIT
  /* other placements follow here... */
}

Note that the memory layout here is for 32 kB RAM version. Adjust accordingly for the 16kB version.

Try also to see if your device is advertising with another address (normal address + 1), as this will also indicate that the peer data has not been succesfully exchanged.