search

go-daq / canbus

CAN-bus interface
BSD 3-Clause "New" or "Revised" License
31 stars 13 forks source link

Support for standard RTR frame #8

Open Theiremi opened 8 months ago

Theiremi commented 8 months ago

Hi, First of all, thank you for the great work you did with this library !

After looking at the source code, it looks like all RTR frames are treated as extended frame, even if standard RTR frames can occurs. I also found this comment, which seems to address the same issue.

sbinet commented 8 months ago

Thanks for your interest in this library.

It's been a long time since I dwelved in canbus matters...

Do you have a reproducer and/or a C program that shows the correct expected behavior ?

Theiremi commented 8 months ago

The only example I have in mind is this CAN library for Arduino (a bit off-topic). The ID is formatted to match the way it is returned by SocketCAN, and then the user adapt his code to retrieve the frame metadata from the last three bits. Not sure it's really useful in our case tho