-
Notifications
You must be signed in to change notification settings - Fork 624
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: Move MPI to sysbuild Kconfig #2289
Open
tomchy
wants to merge
10
commits into
nrfconnect:main
Choose a base branch
from
tomchy:feature/suit/NCSDK-30461_Move_mpi_to_sysbuild
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
suit: Move MPI to sysbuild Kconfig #2289
tomchy
wants to merge
10
commits into
nrfconnect:main
from
tomchy:feature/suit/NCSDK-30461_Move_mpi_to_sysbuild
+248
−170
Conversation
This file contains 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
This was referenced Nov 21, 2024
tomchy
force-pushed
the
feature/suit/NCSDK-30461_Move_mpi_to_sysbuild
branch
9 times, most recently
from
November 26, 2024 12:48
c69afba
to
ffbdc7b
Compare
…size option Adds a --download-buffer-size option to the canopen runner to allow specifying the buffer size of the SDO download. Signed-off-by: Fabian Blatz <[email protected]> (cherry picked from commit 11c83c8) Signed-off-by: Tomasz Chyrowicz <[email protected]>
This adds support for J-Link tunnels, which run on top of an IP network and therefore uses the -IP option. J-Link tunnels are identified by a tunnel: prefix instead of a bare IP address. This change checks for the presence of such a prefix, and choses the -IP transport option if the tunnel prefix is found. This has been tested with J-Link Remote Server v7.98g and the SEGGER tunnel option. Signed-off-by: Adam Dunkels <[email protected]> (cherry picked from commit 221199e) Signed-off-by: Tomasz Chyrowicz <[email protected]>
…emihost LinkServer manages port numbers for gdb and semihost as separate linear sequences when invoked to debug multi-core applications, e.g the gdb-server instance for cpu0 will have the default GDB port 3333 and the next gdb-server instance will be assigned the port 3334. The latter will conflict with the default port for semihost which is 3334. This patch changes the default port for semihost to 8888. Port numbers can be changed when invoking the linkserver runner. Signed-off-by: Yves Vandervennet <[email protected]> (cherry picked from commit 7b29b66bd5ff3370409444b36865c9fea7450dfe) Signed-off-by: Tomasz Chyrowicz <[email protected]>
…t ignores These ignores were added with an older version of pylint and are either removed or ignored by default. Signed-off-by: Pieter De Gendt <[email protected]> (cherry picked from commit aae018be873236a136264c8b65de1397c58cbeb5) Signed-off-by: Tomasz Chyrowicz <[email protected]>
…sues Variables are possibly unbound, initialize to None. Signed-off-by: Pieter De Gendt <[email protected]> (cherry picked from commit 25be1501abb11eefa223575c7c148e51adaeccc9) Signed-off-by: Tomasz Chyrowicz <[email protected]>
…placeholders (F541) Constant strings should not be f-strings. See https://docs.astral.sh/ruff/rules/f-string-missing-placeholders/ Signed-off-by: Pieter De Gendt <[email protected]> (cherry picked from commit 7f72dd3a0f338206a8112d7deca89f0f19be2461) Signed-off-by: Tomasz Chyrowicz <[email protected]>
… & B904) Catch explicit exceptions and pass them them to the ValueError See https://docs.astral.sh/ruff/rules/bare-except/ https://docs.astral.sh/ruff/rules/raise-without-from-inside-except/ Signed-off-by: Pieter De Gendt <[email protected]> (cherry picked from commit f7902bc817d7e3e7655602bc44f0dccc6e09d394) Signed-off-by: Tomasz Chyrowicz <[email protected]>
Replace .format() calls with f-strings. See https://docs.astral.sh/ruff/rules/f-string/ Signed-off-by: Pieter De Gendt <[email protected]> (cherry picked from commit 775db0e63a547ceabbf55d44f5f651901e1c36a7) Signed-off-by: Tomasz Chyrowicz <[email protected]>
Wrap long lines to max 100 characters. See https://docs.astral.sh/ruff/rules/line-too-long/ Signed-off-by: Pieter De Gendt <[email protected]> (cherry picked from commit 336c7dadac30ac41773aa86e01d43e05ccd036c2) Signed-off-by: Tomasz Chyrowicz <[email protected]>
Use sysbuild to build and configure SUIT Manifest Provisioning Information. Ref: NCSDK-30461 Upstream PR #: 81723 Signed-off-by: Tomasz Chyrowicz <[email protected]>
tomchy
force-pushed
the
feature/suit/NCSDK-30461_Move_mpi_to_sysbuild
branch
from
November 26, 2024 12:53
ffbdc7b
to
5f2f74a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Use sysbuild to build and configure SUIT Manifest Provisioning Information.
Ref: NCSDK-30461