-
Notifications
You must be signed in to change notification settings - Fork 636
The gs_usb interface support #905
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
Conversation
87930a0
to
52a2aab
Compare
Codecov Report
@@ Coverage Diff @@
## develop #905 +/- ##
===========================================
- Coverage 70.76% 70.34% -0.43%
===========================================
Files 71 72 +1
Lines 7061 7106 +45
===========================================
+ Hits 4997 4999 +2
- Misses 2064 2107 +43 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks very clean, just minor comments mostly around documentation.
Bit of a shame to not have support for periodic messages or filtering.
@ericevenchick any chance you could provide your thoughts? |
@jxltom is right, CANtact is diverging a bit from gs_usb to add FD support. There's already a gs_usb_fd Linux kernel module for this: https://github.com/linklayer/gs_usb_fd The goal was to keep support with the original gs_usb, but that's not my standard to modify. The gs_usb_fd module supports normal gs_usb devices, and FD devices provide a flag to show they have support. |
Co-authored-by: Brian Thorne <[email protected]>
3402873
to
676dd7c
Compare
Hey @hardbyte thanks for your time for reviewing. I've already updated the PR according to the comments. Just let me know if there are more concerns. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making all those changes this looks good to merge.
The
gs_usb
is the interface for Geschwister Schneider USB/CAN devices and bytewerk.org candleLight USB CAN devices such as candlelight, canable, cantact, etc. Refer to https://github.com/torvalds/linux/blob/master/drivers/net/can/usb/gs_usb.c and https://github.com/candle-usb/candleLight_fwThis PR implements the general driver for
gs_usb
devices based onpyusb
, tested on Windows/Linux/Mac.It looks like one of the
gs_usb
devicecantact
is supported in #853. However, that interface might diverge fromgs_usb
since it is going to add CAN FD support. This PR is more universal forgs_usb
devices. Since this interface is using pyusb as backend, it has better crossplatform compatibility.