Skip to content

Conversation

@tomchy
Copy link
Contributor

@tomchy tomchy commented Nov 21, 2024

Use sysbuild to build and configure SUIT Manifest Provisioning Information.

Ref: NCSDK-30461

@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 10 times, most recently from ffbdc7b to 5f2f74a Compare November 26, 2024 12:53
@tomchy tomchy marked this pull request as ready for review November 27, 2024 09:37
@tomchy tomchy requested a review from carlescufi November 27, 2024 09:39
@ahasztag ahasztag self-requested a review November 28, 2024 09:18
@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 3 times, most recently from c3e32de to f859c15 Compare December 2, 2024 12:12
@tomchy tomchy requested a review from nordicjm December 2, 2024 13:26
@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 3 times, most recently from b019c7c to 81e4eca Compare December 5, 2024 08:20
@jukkar jukkar requested review from a team, anangl, de-nordic and nordic-krch as code owners December 5, 2024 13:56
@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from 81e4eca to 69fad0e Compare December 5, 2024 14:33
@tomchy tomchy added this to the ncs-2.9.0-nRF54H20 milestone Dec 6, 2024
@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch 3 times, most recently from 923b6e4 to 24a940a Compare December 9, 2024 08:14
faxe1008 and others added 10 commits December 11, 2024 16:39
…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 7b29b66)

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 aae018b)

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 25be150)

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 7f72dd3)

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 f7902bc)

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 775db0e)

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 336c7da)

Signed-off-by: Tomasz Chyrowicz <[email protected]>
Use sysbuild to build and configure SUIT Manifest Provisioning
Information.

Signed-off-by: Tomasz Chyrowicz <[email protected]>
(cherry picked from commit 866e690)
@tomchy tomchy force-pushed the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch from 24a940a to 6fcb5f7 Compare December 11, 2024 15:39
@rlubos rlubos merged commit 6e53c57 into nrfconnect:main Dec 12, 2024
23 of 24 checks passed
@tomchy tomchy deleted the feature/suit/NCSDK-30461_Move_mpi_to_sysbuild branch November 3, 2025 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants