Skip to content

Conversation

@GrixaYrev
Copy link
Contributor

Before commit 4f2682b
("usb: cdc_acm: avoid spurious interrupt on configured or resume events")
spurious rx-interrupt provided start to read shell tx ring buffer by
CDC ACM UART. Now shell log backend not transmit data, until CDC ACM UART
received data from host.

This PR added invoking interrupt callback if tx interrupt is enable,
when CDC ACM class go to configured state, and flag tx_ready go to true.

Signed-off-by: Grixa Yrev [email protected]

@GrixaYrev GrixaYrev requested a review from jfischer-no as a code owner August 25, 2021 10:29
@jfischer-no jfischer-no added area: USB Universal Serial Bus bug The issue is a bug, or the PR is fixing a bug labels Aug 25, 2021
@jfischer-no jfischer-no requested review from anangl and eobalski August 25, 2021 11:36
@jfischer-no jfischer-no added this to the v2.7.0 milestone Aug 25, 2021
@GrixaYrev
Copy link
Contributor Author

Fixed lines length in commit message.

Before commit 4f2682b
("usb: cdc_acm: avoid spurious interrupt on configured or resume
events") spurious rx-interrupt provided start to read shell tx ring
buffer by CDC ACM UART. Now shell log backend not transmit data,
until CDC ACM UART received data from host.

This PR added invoking interrupt callback if tx interrupt is enable,
when CDC ACM class go to configured state, and flag tx_ready go to true.

Signed-off-by: Grixa Yrev <[email protected]>
@nashif nashif merged commit 6a8abcb into zephyrproject-rtos:main Aug 26, 2021
@GrixaYrev GrixaYrev deleted the pr-cdcacm-tx_irq branch August 29, 2021 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: USB Universal Serial Bus bug The issue is a bug, or the PR is fixing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants