[PATCH] [SCSI] mvsas: Add support for Marvell's 88SE9480 chip.

[PATCH] [SCSI] mvsas: Add support for Marvell's 88SE9480 chip.

[Alexander Kurtz]

This patch adds the missing pci id for Marvell's 88SE9480 SAS/SATA HBA
which is also sold with a vendor id of 0x1B4B instead of 0x11AB.

Signed-off-by: Alexander Kurtz <kurtz.alex@googlemail.com>
---
 drivers/scsi/mvsas/mv_init.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/drivers/scsi/mvsas/mv_init.c b/drivers/scsi/mvsas/mv_init.c
index 4e9af66..bd5b8e1 100644
--- a/drivers/scsi/mvsas/mv_init.c
+++ b/drivers/scsi/mvsas/mv_init.c
@@ -723,6 +723,15 @@ static struct pci_device_id __devinitdata mvs_pci_table[] = {
 		.class_mask	= 0,
 		.driver_data	= chip_9485,
 	},
+	{
+		.vendor		= 0x1b4b,
+		.device		= 0x9480,
+		.subvendor	= PCI_ANY_ID,
+		.subdevice	= PCI_ANY_ID,
+		.class		= 0,
+		.class_mask	= 0,
+		.driver_data	= chip_9480,
+	},
 
 	{ }	/* terminate list */
 };
-- 
1.7.2.5

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] [SCSI] mvsas: Add support for Marvell's 88SE9480 chip.

[Alexander Kurtz]

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

Hi,

although the patch made my system recognize the HBA, there were several
severe issues when actually using it: I experienced random reboots
during POST and several freezes during heavy I/O. Additionally the
system always failed to wake up from S3 when the HBA was plugged in.

I'm therefore going to return the card in a few days. You may still
consider merging the patch since it made my system at least recognize
the HBA after all.

Best regards

Alexander Kurtz

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

RE: [PATCH] [SCSI] mvsas: Add support for Marvell's 88SE9480 chip.

[Xiangliang Yu]

Hi, Alexander

> -----Original Message-----
> From: Alexander Kurtz [mailto:kurtz.alex@googlemail.com]
> Sent: 2011年8月13日 18:12
> To: linux-scsi@vger.kernel.org
> Cc: Xiangliang Yu; James Bottomley
> Subject: [PATCH] [SCSI] mvsas: Add support for Marvell's 88SE9480 chip.
> 
> This patch adds the missing pci id for Marvell's 88SE9480 SAS/SATA HBA
> which is also sold with a vendor id of 0x1B4B instead of 0x11AB.
> 
> Signed-off-by: Alexander Kurtz <kurtz.alex@googlemail.com>
> ---
>  drivers/scsi/mvsas/mv_init.c |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/scsi/mvsas/mv_init.c b/drivers/scsi/mvsas/mv_init.c
> index 4e9af66..bd5b8e1 100644
> --- a/drivers/scsi/mvsas/mv_init.c
> +++ b/drivers/scsi/mvsas/mv_init.c
> @@ -723,6 +723,15 @@ static struct pci_device_id __devinitdata mvs_pci_table[]
> = {
>  		.class_mask	= 0,
>  		.driver_data	= chip_9485,
>  	},
> +	{
> +		.vendor		= 0x1b4b,
> +		.device		= 0x9480,
> +		.subvendor	= PCI_ANY_ID,
> +		.subdevice	= PCI_ANY_ID,

Subdevice should be is 0x9480
		.subdevice = 0x9480,

RE: [PATCH] [SCSI] mvsas: Add support for Marvell's 88SE9480 chip.

[Alexander Kurtz]

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

Hi,

On Mon, 2011-08-15 at 19:44 -0700, Xiangliang Yu wrote:
> Subdevice should be is 0x9480
> 		.subdevice = 0x9480,

As announced earlier[1], I have returned the device meanwhile because of
some serious malfunctions, so I can't test this change. If you are still
interested, you might want to read my Debian bug report[2] which
contains some further information.

Best regards

Alexander Kurtz

[1] http://permalink.gmane.org/gmane.linux.scsi/69578
[2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=637591

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]