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

x/sys/unix: add CAN error constants from linux/can/error.h #41067

Closed
skladd opened this issue Aug 27, 2020 · 2 comments
Closed

x/sys/unix: add CAN error constants from linux/can/error.h #41067

skladd opened this issue Aug 27, 2020 · 2 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone
Unreleased

Comments

@skladd
Copy link

skladd commented Aug 27, 2020

Currently these error constants like CAN_ERR_BUSOFF are not included in zerrors_linux.go.
Adding #include <linux/can/error.h> to mkerrors.sh would add 58 constants allowing interpretation of SocketCAN error frames.
@gopherbot gopherbot added this to the Unreleased milestone Aug 27, 2020
skladd added a commit to skladd/sys that referenced this issue Aug 27, 2020
@skladd
unix: add SocketCAN error constants
880c970 
Fixes golang/go#41067
@skladd skladd mentioned this issue Aug 27, 2020
Closed
@gopherbot
Copy link

Change https://golang.org/cl/251037 mentions this issue: unix: add CAN error constants

skladd added a commit to skladd/sys that referenced this issue Aug 27, 2020
@skladd
unix: add SocketCAN error constants
039cf2a 
Fixes golang/go#41067
@cagedmantis cagedmantis added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Aug 27, 2020
@cagedmantis
Copy link
Contributor

/cc @ianlancetaylor @bradfitz @tklauser

skladd added a commit to skladd/sys that referenced this issue Aug 28, 2020
@skladd
unix: add constants for raw CAN sockets
6135a51 
Fixes golang/go#41067
@golang golang locked and limited conversation to collaborators Aug 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Milestone
Unreleased
Development

Successfully merging a pull request may close this issue.

unix: add CAN error constants skladd/sys
3 participants
@cagedmantis @gopherbot @skladd