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

Imix app: dont print error for enoack #85

Merged
merged 1 commit into from Apr 29, 2020
Merged

Conversation

hudson-ayers
Copy link
Contributor

Currently, the Imix test app will print Error sending packet -13 if a receiving device is not also active nearby and acking packets sent by the sender. This is a result of the 15.4 stack now correctly returning ENOACK when packets are transmitted successfully but not Acked (tock/tock#758).

This PR fixes the Imix app to not indicate an error if TOCK_ENOACK is returned when sending a 15.4 packet.

It also removes the "kernel" comment to reflect that LED(0) is no longer the led labeled "kernel" as that LED is no longer accessible from userspace.

Copy link
Member

@ppannuto ppannuto left a comment

Choose a reason for hiding this comment

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

bors r+

@bors
Copy link
Contributor

bors bot commented Apr 29, 2020

Build succeeded:

@bors bors bot merged commit c07b7bd into tock:master Apr 29, 2020
tyler-potyondy pushed a commit to tyler-potyondy/libtock-c that referenced this pull request Mar 13, 2024
85: Imix app: dont print error for enoack r=ppannuto a=hudson-ayers

Currently, the Imix test app will print `Error sending packet -13` if a receiving device is not also active nearby and acking packets sent by the sender. This is a result of the 15.4 stack now correctly returning ENOACK when packets are transmitted successfully but not Acked (tock/tock#758). 

This PR fixes the Imix app to not indicate an error if TOCK_ENOACK is returned when sending a 15.4 packet.

It also removes the "kernel" comment to reflect that LED(0) is no longer the led labeled "kernel" as that LED is no longer accessible from userspace.

Co-authored-by: Hudson Ayers <hayers@stanford.edu>
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.

None yet

2 participants