Highpoint RocketRaid 1720 support in sata_mv

Highpoint RocketRaid 1720 support in sata_mv

[Petr Jelen]

Hello,

I have a simple patch for sata_mv.c, which adds support for Highpoint
RocketRaid 1720 adapter (2 SATA2 ports on PCI bus).
The adapter has Marvel 88SX6042-BCZ1 on board.

The patch is against 2.6.25.9 and I have tested it. I could normally
work with my disks (in JBOD mode).
Could you please accept the patch?

diff -urN linux-2.6.25.9/drivers/ata/sata_mv.c
linux-2.6.25.9-new/drivers/ata/sata_mv.c
--- linux-2.6.25.9/drivers/ata/sata_mv.c        2008-06-24
23:09:06.000000000 +0200
+++ linux-2.6.25.9-new/drivers/ata/sata_mv.c    2008-06-28
22:36:22.000000000 +0200
@@ -695,7 +695,8 @@
        { PCI_VDEVICE(MARVELL, 0x5041), chip_504x },
        { PCI_VDEVICE(MARVELL, 0x5080), chip_5080 },
        { PCI_VDEVICE(MARVELL, 0x5081), chip_508x },
-       /* RocketRAID 1740/174x have different identifiers */
+       /* RocketRAID 1720/174x have different identifiers */
+       { PCI_VDEVICE(TTI, 0x1720), chip_6042 },
        { PCI_VDEVICE(TTI, 0x1740), chip_508x },
        { PCI_VDEVICE(TTI, 0x1742), chip_508x },


Best Regards,
Petr Jelen

Re: Highpoint RocketRaid 1720 support in sata_mv

[Mark Lord]

>Petr Jelen wrote:
> I have a simple patch for sata_mv.c, which adds support for Highpoint
> RocketRaid 1720 adapter (2 SATA2 ports on PCI bus).
> The adapter has Marvel 88SX6042-BCZ1 on board.
> 
> The patch is against 2.6.25.9 and I have tested it. I could normally
> work with my disks (in JBOD mode).
..

Looks simple enough.

Signed-off-by: Mark Lord <mlord@pobox.com>
---

Jeff, can you push this one-liner (or equivalent) out for 2.6.26?
I don't have a dev tree with me on the road here,
so the line offsets probably differ.


diff -urN linux-2.6.25.9/drivers/ata/sata_mv.c
linux-2.6.25.9-new/drivers/ata/sata_mv.c
--- linux-2.6.25.9/drivers/ata/sata_mv.c        2008-06-24
23:09:06.000000000 +0200
+++ linux-2.6.25.9-new/drivers/ata/sata_mv.c    2008-06-28
22:36:22.000000000 +0200
@@ -695,7 +695,8 @@
        { PCI_VDEVICE(MARVELL, 0x5041), chip_504x },
        { PCI_VDEVICE(MARVELL, 0x5080), chip_5080 },
        { PCI_VDEVICE(MARVELL, 0x5081), chip_508x },
-       /* RocketRAID 1740/174x have different identifiers */
+       /* RocketRAID 1720/174x have different identifiers */
+       { PCI_VDEVICE(TTI, 0x1720), chip_6042 },
        { PCI_VDEVICE(TTI, 0x1740), chip_508x },
        { PCI_VDEVICE(TTI, 0x1742), chip_508x },


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

Re: Highpoint RocketRaid 1720 support in sata_mv

[Petr Jelen]

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

Hi Mark,

Somehow the patch didn't make it into 2.6.26.
Could you plz. submit it for 2.6.27?

The patch against 2.6.27-rc4 is attached.

Thank you,
Petr Jelen

2008/6/29 Mark Lord <liml@rtr.ca>:
>> Petr Jelen wrote:
>> I have a simple patch for sata_mv.c, which adds support for Highpoint
>> RocketRaid 1720 adapter (2 SATA2 ports on PCI bus).
>> The adapter has Marvel 88SX6042-BCZ1 on board.
>>
>> The patch is against 2.6.25.9 and I have tested it. I could normally
>> work with my disks (in JBOD mode).
>
> ..
>
> Looks simple enough.
>
> Signed-off-by: Mark Lord <mlord@pobox.com>
> ---
>
> Jeff, can you push this one-liner (or equivalent) out for 2.6.26?
> I don't have a dev tree with me on the road here,
> so the line offsets probably differ.
>
>
> diff -urN linux-2.6.25.9/drivers/ata/sata_mv.c
> linux-2.6.25.9-new/drivers/ata/sata_mv.c
> --- linux-2.6.25.9/drivers/ata/sata_mv.c        2008-06-24
> 23:09:06.000000000 +0200
> +++ linux-2.6.25.9-new/drivers/ata/sata_mv.c    2008-06-28
> 22:36:22.000000000 +0200
> @@ -695,7 +695,8 @@
>       { PCI_VDEVICE(MARVELL, 0x5041), chip_504x },
>       { PCI_VDEVICE(MARVELL, 0x5080), chip_5080 },
>       { PCI_VDEVICE(MARVELL, 0x5081), chip_508x },
> -       /* RocketRAID 1740/174x have different identifiers */
> +       /* RocketRAID 1720/174x have different identifiers */
> +       { PCI_VDEVICE(TTI, 0x1720), chip_6042 },
>       { PCI_VDEVICE(TTI, 0x1740), chip_508x },
>       { PCI_VDEVICE(TTI, 0x1742), chip_508x },
>
>
> Best Regards,
> Petr Jelen
> --
> To unsubscribe from this list: send the line "unsubscribe linux-ide" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: rr1720.patch --]
[-- Type: text/x-diff; name=rr1720.patch, Size: 559 bytes --]

--- linux-2.6.26/drivers/ata/sata_mv.c.orig	2008-08-29 21:35:45.000000000 +0200
+++ linux-2.6.26/drivers/ata/sata_mv.c	2008-08-29 21:36:38.000000000 +0200
@@ -667,7 +667,8 @@
 	{ PCI_VDEVICE(MARVELL, 0x5041), chip_504x },
 	{ PCI_VDEVICE(MARVELL, 0x5080), chip_5080 },
 	{ PCI_VDEVICE(MARVELL, 0x5081), chip_508x },
-	/* RocketRAID 1740/174x have different identifiers */
+	/* RocketRAID 1720/174x have different identifiers */
+	{ PCI_VDEVICE(TTI, 0x1720), chip_6042 },
 	{ PCI_VDEVICE(TTI, 0x1740), chip_508x },
 	{ PCI_VDEVICE(TTI, 0x1742), chip_508x },