Skip to content

topic-counter: nrfx_*: counter_set_top_value inconsistent behavior #12800

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

Closed
pabigot opened this issue Jan 28, 2019 · 3 comments
Closed

topic-counter: nrfx_*: counter_set_top_value inconsistent behavior #12800

pabigot opened this issue Jan 28, 2019 · 3 comments
Assignees
Labels
area: Counter bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@pabigot
Copy link
Collaborator

pabigot commented Jan 28, 2019

The documentation does not indicate whether counter_set_alarm() which maps to counter_set_top_value() should be a one-shot or repeating capability. The behavior for Nordic is inconsistent:

For RTC:

  • If counter_set_top_value is configured with the maximum top value the callback is invoked once, but not rescheduled, though the callback remains associated with the channel.
  • If counter_set_top_value is configured with a top value less than the maximum the RTC clock is cleared and the timer rescheduled, resulting in a repeated callback at the requested interval.

For TIMER:

  • If counter_set_top_value is the callback is repeated at the requested interval without being rescheduled.
@nordic-krch
Copy link
Collaborator

behavior for RTC default top value was not intended. Basically, calling nrfx_rtc_cc_set should be outside if statement.

@galak galak added the priority: medium Medium impact/importance bug label Jan 29, 2019
@nashif
Copy link
Member

nashif commented Feb 11, 2019

@pabigot @nordic-krch is this still an issue?

@pabigot
Copy link
Collaborator Author

pabigot commented Feb 11, 2019

AFAIK it was closed by #12815.

@pabigot pabigot closed this as completed Feb 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Counter bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

No branches or pull requests

4 participants