buttonless DFU - bootloader hangs

RE: buttonless DFU - bootloader hangs

Hung Bui — Tue, 21 Jul 2015 10:35:43 GMT

@Wim: So you can add break point and debug now ? Thanks for the suggestion, I will update the blog. I think we can close the case. Could you mark the answer as correct ?

RE: buttonless DFU - bootloader hangs

wim — Tue, 21 Jul 2015 08:47:28 GMT

that's what I do, did not have the -O option, do not do any optimisations, for easy debugging; I think that if I would add -O, I would probably not need the other, that is what I read the link you sent ( tls.mbed.org/.../arm-thumb-error-r7-cannot-be-used-in-asm-here), so maybe it would be a good idea to add that in your side note as well ?

RE: buttonless DFU - bootloader hangs

Hung Bui — Tue, 21 Jul 2015 08:30:09 GMT

I'm glad that it worked now:) If you only add -fomit-frame-pointer but not -O, would it be linked ?

RE: buttonless DFU - bootloader hangs

wim — Mon, 20 Jul 2015 16:37:15 GMT

That seemed to do the trick! I was able to flash in another application when switching from my own application to dfu. Now I will have to find a way to use this option only when compiling for release, because apparently, when adding that linker option, it makes debugging virtually impossible. Of course, debugging and stepping isn't really possible with the soft device and its real time behavior, so I am bound to using the RTT printfs anyway. But hurray, it works!

RE: buttonless DFU - bootloader hangs

Hung Bui — Mon, 20 Jul 2015 16:07:23 GMT

