Re: [PATCH] sata_mv: Test patch for Hightpoint RocketRaid 1740/1742

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Alan Cox wrote:
On Sat, 01 Sep 2007 15:38:43 +0200
Krzysztof Oledzki <[EMAIL PROTECTED]> wrote:

On 2007-08-01 17:46, Alan Cox wrote:
> On Wed, 01 Aug 2007 09:49:19 -0400
> Jeff Garzik <[EMAIL PROTECTED]> wrote:
> >> Alan Cox wrote:
>>> Underneath all the HPT packaging, PCI identifiers, binary driver modules
>>> and stuff you find that ...
>>>
>>> Signed-off-by: Alan Cox <[EMAIL PROTECTED]>
>>>
>>> --- drivers/ata/sata_mv.c~        2007-07-09 13:19:57.003052904 +0100
>>> +++ drivers/ata/sata_mv.c 2007-07-09 13:19:57.004052752 +0100
>>> @@ -573,6 +573,9 @@
>>>   { PCI_VDEVICE(MARVELL, 0x5041), chip_504x },
>>>   { PCI_VDEVICE(MARVELL, 0x5080), chip_5080 },
>>>   { PCI_VDEVICE(MARVELL, 0x5081), chip_508x },
>>> + /* RocketRAID 1740/174x have different identifiers */
>>> + { PCI_VDEVICE(TTI, 0x1740), chip_508x },
>>> + { PCI_VDEVICE(TTI, 0x1742), chip_508x },
>> Is this still a test patch, or can it go upstream?
> > Upstream

Are you sure that 1740/1742 are 508x (GEN I)? According to these [1] pictures there may be a 6042 chipset (GEN IIe) placed onboard.

The ones I checked and the code used as a reference clearly indicate a
508x. I've had two reports of a TTI 0x23xx ID number which may perhaps be
a 6042 as it didn't appear to work with a 508x driver.

Alan
..

Dredging up an old posting here:

Alan/Jeff, I've now got two reports that the above patch is not correct
for the Highpoint RocketRAID 1740 cards.

PCI ID 1103:1740 is clearly a Marvell 6042 chip, not a 5081.

Patch to correct this is coming shortly.

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

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux