Re: [PATCH 1/1] Big endian swap for Marvell 6440 SAS driver

2011-06-14 Thread Cédric Cano
Hi Ben, You're alright! I hadn't thought to use le32_to_cpu. Do you want I make a new patch? Cedric Le 12/06/2011 00:45, Benjamin Herrenschmidt a écrit : On Fri, 2011-06-10 at 15:56 +0200, Cédric Cano wrote: Hi, Marvell SAS driver doesn't work on powerpc architecture due to big endian swap.

Re: [PATCH 1/1] Big endian swap for Marvell 6440 SAS driver

2011-06-14 Thread Benjamin Herrenschmidt
On Tue, 2011-06-14 at 08:42 +0200, Cédric Cano wrote: Hi Ben, You're alright! I hadn't thought to use le32_to_cpu. Do you want I make a new patch? Well, I'm not the maintainer of the subsystem or of the driver, so I'm not going to be the one accepting/merging it :-) I'm just reviewing the

Re: [PATCH 1/1] Big endian swap for Marvell 6440 SAS driver

2011-06-11 Thread Benjamin Herrenschmidt
On Fri, 2011-06-10 at 15:56 +0200, Cédric Cano wrote: Hi, Marvell SAS driver doesn't work on powerpc architecture due to big endian swap. I've tested it with the MV6440 chip. Here you can find the patch for Linux 2.6.39.1. Cedric Cano Signed-off-by: Cedric Cano

[PATCH 1/1] Big endian swap for Marvell 6440 SAS driver

2011-06-10 Thread Cédric Cano
Hi, Marvell SAS driver doesn't work on powerpc architecture due to big endian swap. I've tested it with the MV6440 chip. Here you can find the patch for Linux 2.6.39.1. Cedric Cano Signed-off-by: Cedric Cano cc...@interfaceconcept.com --- --- drivers/scsi/mvsas/mv_sas.c2011-06-10