@wim: sorry, I missed that :( You know, it's Monday :P

Could you try this https://tls.mbed.org/kb/development/arm-thumb-error-r7-cannot-be-used-in-asm-here ?

It's also mentioned here: https://devzone.nordicsemi.com/question/42043/bootloader_utilc-does-not-compile-with-gcc-without-os-option/

RE: buttonless DFU - bootloader hangs

wim — Mon, 20 Jul 2015 16:01:18 GMT

That is the toolchain I use - see log.

RE: buttonless DFU - bootloader hangs

Hung Bui — Mon, 20 Jul 2015 15:57:07 GMT

@wim: I would suggest you to try with GNU Tools ARM Embedded 4.9 2015q1. If it works then we know that it's the toolchain issue. Note that you don't really have to use the same toolchain for application and bootloader.

RE: buttonless DFU - bootloader hangs

wim — Mon, 20 Jul 2015 15:46:22 GMT

Hi, i just recompiled with your boot loader_util_gcc.c. Previous compilation was with just the sdk9 file copied into my project. Here is the complete log (please note that I do not use make, but a qbs project in QtCreator with bare metal plugin (see my blog on this topic). So I could send you my project in this format if you'd find this useful):

compiling SEGGER_RTT.c
compiling app_timer.c
compiling softdevice_handler.c
compiling ble_my_service.c
compiling app.c
compiling nrf_assert.c
compiling dfu_app_handler.c
compiling device_manager_peripheral.c
compiling app_ble.c
compiling main.c
compiling ble_advdata.c
compiling ble_dfu.c
compiling bootloader_util_gcc.c
compiling ble_srv_common.c
compiling hw_init.c
compiling pstorage.c
compiling system_nrf51.c
compiling app_gpiote.c
compiling ble_conn_params.c
compiling app_button.c
compiling nrf_delay.c
compiling SEGGER_RTT_printf.c
compiling spi_master.c
compiling gcc_startup_nrf51.s
compiling interval_record.c
compiling app_scheduler.c
compiling app_error.c
linking myproject.elf
/Applications/gcc-arm-none-eabi-4_9-2015q1/bin/arm-none-eabi-gcc /myproject/build/.obj/0da82cc593b2789a/softdevice_handler.c.o /myproject/build/.obj/10b4b06de9fa4197/app_timer.c.o /myproject/build/.obj/2119f2cbff76308c/ble_my_service.c.o /myproject/build/.obj/2a94b74f64a12597/hw_init.c.o /myproject/build/.obj/3f21d9844c5aeb60/spi_master.c.o /myproject/build/.obj/52e6ddd7334b868d/app.c.o /myproject/build/.obj/52e6ddd7334b868d/app_ble.c.o /myproject/build/.obj/53d20156d67b39d5/interval_record.c.o /myproject/build/.obj/65c02569a3cb2e93/pstorage.c.o /myproject/build/.obj/7e838a4599280656/gcc_startup_nrf51.s.o /myproject/build/.obj/83b245554f8d0dc8/app_gpiote.c.o /myproject/build/.obj/84f1db46c2de65db/SEGGER_RTT.c.o /myproject/build/.obj/84f1db46c2de65db/SEGGER_RTT_printf.c.o /myproject/build/.obj/9a6794832d15ad24/nrf_delay.c.o /myproject/build/.obj/9b63ec1070f3e8e0/app_error.c.o /myproject/build/.obj/9b63ec1070f3e8e0/nrf_assert.c.o /myproject/build/.obj/a232a46d3525d371/device_manager_peripheral.c.o /myproject/build/.obj/a64b663017d02128/app_scheduler.c.o /myproject/build/.obj/bfa30bb80df21156/system_nrf51.c.o /myproject/build/.obj/d814ef6a0ceb640b/ble_advdata.c.o /myproject/build/.obj/d814ef6a0ceb640b/ble_conn_params.c.o /myproject/build/.obj/d814ef6a0ceb640b/ble_srv_common.c.o /myproject/build/.obj/d9b86cfc338eea47/ble_dfu.c.o /myproject/build/.obj/d9b86cfc338eea47/bootloader_util_gcc.c.o /myproject/build/.obj/d9b86cfc338eea47/dfu_app_handler.c.o /myproject/build/.obj/e0aeaa9c3d490126/app_button.c.o /myproject/build/.obj/f27fede2220bcd32/main.c.o -g -Wl,-undefined,error -Xlinker -Map=/myproject/build/myproject.map -mthumb -mabi=aapcs -L/Users/wim/Projects/nordic/myproject/../nRF51_SDK_7.2.0/components/toolchain/gcc/ -T/Users/wim/Projects/nordic/myproject/../nRF51_SDK_7.2.0/components/toolchain/gcc/gcc_nrf51_s110_xxaa.ld -mcpu=cortex-m0 -Wl,--gc-sections --specs=nano.specs -lc -lnosys -o /myproject/build/myproject.elf
/Users/wim/Projects/nordic/myproject_no_rf_sdk720/src/dfu/bootloader_util_gcc.c: In function 'StartApplication':
/Users/wim/Projects/nordic/myproject_no_rf_sdk720/src/dfu/bootloader_util_gcc.c:64:1: error: r7 cannot be used in asm here
 }
 ^
lto-wrapper: /Applications/gcc-arm-none-eabi-4_9-2015q1/bin/arm-none-eabi-gcc returned 1 exit status
/Applications/gcc-arm-none-eabi-4_9-2015q1/bin/../lib/gcc/arm-none-eabi/4.9.3/../../../../arm-none-eabi/bin/ld: lto-wrapper failed
collect2: error: ld returned 1 exit status
Process failed with exit code 1.
The following products could not be built for configuration qtc_nrf51822_9693700c-debug:
	myproject
Error while building/deploying project myproject_norf_sdk720 (kit: nrf51822)
When executing step "Qbs Build"
17:33:13: Elapsed time: 00:02.

RE: buttonless DFU - bootloader hangs

Hung Bui — Mon, 20 Jul 2015 15:40:31 GMT

Could you post the error log ? Also I notice that you use bootloader_util.c instead of bootloader_util_gcc.c, do you have other modification ?

RE: buttonless DFU - bootloader hangs

wim — Mon, 20 Jul 2015 15:38:06 GMT

I tried also with your file. The file compiles ok, but the linker complains. I have noticed that in SDK9 each gcc makefile project is accompanied by its own .ld file, while in sdk7.2.0 the general toolchain .ld file is used. Should I do the same? are there any differences? Also, I use version 4.9 of gcc, is that ok?

RE: buttonless DFU - bootloader hangs

Hung Bui — Mon, 20 Jul 2015 15:14:03 GMT

@wim: I'm testing here, the compiler didn't complain. I attached the file in my answer. Could you try my file ?

If you still have the issue, please send us your project (including the whole project, where all other example is removed).

RE: buttonless DFU - bootloader hangs

wim — Mon, 20 Jul 2015 14:48:23 GMT

I did exactly that but got the error message during the linking stage as described in my previous comment:

/.../src/dfu/bootloader_util.c: In function 'bootloader_util_reset':
/.../src/dfu/bootloader_util.c:106:1: error: r7 cannot be used in asm here
 }
 ^
lto-wrapper: /Applications/gcc-arm-none-eabi-4_9-2015q1/bin/arm-none-eabi-gcc returned 1 exit status
/Applications/gcc-arm-none-eabi-4_9-2015q1/bin/../lib/gcc/arm-none-eabi/4.9.3/../../../../arm-none-eabi/bin/ld: lto-wrapper failed
collect2: error: ld returned 1 exit status

So it complains about the last line, about r7. How can I solve this? Is it a linker script issue with the SDK7.2.0 linker script?

RE: buttonless DFU - bootloader hangs

Hung Bui — Mon, 20 Jul 2015 13:34:14 GMT

@wim: I'm sorry, you were correct, in the bootloader_util_gcc.c we called the function "StartApplication()"

But the principle is the same, you replace the code you found in side bootloader_util_reset() in bootloader_util.c in SDK 9.0 to the code in bootloader_util_gcc.c in StartApplication() in SDK v7.2.

Your function should look like this:

