In SDK 1.9.1 littlefs can not be assigned to external flash region

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user97039 — Fri, 10 Jun 2022 16:29:42 GMT

I also looked over the PR and it appears that it should fix the original issue. I look forward to the upcoming SDK release. Thanks

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Fri, 10 Jun 2022 12:33:05 GMT

Yes, it definitely addresses my concern and, as expected, in a much cleaner and more thorough way that I would have been able to implement. Looking forward to this change. Thanks!

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user73165 — Fri, 10 Jun 2022 10:51:52 GMT

Could you check if the suggested solution fixes your problem?

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Thu, 09 Jun 2022 12:13:46 GMT

Wonderful! Thanks!

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user73165 — Thu, 09 Jun 2022 07:55:53 GMT

Hi,

[quote user="KRich"]Great. Thanks for the response. Is there a pull request or open issue that we could follow for progress, so that we don't have to continually follow up here?[/quote]

Yes

regards

Jared

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Wed, 08 Jun 2022 14:06:38 GMT

Great. Thanks for the response. Is there a pull request or open issue that we could follow for progress, so that we don't have to continually follow up here?

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user73165 — Wed, 08 Jun 2022 13:35:16 GMT

It's still under evaluation. I would expect the fix to be included in the next NCS release.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Mon, 06 Jun 2022 16:17:19 GMT

[quote userid="73165" url="~/f/nordic-q-a/86507/in-sdk-1-9-1-littlefs-can-not-be-assigned-to-external-flash-region/369307#369307"]Aright, I'll share the link when it has been submitted. [/quote]

Hi Jared,

Do you have any updates on this change?

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user73165 — Tue, 24 May 2022 13:46:04 GMT

Aright, I'll share the link when it has been submitted.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Tue, 24 May 2022 13:40:58 GMT

Thanks Jared, if possible, it would be great to get a link to the PR if/when the developers submit it. That makes it a bit easier to track the resolution and availability. Thanks!

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user73165 — Tue, 24 May 2022 13:31:19 GMT

Hi,

I've informed our developers, they'll look at the suggested fix and add the PR.

Thanks!

regards

Jared

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Mon, 23 May 2022 17:08:03 GMT

Hi @Jared,

Could you take a look at the discussion below and let us know if someone from Nordic can help get this change through (or let us know how to request this change more formally)?

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Thu, 19 May 2022 11:31:54 GMT

I think the DCO may be a little difference. The CLA came up as I was attempting to create the Pull Request. Here is the link I got, hopefully it will work for you too: https://cla-assistant.io/nrfconnect/sdk-nrf. It seems to load really slowly (and maybe not at all depending on the browser and network settings).

Contributing "company work" back to the community is definitely one of the bigger challenges of OSS. Perhaps someone @Jared could re-open the case and someone from Nordic could walk this change through for us.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user97039 — Thu, 19 May 2022 00:02:57 GMT

I think I found it under Developer Certification of Origin (DCO)¶? I'm also on the clock so to speak, so will have to check a bit further. The downside of OSS?

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user97039 — Wed, 18 May 2022 23:59:59 GMT

Interesting, where is this Contributor License Agreement to be found? Thanks

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Tue, 17 May 2022 17:44:09 GMT

Unfortunately, I got a little ahead of myself. Since I'm doing this "on company time" I am not able to sign the Contributor License Agreement so I will not be able to push the change through the PR process. Hopefully you, or someone from Nordic can take the patch above through the process.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user97039 — Tue, 17 May 2022 03:47:25 GMT

Looks good, what is the link for the PR? Thanks

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Wed, 11 May 2022 11:31:32 GMT

This looks great. Thanks for posting it. I took things a little farther to try to implement the configurability you mentioned. Here is what I have so far.:

diff --git a/subsys/partition_manager/Kconfig b/subsys/partition_manager/Kconfig
index b4fb5050c..721011146 100644
--- a/subsys/partition_manager/Kconfig
+++ b/subsys/partition_manager/Kconfig
@@ -118,6 +118,18 @@ config PM_EXTERNAL_FLASH_MCUBOOT_SECONDARY
 	  of the internal flash. This option should only be enabled by the user
 	  in the parent image.
 
+config PM_EXTERNAL_FLASH_LITTLEFS
+	bool "Place the LittleFS partition in external flash"
+	depends on FILE_SYSTEM_LITTLEFS
+	# Only depend on the chosen for the parent image. The value of this
+	# option is propagated to the MCUboot child image automatically.
+	depends on $(dt_chosen_enabled,$(DT_CHOSEN_EXT_FLASH))
+	default n
+	help
+	  Place the partition for LittleFS in the external flash instead
+	  of the internal flash. This option should only be enabled by the user
+	  in the parent image.
+
 config PM_IMAGE_NOT_BUILT_FROM_SOURCE
 	bool
 	help
diff --git a/subsys/partition_manager/pm.yml.file_system b/subsys/partition_manager/pm.yml.file_system
index 7c5462a87..3ff79a7bd 100644
--- a/subsys/partition_manager/pm.yml.file_system
+++ b/subsys/partition_manager/pm.yml.file_system
@@ -2,6 +2,9 @@
 
 #ifdef CONFIG_FILE_SYSTEM_LITTLEFS
 littlefs_storage:
+#if defined(CONFIG_PM_EXTERNAL_FLASH_LITTLEFS)
+  region: external_flash
+#endif
   placement: {before: [tfm_storage, end]}
   size: CONFIG_PM_PARTITION_SIZE_LITTLEFS
 #endif

I will submit this as the starting point for a PR later today

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user97039 — Tue, 10 May 2022 23:47:31 GMT

Here was the key change in the SDK, it seems by default there was no way to get the littlefs to goto external memory in the partition manager, but it should be selectable in a PR.

There were a number of other changes that were needed as well to get it full configured.

diff --git a/subsys/partition_manager/pm.yml.file_system b/subsys/partition_manager/pm.yml.file_system
index 2b9907380..73008bbde 100644
--- a/subsys/partition_manager/pm.yml.file_system
+++ b/subsys/partition_manager/pm.yml.file_system
@@ -2,6 +2,7 @@
 
 #ifdef CONFIG_FILE_SYSTEM_LITTLEFS
 littlefs_storage:
+  region: external_flash
   placement: {before: [end]}
   size: CONFIG_PM_PARTITION_SIZE_LITTLEFS
 #endif

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user97039 — Tue, 10 May 2022 22:21:30 GMT

I have a solution that seemed to work, but had more important items and I was also waiting for a fix for 32bit SFlash addressing from upstream. I'll post it here shortly if you want to take a look and try it. But I have not had much time to work on a more generic fix that is required to get it working in the official SDK.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user110917 — Tue, 10 May 2022 17:24:46 GMT

Have you had an opportunity to work on this yet? I am facing the same issue and very interested in this change. I would be happy to work up a (or help with your) PR if you don't have time.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user73165 — Mon, 04 Apr 2022 09:26:45 GMT

Great, I'll close this case then.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user97039 — Fri, 01 Apr 2022 19:12:49 GMT

I'll work on a PR this weekend.

RE: In SDK 1.9.1 littlefs can not be assigned to external flash region

user73165 — Fri, 01 Apr 2022 11:07:45 GMT

Hi there,

Could you do a pull request to the NCS repository? The appropriate person will then evaluate the issue and fix.

Thank you!

best regards

Jared