Not able to combine OpenThread, MCUBoot and LittleFS

RE: Not able to combine OpenThread, MCUBoot and LittleFS

Riccardo Gaiati — Mon, 25 Apr 2022 09:04:12 GMT

Hi Jørgen, thanks for the prompt reply.

So it is available in the latest version of NCS 1.9.1.

RE: Not able to combine OpenThread, MCUBoot and LittleFS

Jørgen Holmefjord — Mon, 25 Apr 2022 07:30:21 GMT

Hi,

This should be fixed by the following pull requests:

Best regards,
Jørgen

RE: Not able to combine OpenThread, MCUBoot and LittleFS

Riccardo Gaiati — Mon, 25 Apr 2022 07:00:14 GMT

Hi Jorgen,

are there any update for this problem ?

Thanks in advance!

RE: Not able to combine OpenThread, MCUBoot and LittleFS

Jørgen Holmefjord — Thu, 16 Dec 2021 15:59:03 GMT

Hi, we have internal tickets to look into fixing this in the official nRF Connect SDK releases.

RE: Not able to combine OpenThread, MCUBoot and LittleFS

Axel Schlueter — Sun, 12 Dec 2021 20:52:18 GMT

I'd like to +1 Farhang's concern, having to patch ncs isn't a great fix. Any chances of getting the necessary changes into the official ncs tree?

RE: Not able to combine OpenThread, MCUBoot and LittleFS

Farhang — Tue, 07 Dec 2021 19:04:35 GMT

Jørgen Holmefjord The patch you provided works, but it requires commits to both Zephyr and NRF repos.

Is this a good fix? if so, is Nordic going to apply it to the repos?

Alternatively could all the settings (including OpenThread, BT,etc.) be switched over to using FS/LFS?

RE: Not able to combine OpenThread, MCUBoot and LittleFS

Jørgen Holmefjord — Fri, 07 May 2021 08:50:37 GMT

Hi,

The error generated in flash_map_pm.h is only generated to guard the definition of the 'storage' symbol defined above, to make sure that the symbol is set and used correctly.

If you redefine the littlefs_storage variable on line 23 to lfs_storage, like this:

#if (CONFIG_SETTINGS_FCB || CONFIG_SETTINGS_NVS)
#define storage settings_storage
#elif CONFIG_FILE_SYSTEM_LITTLEFS
#define lfs_storage littlefs_storage
#elif CONFIG_NVS
#define storage nvs_storage
#endif

Then you can remove the error check:

//#if (CONFIG_SETTINGS_FCB || CONFIG_SETTINGS_NVS) && CONFIG_FILE_SYSTEM_LITTLEFS
//#error "Not supported"
//#endif

You can then replace the 'storage' symbol in the LittleFS application with littlefs_storage symbol:

static struct fs_mount_t lfs_storage_mnt = {
	.type = FS_LITTLEFS,
	.fs_data = &littlefs_storage,
	.storage_dev = (void *)FLASH_AREA_ID(storage),
	.mnt_point = "/lfs",
};

I have attached patches that enables the modules in the sample and resolves the error, one for Zephyr and one for the nRF repository. Both needs to be applied:

devzone.nordicsemi.com/.../littlefs_5F00_zephyr.patch

devzone.nordicsemi.com/.../littlefs_5F00_nrf.patch