Package Details: ixxat-socketcan 1.1.148.0-7

Git Clone URL: https://aur.archlinux.org/ixxat-socketcan.git (read-only, click to copy)
Package Base: ixxat-socketcan
Description: SocketCAN driver for IXXAT USB-to-CAN v2
Upstream URL: https://www.ixxat.com/products/products-industrial/pc-interfaces/pc-can-interfaces/socketcan
Licenses: GPL2
Submitter: bms
Maintainer: None
Last Packager: bms
Votes: 0
Popularity: 0.000000
First Submitted: 2018-08-05 23:48 (UTC)
Last Updated: 2021-05-29 11:37 (UTC)

Latest Comments

bms commented on 2021-05-13 13:33 (UTC)

Good to hear then. I've already tracked down the kernel changes that broke this and it seems you got it right. The PCI ones look like more of the same.

The point I was stuck on was your change from a can_get_echo_skb to a can_free_echo_skb. I suspect that either introduces a bug or fixes one, but I don't really know the code that well, I'll need a bit more time to check that. If this change was deliberate on your part and you have further context, please send me an email explaining it.

I would say not to worry with the PR though. I only wanted to point out the source of my patches.

bverhagen commented on 2021-05-13 09:48 (UTC)

Hi @bms, I tested in a rather limited fashion with an ixxat USB-to-CAN v2 (both compact & professional). These appear to work fine with the additional patch.

I'll see whether I can find the time to do a PR to your repo. For your information: I also tested with the latest Socketcan driver release from Ixxat, but that one still seems to have the problems you already patched.

bms commented on 2021-05-11 08:26 (UTC)

Hi bverhagen, thanks for the report and the patch. I don't have access to the hardware to test at the moment and I still want to look into the kernel API changes, but the patch is probably fine. I suppose you tested this?

FYI, I keep track of the patches in a repo for convenience: https://gitlab.com/bms-contrib/ixxat-socketcan

bverhagen commented on 2021-05-10 15:02 (UTC)

In the meantime, some more kernel patches are required. My fixes to build and install the USB-based kernel modules are here: https://gist.github.com/bverhagen/2c1ff8ad071c83a4802858023d801322. Though I limited myself to the USB-based kernel modules, I expect the fixes for the PCI-modules to be similar. I have no experience with the CAN kernel interface, so feedback on the patch is very much appreciated.