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

[Oliver Hartkopp <socketcan@hartkopp.net>]

On 18.06.2012 21:13, Marc Kleine-Budde wrote:

> On 06/18/2012 07:39 PM, Oliver Hartkopp wrote:
>> - add new struct canfd_frame


>>  struct can_frame {
>>  	canid_t can_id;  /* 32 bit CAN_ID + EFF/RTR/ERR flags */
>> -	__u8    can_dlc; /* data length code: 0 .. 8 */
>> +	__u8    can_dlc; /* frame payload length in byte (0 .. 8) */
>>  	__u8    data[8] __attribute__((aligned(8)));
> 
> checkpatch complains:
> 
> WARNING: __aligned(size) is preferred over __attribute__((aligned(size)))
> #84: FILE: include/linux/can.h:102:
> +       __u8    data[64] __attribute__((aligned(8)));
> 
> total: 0 errors, 1 warnings, 97 lines checked
> 


Hm, yes. I wanted to have it the same way as in struct can_frame.

I wonder if "__aligened(size)" is also compatible for backports - or
alternative if we should also modify the original can_frame:

>> - 	__u8    data[8] __attribute__((aligned(8)));
>> + 	__u8    data[8] __aligned(8);

???

Regards,
Oliver