sdk 12 bootloader erased after programming

RE: sdk 12 bootloader erased after programming

Aleksander Nowakowski — Fri, 02 Dec 2016 10:42:33 GMT

Some other bugs I found in the experimental buttonless module:

The characteristic should have the Write property. Add:

char_md.char_props.write = 1;

to ble_dfu.c file after line 90.

Change:

if (rsp_code == BLE_DFU_ENTER_BOOTLOADER && p_evt_write->data[0] == BLE_DFU_ENTER_BOOTLOADER)

if (rsp_code == DFU_RSP_SUCCESS && p_evt_write->data[0] == BLE_DFU_ENTER_BOOTLOADER)

in the same file around line 249. BLE_DFU_ENTER_BOOTLOADER and DFU_RSP_SUCCESS accidentally have the same value = 0x01.

Optional:

Try to ignore methods documentation :) It has texts like this: "Handle write events to the Location and Navigation Service Control Point characteristic." or "Function for adding RX characteristic."
Rename rx_char_add method. The name was copied from UART example, which has RX characteristic. I would say cp_char_add (control_point) would be better.

RE: sdk 12 bootloader erased after programming

bjorn-spockeli — Wed, 02 Nov 2016 11:37:14 GMT

Happy to help :) I would appreciate if you could mark the answer as correct/accepted by clicking the checkmark in the circle to the left of the answer.

RE: sdk 12 bootloader erased after programming

vea45 — Wed, 02 Nov 2016 11:33:23 GMT

Perfect! This was the bug! Now the code don't overwrites the bootloader! Thank you!

RE: sdk 12 bootloader erased after programming

vea45 — Thu, 27 Oct 2016 13:11:21 GMT

First I flash the SD and then the bootloader, on the nrfGo the bootloader appears with 32kB of size and address: 0x78000, but with the parameters that you said before the aplication overwrites the bootloader anyway...

RE: sdk 12 bootloader erased after programming

vea45 — Thu, 27 Oct 2016 13:08:59 GMT

My aplication parameteters are: IROM1: 0x1F000, 0x59000 IRAM1:0x20002148, 0xDED8.

My Bootloader parameters for the debug one: IROM1:0x73000, 0xB000 IRAM1:0x20002C00, 0xD380.

RE: sdk 12 bootloader erased after programming

vea45 — Thu, 27 Oct 2016 13:01:19 GMT

Yes, I'm trying also with the buttonless example...

RE: sdk 12 bootloader erased after programming

bjorn-spockeli — Thu, 27 Oct 2016 11:34:14 GMT

Are you using the experimental buttonless example application?

RE: sdk 12 bootloader erased after programming

bjorn-spockeli — Thu, 27 Oct 2016 06:55:26 GMT

The IROM and IRAM settings for an application using the S132 v3.0.0 with no bootloader are:

IROM1 Start: 0x1F000 Size: 0x61000
IRAM1 Start: 0x20002128 Size: 0xDED8

The Secure Bootloader start address is 0x78000 and 0x1F000 + 0x61000 = 0x80000. Thus, you have to alter the application IROM settings so that the application does not overwrite the bootloader, i.e.

IROM1 Start: 0x1F000 Size: 0x59000
IRAM1 Start: 0x20002128 Size: 0xDED8

If you're using GCC you have to alter the application linker script in the armgcc folder, i.e.

MEMORY
{
  FLASH (rx) : ORIGIN = 0x1f000, LENGTH = 0x59000
  RAM (rwx) :  ORIGIN = 0x20002128, LENGTH = 0xded8
}

EDIT 28.10.16:

Its another bug in the buttonless DFU example. It reserves a codepage at 0x7E000 for MBR parameters, which is not necessary and should not been added. The nRF52 flash algorithm in Keil will see this and perform a sector erase from 0x7F000 to 0x1F000 before flashing the application.

Thus, the entire section in nrf_dfu_settings.c may should be commented out. However, be sure that you do not compile the secure bootloader with this section commented out, as it also uses this file!

#if defined ( NRF52 )

/**@brief   This variable reserves a codepage for mbr parameters, to ensure the compiler doesn't
 *          locate any code or variables at his location.
 */
    
    
#if defined ( __CC_ARM )

    uint8_t m_mbr_params_page[CODE_PAGE_SIZE]       __attribute__((at(NRF_MBR_PARAMS_PAGE_ADDRESS))) __attribute__((used));

#elif defined ( __GNUC__ )

    uint8_t m_mbr_params_page[CODE_PAGE_SIZE]       __attribute__ ((section(".mbrParamsPage")));

#elif defined ( __ICCARM__ )

    __no_init uint8_t m_mbr_params_page[CODE_PAGE_SIZE]     @ NRF_MBR_PARAMS_PAGE_ADDRESS;

#else

    #error Not a valid compiler/linker for m_mbr_params_page placement.

#endif


/**@brief   This variable makes the linker script write the mbr parameters page address to the
 *          UICR register. This value will be written in the HEX file and thus written to the
 *          UICR when the bootloader is flashed into the chip.
 */
 
#if defined ( __CC_ARM )
    uint32_t m_uicr_mbr_params_page_address __attribute__((at(NRF_UICR_MBR_PARAMS_PAGE_ADDRESS)))
                                                    = NRF_MBR_PARAMS_PAGE_ADDRESS;

#elif defined ( __GNUC__ )
    volatile uint32_t m_uicr_mbr_params_page_address    __attribute__ ((section(".uicrMbrParamsPageAddress")))
                                                    = NRF_MBR_PARAMS_PAGE_ADDRESS;
#elif defined ( __ICCARM__ )

    __root    const uint32_t m_uicr_mbr_params_page_address @ NRF_UICR_MBR_PARAMS_PAGE_ADDRESS
                                                    = NRF_MBR_PARAMS_PAGE_ADDRESS;

#else

    #error Not a valid compiler/linker for m_mbr_params_page placement.

#endif

#endif // defined ( NRF52 )

RE: sdk 12 bootloader erased after programming

bjorn-spockeli — Thu, 27 Oct 2016 06:45:07 GMT

Could you post the IROM and IROM settings (Keil) or the linker script (GCC) for the application?