Skip to content

[nrf fromlist] drivers: timer: nrf_grtc_timer: add last_count initial… #2949

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

Merged

Conversation

adamkondraciuk
Copy link
Contributor

…ization

The GRTC counter is not cleared at startup, therefore the last_count variable needs to be initialized accordingly. This change:

  • Prevents overflow of the sys_clock_announce() int32_t parameter
  • Ensures the correct uptime value, which should be reset during initialization

Upstream PR #: 91432

@adamkondraciuk adamkondraciuk force-pushed the NRFX-7834-update-last-count-at-grtc-init branch 5 times, most recently from 729b57c to 31fef37 Compare June 12, 2025 11:00
@adamkondraciuk adamkondraciuk force-pushed the NRFX-7834-update-last-count-at-grtc-init branch 3 times, most recently from 230afeb to c9b668d Compare June 17, 2025 11:41
…ization

The GRTC counter is not cleared at startup, therefore the
`last_count` variable needs to be initialized accordingly.
This change:
- Prevents overflow of the `sys_clock_announce()` int32_t parameter
- Ensures the correct uptime value, which should be reset during
  initialization

Upstream PR #: 91432

Signed-off-by: Adam Kondraciuk <adam.kondraciuk@nordicsemi.no>
…h uptime changes

Update GRTC tests to reflect uptime reset at startup.

Upstream PR #: 91432

Signed-off-by: Adam Kondraciuk <adam.kondraciuk@nordicsemi.no>
@adamkondraciuk adamkondraciuk force-pushed the NRFX-7834-update-last-count-at-grtc-init branch from c9b668d to e20c0a8 Compare June 17, 2025 15:19
Copy link

@rlubos rlubos merged commit 442eaf6 into nrfconnect:main Jun 18, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants