ARMmbed / ARMmbed/mbed-hal

us ticker read - should not return timestamp_t type?

Open
#77 1 comment 0 reactions 0 assignees View on GitHub
mirrored
Dominant language
C
Stars
15
Forks
26
PR merge metrics
No merged PRs in 30d

Description

As the title says, us_ticker_read() should probably return timestamp_t return value.

```
/** Read the current counter
*
* @return The current timer's counter value in microseconds
*/
uint32_t us_ticker_read(void);

/** Set interrupt for specified timestamp
*
* @param timestamp The time in microseconds to be set
*/
void us_ticker_set_interrupt(timestamp_t timestamp);
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate the declarations and implementations of us_ticker_read() and us_ticker_set_interrupt(), then compare their timestamp types and documentation. Check the surrounding mbed HAL API for compatibility expectations; the work is done when the read API consistently uses the intended timestamp type and related checks remain valid.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
embedded-iot
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.