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.

Sign up for GitHub

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

Jump to bottom

boards/waspmote-pro: fix ztimer config #20573

Merged
merged 1 commit into from Apr 30, 2024
Merged

boards/waspmote-pro: fix ztimer config #20573

merged 1 commit into from Apr 30, 2024

Conversation

maribu
Copy link
Member

@maribu maribu commented Apr 14, 2024 •
edited

Contribution description

This fixes a typo in the ztimer config.

Testing procedure

Check the spelling indeed matches what is used in the code:

RIOT/sys/ztimer/init.c

Lines 71 to 75 in 3cdc437

#if MODULE_ZTIMER_PERIPH_TIMER
# define ZTIMER_TIMER _ztimer_periph_timer
# define ZTIMER_TIMER_CLK _ztimer_periph_timer.super
# define ZTIMER_TIMER_FREQ CONFIG_ZTIMER_USEC_BASE_FREQ
#endif

One could also test with the board, but it sadly is made of Unobtainium...

Issues/PRs references

None

@maribu maribu added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Apr 14, 2024
@github-actions github-actions bot added the Area: boards Area: Board ports label Apr 14, 2024
@riot-ci
Copy link

riot-ci commented Apr 14, 2024 •
edited

Murdock results

✔️ PASSED

162bb6e boards/waspmote-pro: fix ztimer config

Success Failures Total Runtime
466 0 467 02m:26s

Artifacts

Teufelchen1
Teufelchen1 approved these changes Apr 30, 2024
View reviewed changes
Copy link
Contributor

@Teufelchen1 Teufelchen1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@maribu
Copy link
Member Author

maribu commented Apr 30, 2024

With the Doxygen comments added the CI should also be happy.

@maribu maribu added this pull request to the merge queue Apr 30, 2024
Merged via the queue into RIOT-OS:master with commit a620a3c Apr 30, 2024
25 checks passed
@maribu maribu deleted the boards/waspmote-pro branch April 30, 2024 17:17
@maribu
Copy link
Member Author

maribu commented Apr 30, 2024

Thx :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Teufelchen1 Teufelchen1 Teufelchen1 approved these changes

Assignees
No one assigned
Labels
Area: boards Area: Board ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@maribu @riot-ci @Teufelchen1