On 09/03/2012 07:13 PM, Benoît wrote:
[...]

>> Hello Benoit,
>>
>> can you please test this patch, if it still fixes your problem. In
>> order to
>> integrate this patch into the kernel, I need you Signed-off-by [1]
>> with you
>> real name.
> 
> I just carefully ran some before/after tests with your updated patch and
> I confirm the problem is still fixed with it.

Thanks.

> 
> As for my "Signed-off-by", here it goes:
> 
> Signed-off-by: Benoît Locher <Benoit.Locher@skf.com>

I'll change the author of the patch to that address then.

> By the way, about using spi_write()instead of mcp251x_spi_trans(): I
> innocently thought it would make less function call nesting,
> trying to improve response time of the driver particularly on my
> Raspberry Pi.

Ahh, I see. but the mcp251x_spi_trans() function takes does the setup
thats needed for DMA.

regards,
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   |