Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

modules: littlefs: update to v2.10.1 #84003

Merged

Conversation

mkaranki
Copy link

@mkaranki mkaranki commented Jan 15, 2025

Update LitteFS to latest upstream released at the moment.

@zephyrbot
Copy link
Collaborator

zephyrbot commented Jan 15, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
littlefs zephyrproject-rtos/littlefs@009bcff zephyrproject-rtos/littlefs@424a5e0 (zephyr) zephyrproject-rtos/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@zephyrbot zephyrbot added manifest manifest-littlefs DNM This PR should not be merged (Do Not Merge) labels Jan 15, 2025
@mkaranki mkaranki changed the title DNM: modules: littlefs: update to v2.10.1 modules: littlefs: update to v2.10.1 Jan 15, 2025
@mkaranki mkaranki force-pushed the update-littlefs-to-v2.10.1 branch from 068b7ee to 75c5176 Compare January 15, 2025 09:24
@@ -290,7 +290,7 @@ manifest:
path: modules/fs/littlefs
groups:
- fs
revision: 009bcff0ed4853a53df8256039fa815bda6854dd
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

424a5e0d5ab273ab6833453288ccf64342c7a5cd

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done.

Update LittleFS to version 2.10.1.

Signed-off-by: Miika Karanki <[email protected]>
@mkaranki mkaranki force-pushed the update-littlefs-to-v2.10.1 branch from 75c5176 to 0be9b2b Compare January 15, 2025 17:31
@mkaranki mkaranki marked this pull request as ready for review January 15, 2025 17:32
@zephyrbot zephyrbot added size: XS A PR changing only a single line of code and removed DNM This PR should not be merged (Do Not Merge) labels Jan 15, 2025
@mkaranki
Copy link
Author

The main motivation of littlefs uplift is this bug: littlefs-project/littlefs#1057 which seems get fixed in 2.10.1. When that is confirmed, to me, it sounds it could be worth backporting this to the supported Zephyr versions as well. Zephyr 3.7 LTS has littlefs version 2.8.1. I don't know if uplifting it to 2.10.1 is viable option here?

@de-nordic de-nordic added this to the v4.1.0 milestone Jan 16, 2025
@kartben
Copy link
Collaborator

kartben commented Jan 17, 2025

tangential: @mkaranki @de-nordic just wonder if any of you could help give some love to #79436 ?

@kartben kartben merged commit 0ae89e9 into zephyrproject-rtos:main Jan 17, 2025
29 of 30 checks passed
@de-nordic
Copy link
Collaborator

tangential: @mkaranki @de-nordic just wonder if any of you could help give some love to #79436 ?

The love is needed here zephyrproject-rtos/littlefs#15 first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
manifest manifest-littlefs size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants