Skip to content

drivers/netdev_ieee802154_submac: port to netdev_new_api #20992

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
Merged
merged 3 commits into from
May 12, 2025
Merged

drivers/netdev_ieee802154_submac: port to netdev_new_api #20992

merged 3 commits into from
May 12, 2025

Conversation

benpicco
Copy link
Contributor

Contribution description

We only have NETDEV_EVENT_TX_COMPLETE now, store error codes in device struct.

Testing procedure

only lightly tested

Issues/PRs references

@benpicco benpicco requested a review from jia200x November 14, 2024 16:01
@github-actions github-actions bot added Area: drivers Area: Device drivers Area: sys Area: System labels Nov 14, 2024
@benpicco benpicco requested a review from maribu November 14, 2024 16:02
@maribu
Copy link
Member

maribu commented Nov 19, 2024

@jia200x: Do you also want to take a look?

@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 Nov 21, 2024
@riot-ci
Copy link

riot-ci commented Nov 21, 2024 •
edited
Loading

Murdock results

✔️ PASSED

ca35c7f drivers/netdev: fix documentation of netdev_new_api

Success Failures Total Runtime
10320 0 10320 13m:47s

Artifacts

@benpicco benpicco added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Dec 22, 2024
@benpicco benpicco force-pushed the netdev_ieee802154_submac-new_api branch from 6db1ff9 to ca35c7f Compare December 23, 2024 21:00
@github-actions github-actions bot added Area: network Area: Networking Area: pkg Area: External package ports labels Dec 23, 2024
@benpicco benpicco added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Apr 24, 2025
@benpicco benpicco requested a review from fabian18 April 25, 2025 16:12
@benpicco benpicco added this pull request to the merge queue May 12, 2025
@crasbe crasbe added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label May 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 12, 2025
@benpicco benpicco added this pull request to the merge queue May 12, 2025
Merged via the queue into RIOT-OS:master with commit 8a6bb51 May 12, 2025
28 checks passed
@gschorcht gschorcht mentioned this pull request May 13, 2025
Merged
7 tasks
@benpicco benpicco deleted the netdev_ieee802154_submac-new_api branch May 13, 2025 09:13
@Teufelchen1 Teufelchen1 added this to the Release 2025.07 milestone Jul 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maribu maribu maribu approved these changes

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

@fabian18 fabian18 Awaiting requested review from fabian18

Assignees
No one assigned
Labels
Area: drivers Area: Device drivers Area: network Area: Networking Area: pkg Area: External package ports Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
Release 2025.07
Development

Successfully merging this pull request may close these issues.
5 participants
@benpicco @maribu @riot-ci @crasbe @Teufelchen1