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

cpu/nrf5x/periph/wdt: enable support for nRF53/nRF9160 #19789

Merged
merged 3 commits into from
Jul 6, 2023

Conversation

dylad
Copy link
Member

@dylad dylad commented Jul 3, 2023

Contribution description

This PR enables support for the watchdog driver on nRF53.
This MCU has two watchdog peripherals, for now, it only uses the first one.
The use of the second watchdog can be add in a followup PR later.

Testing procedure

Flash and play with tests/periph/wdt application.

Issues/PRs references

None.

@dylad dylad added the State: waiting for other PR State: The PR requires another PR to be merged first label Jul 3, 2023
@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: doc Area: Documentation Area: boards Area: Board ports Area: cpu Area: CPU/MCU ports Area: Kconfig Area: Kconfig integration labels Jul 3, 2023
dylad added 3 commits July 5, 2023 09:48
Signed-off-by: Dylan Laduranty <dylan.laduranty@mesotic.com>
Signed-off-by: Dylan Laduranty <dylan.laduranty@mesotic.com>
Signed-off-by: Dylan Laduranty <dylan.laduranty@mesotic.com>
@dylad dylad force-pushed the cpu/nrf53/add_wdt_support branch from abcd7db to 1df1279 Compare July 5, 2023 07:51
@github-actions github-actions bot removed Area: doc Area: Documentation Area: boards Area: Board ports labels Jul 5, 2023
@dylad dylad added Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: boards Area: Board ports and removed State: waiting for other PR State: The PR requires another PR to be merged first labels Jul 5, 2023
@dylad dylad changed the title cpu/nrf5x/periph/wdt: enable support for nRF53 cpu/nrf5x/periph/wdt: enable support for nRF53/nRF9160 Jul 5, 2023
@dylad
Copy link
Member Author

dylad commented Jul 5, 2023

Rebased, this PR no longer waits for others.
I also enable watchdog support for nRF9160 while at it... I don't have a nRF9160-based board to test but it works fine on nRF5340DK-APP.

@riot-ci
Copy link

riot-ci commented Jul 5, 2023

Murdock results

✔️ PASSED

1df1279 cpu/nrf{53,9160}: enable periph_wdt in Kconfig

Success Failures Total Runtime
6931 0 6931 11m:13s

Artifacts

@benpicco
Copy link
Contributor

benpicco commented Jul 6, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Jul 6, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit a345e00 into RIOT-OS:master Jul 6, 2023
@dylad dylad deleted the cpu/nrf53/add_wdt_support branch July 6, 2023 18:00
@benpicco benpicco added this to the Release 2023.07 milestone Aug 2, 2023
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: cpu Area: CPU/MCU ports Area: doc Area: Documentation Area: Kconfig Area: Kconfig integration CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants