[nrf fromtree dirty] platform: nordic_nrf: Add SPDX tags to device_cfg.h files - #272
Merged
Merged
Conversation
sigvartmh
requested review from
a team,
adrianiainlam,
degjorva,
tomi-font and
travis3630
August 11, 2026 12:03
adrianiainlam
approved these changes
Aug 11, 2026
Vge0rge
approved these changes
Aug 11, 2026
sigvartmh
force-pushed
the
fix/add-spdx2-license
branch
from
August 11, 2026 12:18
2ac2aab to
5b47ea4
Compare
Contributor
Author
|
This is now dependent on #263 to go in first so that we don't have dirty cherry-picks |
NordicBuilder
added a commit
to NordicBuilder/sdk-nrf
that referenced
this pull request
Aug 11, 2026
Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-trusted-firmware-m#272 Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
sigvartmh
changed the title
tomi-font
changed the title
tomi-font
approved these changes
Aug 12, 2026
sigvartmh
force-pushed
the
fix/add-spdx2-license
branch
2 times, most recently
from
August 13, 2026 10:30
a486ba1 to
680db2d
Compare
sigvartmh
changed the title
tomi-font
reviewed
Aug 13, 2026
…g.h files The Apache-2.0 notice in these files is missing the punctuation of the standard boilerplate, so license scanners cannot match it by full text, and there is no SPDX tag to fall back on. Tools consequently report the files as carrying no license at all. Add an SPDX-License-Identifier tag stating the license each notice already grants. This follows nrf9161dk_nrf9161, previously the only Nordic board whose device_cfg.h carried a tag. The notice text itself is left untouched. All ten remaining Nordic device_cfg.h files share the same ARM header and are covered here, so a newly added board no longer reintroduces the problem. Change-Id: Ifc30600d7e9a8a9801190a63c2266e4a9d37e970 Signed-off-by: Sigvart Hovland <sigvart.m@gmail.com> (cherry picked from commit 717da8af8f955b6aa745b95e363c815f8b83a43c)
sigvartmh
force-pushed
the
fix/add-spdx2-license
branch
from
August 13, 2026 10:37
680db2d to
4072656
Compare
NordicBuilder
added a commit
to NordicBuilder/sdk-nrf
that referenced
this pull request
Aug 13, 2026
Automatically created by action-manifest-pr GH action from PR: nrfconnect/sdk-trusted-firmware-m#272 Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This now also adds
To avoid cherry-pick conflict.
Also adds 2 commits from #263