Skip to content

Conversation

@evading
Copy link
Contributor

@evading evading commented Apr 15, 2025

To help when diagnosing the reason for a failed transmission, the TFQF, Tx FIFO/Queue Full, status is exposed through the DynAux trait.

Thank you!

Thank you for your contribution.
Please make sure that your submission includes the following:

Must

  • The code compiles without errors or warnings.
  • All tests pass and in the best case you also added new tests.
  • cargo +stable fmt was run.
  • (Not related to this change, see other PR) cargo +stable clippy yields no warnings.
  • Your changes were added to the CHANGELOG.md in the proper section.
  • You add a description of your work to this PR.
  • You added proper docs (in code, rustdoc and README.md) for your
    newly added features and code.

@evading evading force-pushed the work/evading/tx-buffer-full branch from c39cfbb to a5c7c4f Compare April 15, 2025 13:02
@evading evading marked this pull request as ready for review April 15, 2025 13:02
@evading evading requested a review from a team as a code owner April 15, 2025 13:02
Copy link
Collaborator

@epontan epontan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a tiny comment.

To help when diagnosing the reason for a failed transmission, the TFQF,
Tx FIFO/Queue Full, status is exposed through the DynAux trait.
@evading evading force-pushed the work/evading/tx-buffer-full branch from a5c7c4f to 1fbd8d7 Compare April 16, 2025 05:07
Copy link
Collaborator

@epontan epontan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@evading
Copy link
Contributor Author

evading commented Apr 23, 2025

@Ironedde would you mind taking a look at this?

@Ironedde
Copy link
Contributor

Absolutely! LGTM 👍
I'll merge this in and prepare a release.

@Ironedde Ironedde merged commit 41259f8 into GrepitAB:master Apr 23, 2025
5 checks passed
Ironedde added a commit to Ironedde/mcan-clone that referenced this pull request Apr 23, 2025
- Add method to query the Tx FIFO/Queue full status (GrepitAB#56)
@Ironedde Ironedde mentioned this pull request Apr 23, 2025
Ironedde added a commit that referenced this pull request Apr 23, 2025
### Added
- Add method to query the Tx FIFO/Queue full status (#56)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants