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

watchdog: adding driver for the nrf52 family #6047

Conversation

GiulianoFranchetto
Copy link
Contributor

Origin: Original

This driver is a direct port from the nordic SDK, rewritten to be
Zephyr-compatible.
This driver has been tested on a nrf52840 chip.
Should work on a nrf52832,but it should be tested to be sure.

Signed-off-by: Giuliano Franchetto [email protected]

@codecov-io
Copy link

codecov-io commented Feb 8, 2018

Codecov Report

Merging #6047 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6047   +/-   ##
=======================================
  Coverage   52.63%   52.63%           
=======================================
  Files         410      410           
  Lines       40048    40048           
  Branches     7781     7781           
=======================================
  Hits        21078    21078           
  Misses      15768    15768           
  Partials     3202     3202

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa8850b...e455021. Read the comment docs.

Origin: Original

This driver is a direct port from the nordic SDK, rewritten to be
Zephyr-compatible.
This driver has been tested on a nrf52840 chip.
Should work on a nrf52832,but it should be tested to be sure.

Signed-off-by: Giuliano Franchetto <[email protected]>
@anangl
Copy link
Member

anangl commented Feb 13, 2018

Since a new watchdog API (#1260) will be introduced soon (hopefully), I think we should not add an implementation for the old one now.

@SebastianBoe SebastianBoe removed their request for review February 13, 2018 13:35
@galak galak added the platform: nRF Nordic nRFx label Apr 12, 2018
@carlescufi
Copy link
Member

Closing this one since the nrfx driver has been merged

@carlescufi carlescufi closed this Jul 6, 2018
@GiulianoFranchetto GiulianoFranchetto deleted the nrf5_watchdog branch April 1, 2019 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: nRF Nordic nRFx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants