Skip to content

Conversation

@kartben
Copy link
Contributor

@kartben kartben commented Nov 26, 2025

Several services are better off grouped under the existing storage "meta" category instead of polluting the top-level table of contents.

https://builds.zephyrproject.io/zephyr/pr/100125/docs/services/index.html
https://builds.zephyrproject.io/zephyr/pr/100125/docs/services/storage/index.html

pdgendt
pdgendt previously approved these changes Nov 26, 2025
nashif
nashif previously approved these changes Nov 28, 2025
This moves retention into a better suited home since we already have
a section for all things storage.

Signed-off-by: Benjamin Cabé <[email protected]>
This moves fs docs to a better suited home since we already have
a section for all things storage.

Signed-off-by: Benjamin Cabé <[email protected]>
This moves nvmem docs to a better suited home since we already have
a section for all things storage.

Signed-off-by: Benjamin Cabé <[email protected]>
Sort entries alphabetically instead of the current somewhat random order.

Signed-off-by: Benjamin Cabé <[email protected]>
@kartben kartben dismissed stale reviews from nashif and pdgendt via 089688a November 28, 2025 13:33
@kartben kartben force-pushed the consolidate_storage branch from 93a5aa3 to 089688a Compare November 28, 2025 13:33
@kartben
Copy link
Contributor Author

kartben commented Nov 28, 2025

I had to rebase as GitHub was seeing a conflict in nvmem/index.rst but there were actually none and local rebase was a no-op.

@kartben kartben requested review from nashif and pdgendt November 28, 2025 13:34
@sonarqubecloud
Copy link

@kartben kartben merged commit da95e84 into zephyrproject-rtos:main Dec 1, 2025
20 checks passed
@nordicjm
Copy link
Contributor

nordicjm commented Dec 1, 2025

retention should not have been moved, storage is for e.g. flash and retention is purely for RAM (or RAM-like peripherals), it does not belong with storage

@kartben kartben deleted the consolidate_storage branch December 1, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants