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.

Sign up for GitHub

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

Jump to bottom

tests/subsys/settings/functional/nvs: add nrf52xx configuration #25708

Merged
merged 1 commit into from
May 28, 2020
Merged

tests/subsys/settings/functional/nvs: add nrf52xx configuration #25708

merged 1 commit into from
May 28, 2020

Conversation

nvlsianpu
Copy link
Collaborator

Added configuration which allows to run test-suite
on nrf52840dk_nrf52840 and nrf52dk_rf52832 boards.

fixes #25701

Signed-off-by: Andrzej Puzdrowski andrzej.puzdrowski@nordicsemi.no

@nvlsianpu
tests/subsys/settings/functional/nvs: add nrf52xx configuration
90f5a28 
Added configuration which allows to run test-suite
on nrf52840dk_nrf52840 and nrf52dk_rf52832 boards.

fixes zephyrproject-rtos#25701

Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
@nvlsianpu nvlsianpu mentioned this pull request May 28, 2020
Closed
@zephyrbot zephyrbot added the area: Tests Issues related to a particular existing or missing test label May 28, 2020
de-nordic
de-nordic approved these changes May 28, 2020
View reviewed changes
Copy link
Collaborator

@de-nordic de-nordic left a comment

Choose a reason for hiding this comment

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

Looks fine.

ioannisg
ioannisg approved these changes May 28, 2020
View reviewed changes
Copy link
Member

@ioannisg ioannisg left a comment

Choose a reason for hiding this comment

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

I think this is on the right direction, but I assume any Cortex-M platform with MPU can potentially run this test, not only nRF52-based targets.

@ioannisg ioannisg requested a review from carlescufi May 28, 2020 15:08
@carlescufi carlescufi merged commit 48af970 into zephyrproject-rtos:master May 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@de-nordic de-nordic de-nordic approved these changes

@kapi-no kapi-no kapi-no approved these changes

@carlescufi carlescufi carlescufi approved these changes

@ioannisg ioannisg ioannisg approved these changes

Assignees
No one assigned
Labels
area: Tests Issues related to a particular existing or missing test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

MPU FAULT in nvs test on nrf52840dk_nrf52840
6 participants
@nvlsianpu @ioannisg @carlescufi @kapi-no @de-nordic @zephyrbot