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

sys/board_software_reset: add module for soft-reset button #19833

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

benpicco
Copy link
Contributor

Contribution description

This moves the board_software_reset module from e104-bt50xxa-tb to a common place and let's nrf52840dk make use of it.

Testing procedure

Flash any application on nrf52840dk - a press on the button labeled RESET (SW5) should now reset the board.
In master this button did nothing.

Issues/PRs references

@github-actions github-actions bot added Area: build system Area: Build system Area: boards Area: Board ports Area: sys Area: System labels Jul 14, 2023
@benpicco benpicco added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jul 14, 2023
@benpicco benpicco force-pushed the board_software_reset branch from 9e7642a to c8630a7 Compare July 14, 2023 13:41
@riot-ci
Copy link

riot-ci commented Jul 14, 2023

Murdock results

FAILED

c8630a7 boards/nrf52840dk: enable soft-reset by default

Success Failures Total Runtime
117 0 6934 01m:31s

Artifacts

@bergzand
Copy link
Member

Flash any application on nrf52840dk - a press on the button labeled RESET (SW5) should now reset the board.

The reset pin of the nrf52 needs to be flashed once in a non-volatile configuration register, there is https://github.com/RIOT-OS/RIOT/tree/master/dist/tools/nrf52_resetpin_cfg for that purpose. I don't think your contribution here conflicts with that though.

Copy link
Member

@bergzand bergzand left a comment

Choose a reason for hiding this comment

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

Maybe I'm not looking in the right places, but the documentation doesn't seem to show up in the doxygen for me.

*/

/**
* @@defgroup module_board_software_reset board_software_reset
Copy link
Contributor

@aabadie aabadie Sep 6, 2023

Choose a reason for hiding this comment

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

Suggested change
* @@defgroup module_board_software_reset board_software_reset
* @defgroup board_software_reset Board software reset
* @ingroup sys

Should fix the not showing up documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports Area: build system Area: Build system Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants