Skip to content

Commit

Permalink
tests: multicore: Use power states definition from SOC
Browse files Browse the repository at this point in the history
Instead of defining power states per test, add power states
to the nrf54h20/cpuapp target definition.

Remove power states from tests overlays to prevent
build failures due to duplicated node definitions.

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
  • Loading branch information
nordic-segl committed Sep 27, 2024
1 parent fb290bd commit 648ffc8
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 71 deletions.

This file was deleted.

5 changes: 2 additions & 3 deletions tests/benchmarks/multicore/idle/testcase.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@ tests:
remote_CONFIG_PM=y remote_CONFIG_POWEROFF=y remote_CONFIG_CONSOLE=n
remote_CONFIG_UART_CONSOLE=n remote_CONFIG_SERIAL=n remote_CONFIG_GPIO=n
remote_CONFIG_BOOT_BANNER=n remote_CONFIG_NRFS_MRAM_SERVICE_ENABLED=n
DTC_OVERLAY_FILE="boards/nrf54h20dk_nrf54h20_cpuapp_s2ram.overlay"
harness: pytest
harness_config:
fixture: ppk_power_measure
Expand All @@ -98,7 +97,7 @@ tests:
remote_CONFIG_PM=y remote_CONFIG_POWEROFF=y remote_CONFIG_CONSOLE=n
remote_CONFIG_UART_CONSOLE=n remote_CONFIG_SERIAL=n remote_CONFIG_GPIO=n
remote_CONFIG_BOOT_BANNER=n remote_CONFIG_NRFS_MRAM_SERVICE_ENABLED=n
DTC_OVERLAY_FILE="boards/nrf54h20dk_nrf54h20_cpuapp_s2ram.overlay;boards/nrf54h20dk_nrf54h20_cpuapp_ram_high_usage.overlay"
DTC_OVERLAY_FILE="boards/nrf54h20dk_nrf54h20_cpuapp_ram_high_usage.overlay"
harness: pytest
harness_config:
fixture: ppk_power_measure
Expand All @@ -120,7 +119,7 @@ tests:
remote_CONFIG_PM=y remote_CONFIG_POWEROFF=y remote_CONFIG_CONSOLE=n
remote_CONFIG_UART_CONSOLE=n remote_CONFIG_SERIAL=n remote_CONFIG_GPIO=n
remote_CONFIG_BOOT_BANNER=n remote_CONFIG_NRFS_MRAM_SERVICE_ENABLED=n
DTC_OVERLAY_FILE="boards/nrf54h20dk_nrf54h20_cpuapp_s2ram.overlay;boards/nrf54h20dk_nrf54h20_cpuapp_ram_low_usage.overlay"
DTC_OVERLAY_FILE="boards/nrf54h20dk_nrf54h20_cpuapp_ram_low_usage.overlay"
harness: pytest
harness_config:
fixture: ppk_power_measure
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,30 +5,13 @@
*/

/ {
power-states {
idle: idle {
compatible = "zephyr,power-state";
power-state-name = "suspend-to-idle";
min-residency-us = <100000>;
};

s2ram: s2ram {
compatible = "zephyr,power-state";
power-state-name = "suspend-to-ram";
min-residency-us = <800000>;
};
};
aliases {
/delete-property/ sw1;
};
};

/delete-node/ &button1;

&cpu {
cpu-power-states = <&idle &s2ram>;
};

&gpiote130 {
status = "okay";
owned-channels = <0>;
Expand Down

This file was deleted.

2 changes: 1 addition & 1 deletion west.yml
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ manifest:
# https://developer.nordicsemi.com/nRF_Connect_SDK/doc/latest/zephyr/guides/modules.html
- name: zephyr
repo-path: sdk-zephyr
revision: 3d01dcc251bf3aa2a675941e8ac2b244f776724d
revision: pull/2052/head
import:
# In addition to the zephyr repository itself, NCS also
# imports the contents of zephyr/west.yml at the above
Expand Down

0 comments on commit 648ffc8

Please sign in to comment.