Skip to content

Conversation

tokangas
Copy link
Contributor

@tokangas tokangas commented Aug 8, 2025

Added SoC support for Sercomm TPM530M module and board configuration for TPM530M EVK.

NCSDK-27893

@tokangas tokangas requested review from a team as code owners August 8, 2025 09:08
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Aug 8, 2025
Copy link
Contributor

@tejlmand tejlmand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, couple of minor observations.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 8, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 10

Inputs:

Sources:

sdk-nrf: PR head: f66523fa665991aa464e221203239fcbe2743401
zephyr: PR head: a922397417e49858aa7a6fd98d8ef79f6dee3666

more details

sdk-nrf:

PR head: f66523fa665991aa464e221203239fcbe2743401
merge base: 4eb29ffba260d2138cccfa18b59fe92d63e7239a
target head (main): 4eb29ffba260d2138cccfa18b59fe92d63e7239a
Diff

zephyr:

PR head: a922397417e49858aa7a6fd98d8ef79f6dee3666
merge base: 8f6f73e398f415c08bfa9ae7e3b02e03eb66c46e
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (39)
CODEOWNERS
boards
│  ├── nordic
│  │  ├── nrf52820dongle
│  │  │  │ nrf52820dongle_nrf52820_defconfig
│  │  ├── nrf52833dongle
│  │  │  │ nrf52833dongle_nrf52833_defconfig
│  │  ├── nrf52840gmouse
│  │  │  │ nrf52840gmouse_nrf52840_defconfig
│  │  ├── nrf52dmouse
│  │  │  │ nrf52dmouse_nrf52832_defconfig
│  │  ├── nrf52kbd
│  │  │  │ nrf52kbd_nrf52832_defconfig
│  │  ├── thingy91
│  │  │  ├── thingy91_nrf52840_defconfig
│  │  │  ├── thingy91_nrf9160_defconfig
│  │  │  │ thingy91_nrf9160_ns_defconfig
│  │  ├── thingy91x
│  │  │  ├── thingy91x_nrf5340_cpuapp_defconfig
│  │  │  ├── thingy91x_nrf5340_cpuapp_ns_defconfig
│  │  │  ├── thingy91x_nrf5340_cpunet_defconfig
│  │  │  ├── thingy91x_nrf9151_defconfig
│  │  │  │ thingy91x_nrf9151_ns_defconfig
│  ├── sercomm
│  │  ├── tpm530mevk
│  │  │  ├── Kconfig.defconfig
│  │  │  ├── Kconfig.tpm530mevk
│  │  │  ├── board.cmake
│  │  │  ├── board.yml
│  │  │  ├── pre_dt_board.cmake
│  │  │  ├── tpm530mevk_tpm530m.dts
│  │  │  ├── tpm530mevk_tpm530m.yaml
│  │  │  ├── tpm530mevk_tpm530m_common-pinctrl.dtsi
│  │  │  ├── tpm530mevk_tpm530m_common.dtsi
│  │  │  ├── tpm530mevk_tpm530m_defconfig
│  │  │  ├── tpm530mevk_tpm530m_ns.dts
│  │  │  ├── tpm530mevk_tpm530m_ns.yaml
│  │  │  │ tpm530mevk_tpm530m_ns_defconfig
dts
│  ├── arm
│  │  ├── sercomm
│  │  │  ├── tpm530m.dtsi
│  │  │  │ tpm530mns.dtsi
│  ├── bindings
│  │  │ vendor-prefixes.txt
soc
│  ├── sercomm
│  │  ├── nrf91
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── Kconfig.defconfig
│  │  │  ├── Kconfig.defconfig.tpm530m
│  │  │  ├── Kconfig.soc
│  │  │  │ soc.yml
west.yml
zephyr
│  ├── scripts
│  │  ├── bindings_properties_allowlist.yaml
│  │  ├── ci
│  │  │  │ check_compliance.py

Outputs:

Toolchain

Version: c5be9c56c7
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:c5be9c56c7_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 285
    • sdk-zephyr test count: 687
  • ✅ Integration tests
    • ✅ desktop52_verification
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

Copy link

github-actions bot commented Aug 8, 2025

You can find the documentation preview for this PR here.

@tokangas tokangas force-pushed the tpm530m_support branch 2 times, most recently from 35cb03b to 06cbee9 Compare August 8, 2025 10:13
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 8, 2025
@tokangas tokangas force-pushed the tpm530m_support branch 2 times, most recently from 4fcb4f5 to 6de11ee Compare August 11, 2025 10:00
@github-actions github-actions bot removed the doc-required PR must not be merged without tech writer approval. label Aug 11, 2025
@tokangas
Copy link
Contributor Author

Removed the board documentation because it is not picked up by the build system. Discussion ongoing where the board documentation should go. It will be handled in a separate PR.

@tokangas tokangas requested a review from a team as a code owner August 11, 2025 12:33
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 11, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@8f6f73e nrfconnect/sdk-zephyr@a922397 (main) nrfconnect/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

tokangas and others added 3 commits August 15, 2025 08:03
Added support for Sercomm TPM530M.

Signed-off-by: Tommi Kangas <[email protected]>
Added board configuration for TPM530M EVK.

Signed-off-by: Tommi Kangas <[email protected]>
CONFIG_PINCTRL=y fron defconfig files.
This is not the right location and should be removed.

Signed-off-by: Torsten Rasmussen <[email protected]>
@tokangas
Copy link
Contributor Author

@nrfconnect/ncs-co-drivers @nrfconnect/ncs-merge @nrfconnect/ncs-co-boards @nrfconnect/ncs-cia @nrfconnect/ncs-si-bluebagel Please review.

@tejlmand tejlmand merged commit 580c8db into nrfconnect:main Aug 15, 2025
14 of 15 checks passed
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.

7 participants