-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
suit: General fixup #20064
suit: General fixup #20064
Conversation
nordicjm
commented
Jan 24, 2025
•
edited
Loading
edited
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 7412166cbd009daedc1e8389ecbcb2172733a51e more detailssdk-nrf:
Github labels
List of changed files detected by CI (32)
Outputs:ToolchainVersion: 342151af73 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. Note: This comment is automatically posted by the Documentation Publish GitHub Action. |
af92dee
to
ea6ee47
Compare
# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause | ||
# | ||
|
||
config SUPPORT_SUIT_STORAGE_LAYOUT_TEST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a mixed feelings about putting this file inside sdfw
directory, but I do not have a better idea to propose.
Interestingly, this file configures storage symbol, but does not MPI nor validator. This is not necessary today, but it justifies the generic naming scheme of this file.
Fixes various styling issues with Kconfig files and changes Kconfig option names that duplicate information which is already known which would lead to more unselectable Kconfig options in future for additional SoCs, lessening the complexity and maintenance burden. Also fixes an issue that was completely hidden whereby an existing Zephyr Kconfig was re-defined, completely obscuring the fact that the original Zephyr Kconfig was removed in 4.0 and had been doing nothing since the upmerge to that version of Zephyr Signed-off-by: Jamie McCrae <[email protected]>
Updates to use new Kconfig symbol names, and enables usage of the test Kconfig only in tests to prevent accidental selection by users where it should not be able to be selected Signed-off-by: Jamie McCrae <[email protected]>