From: Jeff Garzik <jeff@garzik.org>
To: Krzysztof Oledzki <olel@ans.pl>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
linux-ide@vger.kernel.org, akpm@osdl.org
Subject: Re: [PATCH] sata_mv: Test patch for Hightpoint RocketRaid 1740/1742
Date: Sat, 01 Sep 2007 09:50:23 -0400 [thread overview]
Message-ID: <46D96E1F.7080903@garzik.org> (raw)
In-Reply-To: <46D96B63.7050302@ans.pl>
Krzysztof Oledzki wrote:
> On 2007-08-01 17:46, Alan Cox wrote:
>> On Wed, 01 Aug 2007 09:49:19 -0400
>> Jeff Garzik <jeff@garzik.org> wrote:
>>
>>> Alan Cox wrote:
>>>> Underneath all the HPT packaging, PCI identifiers, binary driver
>>>> modules
>>>> and stuff you find that ...
>>>>
>>>> Signed-off-by: Alan Cox <alan@redhat.com>
>>>>
>>>> --- 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.
>
> [1] http://www.highpoint-tech.com/image/products/SATAII/RR1740_card_big.jpg
> http://www.highpoint-tech.com/image/products/eSATA/RR1742_card_big.jpg
Good evidence!
"SATAII" is also an indicator that it is not 508x.
Jeff
next prev parent reply other threads:[~2007-09-01 13:50 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-07-09 13:38 [PATCH] sata_mv: Test patch for Hightpoint RocketRaid 1740/1742 Alan Cox
2007-07-12 21:00 ` Jeff Garzik
2007-07-13 14:07 ` Alan Cox
2009-01-27 21:15 ` Mark Lord
2009-01-27 21:21 ` [PATCH] sata_mv: Fix chip type for Hightpoint RocketRaid 1740 Mark Lord
2009-01-27 21:25 ` Mark Lord
2009-01-27 21:33 ` [PATCH] sata_mv: Fix chip type for Hightpoint RocketRaid 1740/1742 Mark Lord
2009-02-03 3:42 ` Jeff Garzik
2007-08-01 13:49 ` [PATCH] sata_mv: Test patch " Jeff Garzik
2007-08-01 15:46 ` Alan Cox
2007-09-01 13:38 ` Krzysztof Oledzki
2007-09-01 13:50 ` Jeff Garzik [this message]
2007-09-01 14:49 ` Alan Cox
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=46D96E1F.7080903@garzik.org \
--to=jeff@garzik.org \
--cc=akpm@osdl.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=linux-ide@vger.kernel.org \
--cc=olel@ans.pl \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.