static inline void StartApplication(uint32_t start_addr)
{
 __asm volatile(
        "ldr   r0, [%0]\t\n"            // Get App initial MSP for bootloader.
        "msr   msp, r0\t\n"             // Set the main stack pointer to the applications MSP.
        "ldr   r0, [%0, #0x04]\t\n"     // Load Reset handler into R0.

        "movs  r4, #0xFF\t\n"           // Move ones to R4.
        "sxtb  r4, r4\t\n"              // Sign extend R4 to obtain 0xFFFFFFFF instead of 0xFF.

        "mrs   r5, IPSR\t\n"            // Load IPSR to R5 to check for handler or thread mode.
        "cmp   r5, #0x00\t\n"           // Compare, if 0 then we are in thread mode and can continue to reset handler of bootloader.
        "bne   isr_abort\t\n"           // If not zero we need to exit current ISR and jump to reset handler of bootloader.

        "mov   lr, r4\t\n"              // Clear the link register and set to ones to ensure no return.
        "bx    r0\t\n"                  // Branch to reset handler of bootloader.

        "isr_abort:  \t\n"

        "mov   r5, r4\t\n"              // Fill with ones before jumping to reset handling. Will be popped as LR when exiting ISR. Ensures no return to application.
        "mov   r6, r0\t\n"              // Move address of reset handler to R6. Will be popped as PC when exiting ISR. Ensures the reset handler will be executed when exist ISR.
        "movs  r7, #0x21\t\n"           // Move MSB reset value of xPSR to R7. Will be popped as xPSR when exiting ISR. xPSR is 0x21000000 thus MSB is 0x21.
        "rev   r7, r7\t\n"              // Reverse byte order to put 0x21 as MSB.
        "push  {r4-r7}\t\n"             // Push everything to new stack to allow interrupt handler to fetch it on exiting the ISR.

        "movs  r4, #0x00\t\n"           // Fill with zeros before jumping to reset handling. We be popped as R0 when exiting ISR (Cleaning up of the registers).
        "movs  r5, #0x00\t\n"           // Fill with zeros before jumping to reset handling. We be popped as R1 when exiting ISR (Cleaning up of the registers).
        "movs  r6, #0x00\t\n"           // Fill with zeros before jumping to reset handling. We be popped as R2 when exiting ISR (Cleaning up of the registers).
        "movs  r7, #0x00\t\n"           // Fill with zeros before jumping to reset handling. We be popped as R3 when exiting ISR (Cleaning up of the registers).
        "push  {r4-r7}\t\n"             // Push zeros (R4-R7) to stack to prepare for exiting the interrupt routine.

        "movs  r0, #0xF9\t\n"           // Move the execution return command into register, 0xFFFFFFF9.
        "sxtb  r0, r0\t\n"              // Sign extend R0 to obtain 0xFFFFFFF9 instead of 0xF9.
        "bx    r0\t\n"                  // No return - Handler mode will be exited. Stack will be popped and execution will continue in reset handler initializing other application.
        ".align\t\n"
        :: "r" (start_addr)             // Argument list for the gcc assembly. start_addr is %0.
        :  "r0", "r4", "r5", "r6", "r7" // List of register maintained manually.
    );
}

bootloader_util_gcc.c

RE: buttonless DFU - bootloader hangs

wim — Mon, 20 Jul 2015 13:14:17 GMT

In the side note a reference is made to bootloader_util_reset() in bootloader_util.c. In SDK7.2 I only have a bootloader_util_gcc.c, and there is no bootloader_util_reset function in there, only a 'StartApplication'. Is it that function I need to replace withbootloader_util_reset of 9.0.0? Do i need some change to the linker script too? Because after replacing the code with the code of sdk9, I get following linker error: /.../myproject/src/dfu/bootloader_util_gcc.c: In function 'bootloader_util_reset': /.../myproject/src/dfu/bootloader_util_gcc.c:73:1: error: r7 cannot be used in asm here } ^ lto-wrapper: /Applications/gcc-arm-none-eabi-4_9-2015q1/bin/arm-none-eabi-g++ returned 1 exit status /Applications/gcc-arm-none-eabi-4_9-2015q1/bin/../lib/gcc/arm-none-eabi/4.9.3/../../../../arm-none-eabi/bin/ld: lto-wrapper failed collect2: error: ld returned 1 exit status

RE: buttonless DFU - bootloader hangs

Hung Bui — Mon, 20 Jul 2015 12:09:06 GMT

@wim: I updated a side note in Question C in this FAQ which may solve your issue.

RE: buttonless DFU - bootloader hangs

wim — Fri, 17 Jul 2015 15:13:11 GMT

ble_app_hrs with dfu does work when I compile it with Keil. But when I compile it with gcc then it doesn't work.

RE: buttonless DFU - bootloader hangs

Hung Bui — Fri, 17 Jul 2015 14:36:04 GMT

@wim: Could you check if you use our experimental ble_app_hrs with dfu it would work for you ? That you can set the device in bootloader mode by writing to the DFU characteristic and can overwrite the hrs firmware with something else ?

You should also check if your initdata is prepared correctly. But I assume that it's correct since you mentioned that you managed to update your application using button+reset.

Have you made sure you enabled IS_SRVC_CHANGED_CHARACT_PRESENT = 1 in your application main.c ?