Skip to content

Conversation

@Rallare
Copy link
Contributor

@Rallare Rallare commented Nov 18, 2025

As per Zephyr upstream PR #91219, SUBALIGN functionality has been misunderstood throughout the upstream project.
Aligning our downstream accordingly.

@Rallare Rallare requested a review from a team as a code owner November 18, 2025 12:42
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Nov 18, 2025
@NordicBuilder NordicBuilder requested review from a team and Balaklaka November 18, 2025 12:42
As per Zephyr upstream PR #91219, SUBALIGN functionality has been
misunderstood throughout the upstream project.
Aligning our downstream accordingly.

Signed-off-by: Håkon Alseth <[email protected]>
@Rallare Rallare force-pushed the linker_subalign_alignment branch from 76bc11c to bd64f88 Compare November 18, 2025 12:46
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 18, 2025

CI Information

To view the history of this post, click the 'edited' button above
Build number: 2

Inputs:

Sources:

sdk-nrf: PR head: bd64f88559fde8f89778fd0470acd0c7f64664bb

more details

sdk-nrf:

PR head: bd64f88559fde8f89778fd0470acd0c7f64664bb
merge base: 02d4cdce6b62b644839c4d2eaea0d39a4e935412
target head (main): 73a74b7ff6d54f07f5bfe61ed6764fb179e482fd
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 (6)
subsys
│  ├── app_event_manager
│  │  │ CMakeLists.txt
│  ├── app_event_manager_profiler_tracer
│  │  │ CMakeLists.txt
│  ├── bluetooth
│  │  ├── adv_prov
│  │  │  │ CMakeLists.txt
│  │  ├── services
│  │  │  ├── fast_pair
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── fp_storage
│  │  │  │  │  │ CMakeLists.txt
│  ├── emds
│  │  │ CMakeLists.txt

Outputs:

Toolchain

Version: df3cc9d822
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:df3cc9d822_e595b21c39

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

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

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

Copy link
Contributor

@alxelax alxelax left a comment

Choose a reason for hiding this comment

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

emds is Ok

@carlescufi carlescufi merged commit 0638391 into nrfconnect:main Nov 19, 2025
18 checks passed
@Rallare Rallare deleted the linker_subalign_alignment branch November 19, 2025 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants