#include "compat.h" obsolete? about can-isotp HOT 7 CLOSED

This compat.h is only pulled on Linux Kernels < 2.6.25
https://github.com/hartkopp/can-isotp/blob/master/net/can/isotp.c#L80
So it shouldn't affect your build process - or for which kernel version do you plan to build the code??

from can-isotp.

I am trying to include the module in a buildroot environment with Linux 2.6.31 which ends up failing because of a few missing defines in supposedly linux/can.h. This is quite the sudden deep dive into linux-stuff, so I was just looking around noticing that include without finding the actual file for it in this repo.

from can-isotp.

Yes. The latest version of ISO-TP supports also CAN FD which was introduced in Linux 3.6. So the implementation requires these definition from can.h even if you can't use CAN FD with your 2.6.31 kernel.
I backported the entire CAN stuff for a 2.6.13 kernel a long time ago :-D
It should work when you update some of theses include bits like can.h.

The CAN FD support has been added in the (really obsolete) can-isotp-modules repository here:
hartkopp/can-isotp-modules@b990ca6

So if you can't get it to work, you might also take a look at that ancient code (but not really recommended).

from can-isotp.

So... I managed to include a patched can.h-header file in my buildroot build, but I still get an error about a can_ioctl function. Any hints on that? :)

from can-isotp.

Can you check the issue with this updated check for your kernel version?

from can-isotp.

Yes, seems like it wasn't even needed... works like a charm now (tested with the isotp can-util tools). Thanks a lot!

from can-isotp.

Have fun!

from can-isotp.