Skip to content
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

samples: benchmark: coremark: add FLPR support for nRF54L15 DK #20462

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kapi-no
Copy link
Contributor

@kapi-no kapi-no commented Feb 18, 2025

Added the FLPR core support for the nRF54L15 DK board target in the CoreMark sample.

Ref: NCSDK-30327

@kapi-no kapi-no requested review from a team as code owners February 18, 2025 14:13
@github-actions github-actions bot added the doc-required PR must not be merged without tech writer approval. label Feb 18, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 18, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: f37376522946aff138cc65cf756638ee38e4434b

more details

sdk-nrf:

PR head: f37376522946aff138cc65cf756638ee38e4434b
merge base: cbeb54f710176f373585e10aafc8bf8c64582031
target head (main): ae518d055670fcca964fbe0f10809a4dd4481e8b
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 (8)
doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
samples
│  ├── benchmarks
│  │  ├── coremark
│  │  │  ├── Kconfig
│  │  │  ├── Kconfig.sysbuild
│  │  │  ├── README.rst
│  │  │  ├── boards
│  │  │  │  ├── nrf54l15dk_nrf54l15_cpuapp.overlay
│  │  │  │  ├── nrf54l15dk_nrf54l15_cpuflpr.conf
│  │  │  │  │ nrf54l15dk_nrf54l15_cpuflpr.overlay
│  │  │  │ sysbuild.cmake

Outputs:

Toolchain

Version: 4cff34261a
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4cff34261a_bece0367df

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 41
  • ✅ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • 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
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

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

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link.

Note: This comment is automatically posted by the Documentation Publish GitHub Action.

Comment on lines 11 to 14
!(SOC_NRF54H20_CPUPPR)
config APP_BUTTON_AND_LED_UNSUPPORTED
bool
default y if SOC_NRF54H20_CPUPPR
default y if SOC_NRF54L05_CPUFLPR
default y if SOC_NRF54L10_CPUFLPR
default y if SOC_NRF54L15_CPUFLPR
Copy link
Contributor

Choose a reason for hiding this comment

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

consider: please add help why it is not supported if possible

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added a comment for the nRF54H20 DK as it seems that one button on the nRF54L15 DK is assigned to a different GPIO port. Thanks to that, I managed to have a different button assigned for the App core (Button 0) and a different button assigned for the FLPR core (Button 3).

@kapi-no kapi-no force-pushed the coremark_flpr_support_nrf54l branch from b31f082 to 136595e Compare February 20, 2025 12:37
@kapi-no kapi-no force-pushed the coremark_flpr_support_nrf54l branch 4 times, most recently from a05ccc4 to f057a5e Compare February 20, 2025 12:54
Added the FLPR core support for the nRF54L15 DK board target in the
CoreMark sample.

Ref: NCSDK-30327

Signed-off-by: Kamil Piszczek <[email protected]>
@kapi-no kapi-no force-pushed the coremark_flpr_support_nrf54l branch from f057a5e to f373765 Compare February 20, 2025 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants