Re: [PATCH 1/6] canfd: add new data structures and constants

[Marc Kleine-Budde <mkl@pengutronix.de>]

[-- Attachment #1: Type: text/plain, Size: 1302 bytes --]

On 06/18/2012 10:03 PM, Oliver Hartkopp wrote:
>>> Err..this is a userspace header, I just tested it, headers_install
>>> doesn't remove it. And userspace fails to compile with this. For now
>>> stick to __attribute__((aligned(8))).
>>
>> BTW: and it fails to compile due to a __kernel_sa_family_t which isn't
>> defined on my ubuntu with 2.6.32er headers
> 
> 
> __kernel_sa_family_t is defined in include/linux/socket.h but with
> 
> #ifdef __KERNEL__
> 
> around it.
> 
> typedef __kernel_sa_family_t    sa_family_t;
> 
> What do you compile?

canutils, I used linux/can.h generated by install_headers.

> If you compile a kernel -> it is defined.
> If you compile an application -> it is not needed.

You need recent kernel headers on your linux box to work. On the
compiler machine in the office we have:

$ grep __kernel_sa_family_t /usr/include/linux/socket.h
typedef unsigned short __kernel_sa_family_t;
        __kernel_sa_family_t    ss_family;              /* address family */

Marc
-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 262 bytes --]