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

[nrf fromtree] samples: subsys: settings: Enable test on nrf54L20 PDK #2402

Conversation

nordic-segl
Copy link
Contributor

@nordic-segl nordic-segl commented Dec 18, 2024

Almost all overlays only enable NVS and SETTINGS_NVS, something that
should be enabled in sample directly. Remove all overlays and make the
sample more genenric.

Use filter on storage partition to enable testing the sample on more
plaforms.

Signed-off-by: Anas Nashif [email protected]

Upstream PR #: 86479

manifest-pr-skip

nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Dec 18, 2024
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#83165

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from 3436df9 to 0197743 Compare January 8, 2025 08:49
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Jan 8, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#83165

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch 2 times, most recently from 12dd744 to 6b9bfa1 Compare January 9, 2025 07:30
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from 6b9bfa1 to 8100ed7 Compare January 27, 2025 06:25
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Jan 27, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#83165

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from 8100ed7 to c0a2321 Compare February 13, 2025 07:58
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Feb 13, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#83165

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from c0a2321 to a58acd3 Compare February 19, 2025 08:04
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Feb 19, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#83165

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from a58acd3 to bd91b3e Compare February 19, 2025 09:47
@nordic-segl nordic-segl reopened this Feb 19, 2025
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch 2 times, most recently from 3268ee6 to 661a673 Compare February 19, 2025 10:30
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch 5 times, most recently from 6e2c738 to 8d029b6 Compare March 6, 2025 09:51
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from 8d029b6 to a8aebaf Compare March 6, 2025 09:55
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 6, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#83165

Signed-off-by: Sebastian Głąb <[email protected]>
Add USB variant config files for esp32c3 boards.

Signed-off-by: Raffael Rostagno <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 8048a6a)
Enable flash samples for s32z board

Signed-off-by: Cong Nguyen Huu <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 8627063)
…k board

The tests for settings will not work properly with just provided config
file for mimxrt1020:
west build -p always -b mimxrt1020_evk samples/subsys/settings

as there the settings_save_one() would return -2 (-ENOENT) as by default
the CONFIG_SETTINGS_NONE is defined for mimxrt1020 configuration.

For proper operation (i.e. to have a memory medium to store settings
values) the NVS backend needs to be enabled.

Signed-off-by: Lukasz Majewski <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 8719aba)
S2 has 12-bit ADC.

Signed-off-by: Marek Matej <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 37f2f7e)
Remove ADC2 support (identified as adc1 entry)
for additional ESP32-C3 based board targets.
Other boards have those instances already deleted.

Signed-off-by: Sylvio Alves <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 40b474c)
Added build configuration for nRF54L09.

Signed-off-by: Rafał Kuźnia <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 5064401)
Move overlays and conf from boards to socs in following samples :
- samples/drivers/adc/adc_dt
- samples/drivers/counter/alarm
- samples/drivers/dac
- samples/subsys/settings

Signed-off-by: Marek Matej <[email protected]>

add more samples

Signed-off-by: Marek Matej <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit b58c424)
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from a8aebaf to 088345c Compare March 10, 2025 06:19
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 10, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#86479
(zephyrproject-rtos/zephyr#83165)

Signed-off-by: Sebastian Głąb <[email protected]>
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 10, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#86479
(zephyrproject-rtos/zephyr#83165)

Signed-off-by: Sebastian Głąb <[email protected]>
Almost all overlays only enable NVS and SETTINGS_NVS, something that
should be enabled in sample directly. Remove all overlays and make the
sample more genenric.

Use filter on storage partition to enable testing the sample on more
plaforms.

Signed-off-by: Anas Nashif <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 270c4e4)
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch from 088345c to 333e325 Compare March 10, 2025 06:23
@nordic-segl nordic-segl changed the title [nrf fromlist] samples: subsys: settings: Enable test on nrf54L20 PDK [nrf fromtree] samples: subsys: settings: Enable test on nrf54L20 PDK Mar 10, 2025
@PerMac PerMac merged commit 9476b04 into nrfconnect:main Mar 10, 2025
18 checks passed
@nordic-segl nordic-segl deleted the NRFX-6832_Enable-settings-tests-on-nrf54l20pdk branch March 10, 2025 08:09
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Mar 10, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#86479
(zephyrproject-rtos/zephyr#83165)

Signed-off-by: Sebastian Głąb <[email protected]>
nordic-piks pushed a commit to nrfconnect/sdk-nrf that referenced this pull request Mar 10, 2025
Include
samples: subsys: settings: Enable test on nrf54L20 PDK
nrfconnect/sdk-zephyr#2402
zephyrproject-rtos/zephyr#86479
(zephyrproject-rtos/zephyr#83165)

Signed-off-by: Sebastian Głąb <[email protected]>
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.

5 participants