MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Curiosus — Tue, 14 Feb 2023 21:00:49 GMT

Thanks hung , this has been quite helpful !

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Hung Bui — Tue, 14 Feb 2023 12:25:43 GMT

Hi Felipe,
I couldn't find a clear definition on what exactly is done at step 2, from my understanding it's how you define a customized library/driver in Zephyr. I believe that it should be in myboard because that's how your CMakelist.txt will be included when MCUBoot is built (not the CMakelist in your project).

You can find the same method is used in this article in Zephyr: https://www.zephyrproject.org/how-to-build-drivers-for-zephyr-rtos/

And you can find how this macro is defined inside \zephyr\cmake\modules\extensions.cmake

macro(zephyr_library)
  zephyr_library_get_current_dir_lib_name(${ZEPHYR_BASE} lib_name)
  zephyr_library_named(${lib_name})
endmacro()

function(zephyr_library_sources source)
  target_sources(${ZEPHYR_CURRENT_LIBRARY} PRIVATE ${source} ${ARGN})
endfunction()

Regarding the warning error in VS Code, I don't have a solution for now. Most likely VSCode is not aware of the child image include folder. But I don't think it would be a big issue.

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Curiosus — Sun, 12 Feb 2023 07:58:34 GMT

Hi Hung,

I have migrated my application from ncs 2.0.0 to 2.2.0 in order to access the Mcuboot hooks functionality:

Steps followed:

1. In child_image/mcuboot.conf added:

CONFIG_MCUBOOT_ACTION_HOOKS=y

2. Added a CMakeLists.txt file in boards/arm/my_board containing the following:

if(CONFIG_MCUBOOT)

    zephyr_library()    
    zephyr_library_sources(mcuboot_hook.c)


endif()

3. in boards\arm\my_board added a mcuboot_hook.c file containing:

#include "bootutil/image.h"
#include "bootutil/fault_injection_hardening.h"
#include "bootutil/boot_hooks.h"
#include "bootutil/mcuboot_status.h"


void mcuboot_status_change(mcuboot_status_type_t status){

	LOG_INF("mcuboot status: %d ", status);

}

NB:I had to include some extra files here in order to be able to compile it as there was some types missing.

Thank you very much for your help.

Is there a reason for these files to be under my_board directory ?

Could you please elaborate on what was done on step 2 above ? , I know it works but I would like to understand it better.

Lastly, How could I include the mcuboot directory to my path so I can get rid off of these errors:

Cheers,

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Hung Bui — Thu, 09 Feb 2023 12:16:04 GMT

Hi again Felipe,
I got the suggestion from our colleague on how to do it (still you need to update to v2.1 or newer)

In CMakeLists.txt in boards/arm/your_board , you can add something like this:
if(CONFIG_MCUBOOT)
zephyr_library()
zephyr_library_sources(my_mcuboot_hook.c)
endif()

Then place a my_mcuboot_hook.c, that has the mcuboot_status_change() function, in boards/arm/your_board

By doing this you don't have to change mcuboot.

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Hung Bui — Thu, 09 Feb 2023 09:19:34 GMT

Hi Felipe,
As far as I know mcuboot_status_change() and CONFIG_MCUBOOT_ACTION_HOOKS was only added from NCS v2.1.0 .

You may want to consider upgrading to newer SDK.

I understand that you don't want to modify the bootloader. I'm checking internally if we have an example that doesn't need to do that. My understanding is that it should be possible to add the function in a file and then compile the mcuboot as childimage with the extra file.

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Curiosus — Wed, 08 Feb 2023 22:10:19 GMT

Hi Hung,

I had a look at ncs/v2.0.0/bootloader/mcuboot/boot/zephyr/main.c and I find no references to mcuboot_status_change() also line numbers do not match as shown below, we must be looking ad different versions of ncs.

Also, Is there any way this might be achieved without having to modify the bootloader ? otherwise the whole idea of mcuboot hooks is lost.

What is Nordic's recommended way of achieving this indications from the bootloader ?

Cheers,

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Hung Bui — Wed, 08 Feb 2023 10:13:50 GMT

Hi Felipe,
I haven't implemented the hooks myself but what I can see from the code is that you would need to add your own function named mcuboot_status_change() into mcuboot.
If you look into \bootloader\mcuboot\boot\zephyr\main.c you can find that it will call mcuboot_status_change() on each stage of the mcuboot process:

As far as I know this function should be implemented in mcuboot, not in your application because your application will not run when the Mcuboot is doing its job.

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Curiosus — Tue, 07 Feb 2023 18:46:48 GMT

Hi Hung,

Thanks for your prompt response.

Awesome, So we should then wait for the mobile team to get back to us in regards to question 2 and 3.

In regards to the last question (Apologies for the typo) , since the answer in question 1 is that it may take reasonably long to perform these actions, I would want to be able to somehow ( Since at that instance the device will appear unresponsive to the user ) notify the user that the operation is happening i.e ( LCD Display or a GPIO toggle etc. ) is this possible ?

Cheers,

Thanks

RE: MCUBoot DFU OTA Best way to obtain actions ( Hooks ) also Android vs iOS discrepancies.

Hung Bui — Tue, 07 Feb 2023 12:33:22 GMT

Hi Felipe,

I will need help from our Mobile app team to answer most of your questions and get back to you.

I can try to answer question 1 though.

At step 5 I would assume it's when the MCUBoot start the swapping process. The process involve erasing page by page of both Primary Slot and Secondary Slot and copy the image page by page from one slot to another slot.
This actually can take a lot of time considering erasing one page would require up to 90ms. And this is not include the calculation time needed to verify the hash of the image. The larger the image the longer it takes.

In the last question I assume you mean "your own code" not "won code" ?