Skip to content

sys/usb/cdc_acm: don't generate flush event when disconnected#21914

Merged
Teufelchen1 merged 1 commit intoRIOT-OS:masterfrom
benpicco:usbus_cdc_acm_flush-disco
Nov 25, 2025
Merged

sys/usb/cdc_acm: don't generate flush event when disconnected#21914
Teufelchen1 merged 1 commit intoRIOT-OS:masterfrom
benpicco:usbus_cdc_acm_flush-disco

Conversation

@benpicco
Copy link
Copy Markdown
Contributor

Contribution description

The event handler will check the connected state too, but when we are not connected we don't have to generate the event in the first place.

Testing procedure

Issues/PRs references

follow-up to #21890

@github-actions github-actions bot added Area: USB Area: Universal Serial Bus Area: sys Area: System labels Nov 24, 2025
@crasbe crasbe added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 24, 2025
@riot-ci
Copy link
Copy Markdown

riot-ci commented Nov 24, 2025

Murdock results

✔️ PASSED

827dc31 sys/usb/cdc_acm: don't generate flush event when disconnected

Success Failures Total Runtime
10932 0 10932 10m:01s

Artifacts

Copy link
Copy Markdown
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.

lgtm
Did not test it but low impact & easy to bisect.

@Teufelchen1 Teufelchen1 added this pull request to the merge queue Nov 25, 2025
Merged via the queue into RIOT-OS:master with commit 2aafabd Nov 25, 2025
28 checks passed
@benpicco benpicco deleted the usbus_cdc_acm_flush-disco branch November 25, 2025 13:55
@leandrolanzieri leandrolanzieri added this to the Release 2026.01 milestone Jan 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: sys Area: System Area: USB Area: Universal Serial Bus 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

Development

Successfully merging this pull request may close these issues.

5 participants