EatonCorp/kvaser — comment on commit 54066fd2829b06a86f6ebc6c592a79c98f4e13c6
File: leaf/Makefile line 9
By jfoell:

Mark,
Yes I forgot the details about $(KERNELRELEASE). I wrote my Makefile using
the latest examples from the kernel doc. Please read this file, you'll see
an example using "ifneq ($(KERNELRELEASE),)".

http://www.kernel.org/doc/Documentation/kbuild/modules.txt

Getting the Makefile to work correctly was a trial and error thing.

How are you using kvaser products? What linux distro/version are you
using? Are you still confused?

Jake

On Mon, Oct 17, 2011 at 7:50 PM, Jake Foell wrote:

> Mark,
> here is the text from the README file.
>
> The linux developer kit from www.kvaser.com does not work well
> with the latest linux kernels. Also, at Eaton, we are only using
> the Kvaser Leaf/LeafLight, so controlling the code for all the
>
> other interfaces is unnecessary. This repo contains the 'canlib'
> library and 'kvaser leaf' drivers from www.kvaser.com and have been
> modified to build/install on the latest linux kernels (including
> kernel 3.0).
>
> use the linux standard commands to build and install
>
> $ make
> ... then as root ...
> # make install
>
>
>
> Basically,
> I couldn't build the leaf driver with my 3.0 kernel. I also had problems
> with something in the usb drivers. I forget the details. The kvaser.comsolution is also very flexible, too flexible, littered (#ifdef LINUX)
> conditionals. I didn't want to carry all that bloat around so I nuked most
> of the stuff I don't use. (especially the windows stuff). I don't know if
> you've noticed but when you get regular kernel upgrades (as part of apt or
> whatever your distro uses), you'll have to rebuild the driver for the new
> kernel. That is another reason to keep it quick and dirty. (my opinion).
> I haven't looked into your changes, but I wanted to give you my rationale as
> soon as I could.
>
> Jake
>
> On Mon, Oct 17, 2011 at 5:45 PM, Mark Catley <
> reply@reply.github.com>wrote:
>
>> What are you doing with KERNELRELEASE here? I had to modify this Makefile
>> to get this building under DKMS.
>>
>> --
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/EatonCorp/kvaser/commit/54066fd2829b06a86f6ebc6c592a79c98f4e13c6#commitcomment-657111
>>
>
>