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.

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

pkg/lwip: make use of confirm send #18427

Merged
merged 7 commits into from
May 6, 2024
Merged

Conversation

maribu
Copy link
Member

@maribu maribu commented Aug 9, 2022

Contribution description

This prepares the lwIP adaption layer to work with netdevs that provide netdev_driver_t::confirm_send(), allowing to write event based non-blocking netdevs.

Testing procedure

This PR does not update any drivers to actually provide netdev_driver_t::confirm_send().

#18428 ports the STM32 Ethernet driver to the new API, so that this PR can be tested there.

Issues/PRs references

Same as #18139 but for lwIP

depends on and includes:

@github-actions github-actions bot added Area: build system Area: Build system Area: cpu Area: CPU/MCU ports Area: drivers Area: Device drivers Area: LoRa Area: LoRa radio support Area: network Area: Networking Area: pkg Area: External package ports Area: sys Area: System Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: ESP Platform: This PR/issue effects ESP-based platforms labels Aug 9, 2022
@maribu maribu added the State: waiting for other PR State: The PR requires another PR to be merged first label Aug 9, 2022
@benpicco benpicco requested review from yarrick and removed request for gschorcht August 9, 2022 16:14
@maribu maribu force-pushed the lwip/confirm_send branch from cff692e to 22f9a81 Compare August 22, 2022 09:03
@github-actions github-actions bot removed the Area: LoRa Area: LoRa radio support label Aug 22, 2022
@github-actions github-actions bot added Area: drivers Area: Device drivers Platform: ESP Platform: This PR/issue effects ESP-based platforms Area: cpu Area: CPU/MCU ports labels Apr 26, 2024
@riot-ci
Copy link

riot-ci commented Apr 26, 2024

Murdock results

✔️ PASSED

28f412f pkg/lwip: add support for netdev_driver_t::confirm_send

Success Failures Total Runtime
10066 0 10066 12m:37s

Artifacts

@maribu maribu force-pushed the lwip/confirm_send branch from 1aaccef to 5d389a5 Compare April 27, 2024 08:20
maribu added 4 commits April 30, 2024 08:41
The test netdev implements the legacy API, so we need support for that
from the upper layer.
Pull in lwip_netdev for netdev integration automatically if there is
any netdev to integrate, otherwise don't pull in the module.

As a result, applications no longer need to select that module by hand.
@maribu maribu force-pushed the lwip/confirm_send branch from 5d389a5 to 6e4035d Compare April 30, 2024 06:41
@maribu maribu removed the State: waiting for other PR State: The PR requires another PR to be merged first label Apr 30, 2024
@maribu maribu marked this pull request as draft April 30, 2024 07:12
@maribu
Copy link
Member Author

maribu commented Apr 30, 2024

The lwIP integration has changed a bit since this PR was opened. I will have to adapt a bit.

@maribu maribu force-pushed the lwip/confirm_send branch from 6e4035d to 962fca5 Compare April 30, 2024 07:49
@maribu maribu marked this pull request as ready for review April 30, 2024 07:54
@benpicco
Copy link
Contributor

Murdock found a bug

@maribu maribu force-pushed the lwip/confirm_send branch from 962fca5 to 28f412f Compare April 30, 2024 17:14
@maribu
Copy link
Member Author

maribu commented May 3, 2024

Murdock found a bug

Thanks, fixed :)

@benpicco benpicco enabled auto-merge May 6, 2024 11:04
@benpicco benpicco added this pull request to the merge queue May 6, 2024
Merged via the queue into RIOT-OS:master with commit 3791db3 May 6, 2024
26 checks passed
@maribu maribu deleted the lwip/confirm_send branch May 6, 2024 17:32
@maribu
Copy link
Member Author

maribu commented May 6, 2024

Thx a lot :)

@mguetschow mguetschow added this to the Release 2024.07 milestone Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: cpu Area: CPU/MCU ports Area: drivers Area: Device drivers Area: examples Area: Example Applications Area: network Area: Networking Area: pkg Area: External package ports Area: sys Area: System Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: ESP Platform: This PR/issue effects ESP-based platforms Platform: native Platform: This PR/issue effects the native platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants