NordicSemiconductor / IOS-nRF-Connect

Info page
https://nordicsemiconductor.github.io/IOS-nRF-Connect/
136 stars 32 forks source link

Uploading an image that uses external SPI flash throws “Error: Not Supported” #159

Closed zpm1066 closed 1 month ago

zpm1066 commented 4 months ago

Hello, I’m seeing a potential bug when uploading an image to the external flash of a device.

After the upload, I get the “Error: Not supported” and device disconnects. However, FOTA BLE image upload works fine.

Uploading an image that uses the SoC flash is fine. After the upload, nRF Connect waits for device to swap partitions and reboot with the new image, and messages states “Success!”.

Please review and provide a bug fix.

nRF Connect is a great app. Keep it up! Thank you.

Screen Shot 2024-04-19 at 12 00 18 PM

Screen Shot 2024-04-19 at 12 16 05 PM Screen Shot 2024-04-19 at 12 13 10 PM
philips77 commented 4 months ago

I think this applies to "Erase app storage" command. It doens't matter for the success of upgrading the device. You may also disable erasing storage.

zpm1066 commented 4 months ago

Thank you.

The error is misleading and should say “Success!” if it doesn’t impact the success of the device upgrade process.

More importantly, in the case of an image that is using an external flash , the upload doesn’t display “Awaiting Device Reset…” but simply uploads and drops the BLE connection. This may lead user to think that something is wrong with the upload.

From a user’s perspective, we should present the same behaviour irrespective of what type of flash the uploaded image uses.

philips77 commented 4 months ago

Which upload mode are you using? Are you using Direct XIP as well?

zpm1066 commented 4 months ago

I’m using “DFU” from nRF Connect for mobile iOS and using the defaults.