Skip to content

sys: drop xtimer implementation #21960

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
Open
benpicco wants to merge 3 commits into
base: master
Choose a base branch
from
Open

sys: drop xtimer implementation #21960

benpicco wants to merge 3 commits into from
+3 −1,409

Conversation

@benpicco
Copy link
Contributor

@benpicco benpicco commented Dec 16, 2025 •
edited
Loading

Contribution description

This removes the long deprecated xtimer implementation.
The xtimer is still provided on as a high-level wrapper on top of ztimer.

Testing procedure

This code should already be unused (except by the xtimer_now32_overflow that explicitly disables ztimer implementation - this is now also removed).

Issues/PRs references

depends on #21958

@github-actions github-actions bot added Area: doc Area: Documentation Area: tests Area: tests and testing framework Area: build system Area: Build system Area: timers Area: timer subsystems Area: sys Area: System labels Dec 16, 2025
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 16, 2025
@riot-ci
Copy link

riot-ci commented Dec 16, 2025

Murdock results

❌ FAILED

3166a15 tests/xtimer_now32_overflow: drop xtimer-only test

Success Failures Total Runtime
6161 0 10419 05m:36s

Artifacts

@crasbe crasbe added the Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation label Dec 16, 2025
@benpicco benpicco mentioned this pull request Dec 16, 2025
Open
@benpicco benpicco added the State: waiting for other PR State: The PR requires another PR to be merged first label Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

@kaspar030 kaspar030 Awaiting requested review from kaspar030 kaspar030 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Area: build system Area: Build system Area: doc Area: Documentation Area: sys Area: System Area: tests Area: tests and testing framework Area: timers Area: timer subsystems CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR State: waiting for other PR State: The PR requires another PR to be merged first Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@benpicco @riot-ci @crasbe