Inspection of FW image failed issue

[Mike-Yang-yoyo]

Hi,

I would like to use the smpmgr tool on my Windows PC to perform an OTA update on my nRF5340 device, but I encounter an “Inspection of FW image failed” issue when loading the image file.

`C:\Users\user\Desktop\Mike\smpmgr-0.17.0-windows-amd64>smpmgr.exe --ble E8:B8:C3:0E:CC:23 image upload application.signed.bin
[11:08:31] ERROR Inspection of FW image failed - image_management:187 image_management.py:187
╭───────────────────────────────────── Traceback (most recent call last) ──────────────────────────────────────╮
│ in upload:184 │
│ │
│ in load_file:331 │
│ │
│ in load_from:255 │
│ │
│ in loads:250 │
│ │
│ in init:121 │
│ │
│ in post_init:243 │
╰──────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
MCUBootImageError: TLV info magic is 0x6908, expected 0x6907

C:\Users\user\Desktop\Mike\smpmgr-0.17.0-windows-amd64>`

Could you provide some suggestions?

Thanks!!
Mike

[JPHutchins]

It looks like your image is using the protected image info: https://docs.mcuboot.com/design.html#image-format

We would need to add support for it in the smpclient lib and then update smpmgr:

https://github.com/intercreate/smpclient/blob/main/smpclient/mcuboot.py#L29-L30

https://github.com/intercreate/smpclient/blob/main/smpclient/mcuboot.py#L240-L245

[Mike-Yang-yoyo]

Hi @JPHutchins ,

OK, so protected images are not supported in the current version because of limitations in smpclient.
If I want to support this type of image, would a simple change (For example, changing it to IMAGE_TLV_PROT_INFO_MAGIC) be enough to make it work?

Thanks!!

[JPHutchins]

Hi @JPHutchins ,

OK, so protected images are not supported in the current version because of limitations in smpclient.

If I want to support this type of image, would a simple change (For example, changing it to IMAGE_TLV_PROT_INFO_MAGIC) be enough to make it work?

Thanks!!

Ideally we add support in smpclient lib so that it's fixed in smpmgr. Here's an example of a PR that updated that MCUBoot file: intercreate/smpclient#85

I don't think that any handling other than allowing the value is required.

If you're able to share the binary, it's really nice to have real binary images in the unit tests. No pressure if it's proprietary.