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.

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

testing: Add Serial Error Reporting testing app #2989

Merged
merged 1 commit into from
Mar 24, 2025
Merged

testing: Add Serial Error Reporting testing app #2989

merged 1 commit into from
Mar 24, 2025

Conversation

acassis
Copy link
Contributor

@acassis acassis commented Feb 6, 2025

Summary

This commit adds support to Serial Error Reporting using the ioctl TIOCGICOUNT.

Impact

This testing application is used to display the errors reported by U[S]ART driver.

Testing

STM32F4Discovery

@acassis acassis force-pushed the ser branch 3 times, most recently from 90e0f26 to 7ce3b4a Compare February 6, 2025 23:36
@linguini1
Copy link
Contributor

Should this also have some documentation for users?

cederom
cederom approved these changes Feb 7, 2025
View reviewed changes
Copy link
Contributor

@cederom cederom left a comment

Choose a reason for hiding this comment

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

Thank you @acassis :-) Could you please provide some short log of how it runs.. maybe a documentation too in a free moment? :-)

@acassis
Copy link
Contributor Author

acassis commented Feb 7, 2025 •
edited
Loading

Thank you @acassis :-) Could you please provide some short log of how it runs.. maybe a documentation too in a free moment

I will add a board config with explanation. Since this ioctl() is standard on Linux and POSIX it is very simple to use.

@acassis
testing: Add Serial Error Reporting testing app
1109e5d 
This commit adds support to Serial Error Reporting using the ioctl
TIOCGICOUNT.

This examples was inspired on this sample code:
https://stackoverflow.com/questions/78796301/c-serial-communication-why-does-read-lose-some-data-bytes-at-high-baud-rates/78800245#78800245

Signed-off-by: Alan C. Assis <[email protected]>
@xiaoxiang781216 xiaoxiang781216 merged commit cdc5968 into apache:master Mar 24, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@raiden00pl raiden00pl raiden00pl left review comments

@cederom cederom cederom approved these changes

@lupyuen lupyuen lupyuen approved these changes

@xiaoxiang781216 xiaoxiang781216 xiaoxiang781216 approved these changes

@linguini1 linguini1 linguini1 approved these changes

Assignees
No one assigned
Labels
Area: Testing Size: M
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@acassis @linguini1 @cederom @raiden00pl @lupyuen @xiaoxiang781216