* [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
@ 2007-08-31 4:27 Jason Gaston
2007-08-31 7:47 ` Jeff Garzik
2007-09-20 21:35 ` Jeff Garzik
0 siblings, 2 replies; 9+ messages in thread
From: Jason Gaston @ 2007-08-31 4:27 UTC (permalink / raw)
To: jgarzik, jason.d.gaston, linux-ide, linux-kernel
Resend trying to remove 8-bit characters in the email.
This patch adds the Intel Tolapai RAID controller DID's for SATA support.
Signed-off-by: Jason Gaston <jason.d.gaston@intel.com>
--- linux-2.6.23-rc4/drivers/ata/ahci.c.orig 2007-08-27 18:32:35.000000000 -0700
+++ linux-2.6.23-rc4/drivers/ata/ahci.c 2007-08-28 16:58:11.000000000 -0700
@@ -411,6 +411,8 @@
{ PCI_VDEVICE(INTEL, 0x292f), board_ahci_pi }, /* ICH9M */
{ PCI_VDEVICE(INTEL, 0x294d), board_ahci_pi }, /* ICH9 */
{ PCI_VDEVICE(INTEL, 0x294e), board_ahci_pi }, /* ICH9M */
+ { PCI_VDEVICE(INTEL, 0x502a), board_ahci }, /* Tolapai */
+ { PCI_VDEVICE(INTEL, 0x502b), board_ahci }, /* Tolapai */
/* JMicron 360/1/3/5/6, match class to avoid IDE function */
{ PCI_VENDOR_ID_JMICRON, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-08-31 4:27 [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai Jason Gaston
@ 2007-08-31 7:47 ` Jeff Garzik
2007-08-31 17:10 ` Gaston, Jason D
2007-09-06 17:27 ` Gaston, Jason D
2007-09-20 21:35 ` Jeff Garzik
1 sibling, 2 replies; 9+ messages in thread
From: Jeff Garzik @ 2007-08-31 7:47 UTC (permalink / raw)
To: Jason Gaston; +Cc: linux-ide, linux-kernel
Jason Gaston wrote:
> Resend trying to remove 8-bit characters in the email.
>
> This patch adds the Intel Tolapai RAID controller DID's for SATA support.
>
> Signed-off-by: Jason Gaston <jason.d.gaston@intel.com>
>
> --- linux-2.6.23-rc4/drivers/ata/ahci.c.orig 2007-08-27 18:32:35.000000000 -0700
> +++ linux-2.6.23-rc4/drivers/ata/ahci.c 2007-08-28 16:58:11.000000000 -0700
> @@ -411,6 +411,8 @@
> { PCI_VDEVICE(INTEL, 0x292f), board_ahci_pi }, /* ICH9M */
> { PCI_VDEVICE(INTEL, 0x294d), board_ahci_pi }, /* ICH9 */
> { PCI_VDEVICE(INTEL, 0x294e), board_ahci_pi }, /* ICH9M */
> + { PCI_VDEVICE(INTEL, 0x502a), board_ahci }, /* Tolapai */
> + { PCI_VDEVICE(INTEL, 0x502b), board_ahci }, /* Tolapai */
Why did you not use board_ahci_pi? Is the AHCI ports-implemented
register unreliable on this platform?
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-08-31 7:47 ` Jeff Garzik
@ 2007-08-31 17:10 ` Gaston, Jason D
2007-09-06 17:27 ` Gaston, Jason D
1 sibling, 0 replies; 9+ messages in thread
From: Gaston, Jason D @ 2007-08-31 17:10 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-ide, linux-kernel
This device has both AHCI and RAID modes that use the ahci driver. Only
the RAID mode DID's are being added as the PCI class code support will
cover the AHCI mode. Looking at the Generic, PCI class code support
section, it uses "board_ahci". I assumed that they should be the same
as the generic class code support is working on this platform.
Thanks,
Jason
>-----Original Message-----
>From: Jeff Garzik [mailto:jgarzik@pobox.com]
>Sent: Friday, August 31, 2007 12:47 AM
>To: Gaston, Jason D
>Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>Subject: Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch
for
>Intel Tolapai
>
>Jason Gaston wrote:
>> Resend trying to remove 8-bit characters in the email.
>>
>> This patch adds the Intel Tolapai RAID controller DID's for SATA
support.
>>
>> Signed-off-by: Jason Gaston <jason.d.gaston@intel.com>
>>
>> --- linux-2.6.23-rc4/drivers/ata/ahci.c.orig 2007-08-27
>18:32:35.000000000 -0700
>> +++ linux-2.6.23-rc4/drivers/ata/ahci.c 2007-08-28
16:58:11.000000000 -
>0700
>> @@ -411,6 +411,8 @@
>> { PCI_VDEVICE(INTEL, 0x292f), board_ahci_pi }, /* ICH9M */
>> { PCI_VDEVICE(INTEL, 0x294d), board_ahci_pi }, /* ICH9 */
>> { PCI_VDEVICE(INTEL, 0x294e), board_ahci_pi }, /* ICH9M */
>> + { PCI_VDEVICE(INTEL, 0x502a), board_ahci }, /* Tolapai */
>> + { PCI_VDEVICE(INTEL, 0x502b), board_ahci }, /* Tolapai */
>
>Why did you not use board_ahci_pi? Is the AHCI ports-implemented
>register unreliable on this platform?
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-08-31 7:47 ` Jeff Garzik
2007-08-31 17:10 ` Gaston, Jason D
@ 2007-09-06 17:27 ` Gaston, Jason D
2007-09-07 22:39 ` Jeff Garzik
1 sibling, 1 reply; 9+ messages in thread
From: Gaston, Jason D @ 2007-09-06 17:27 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-ide, linux-kernel, Gaston, Jason D
>-----Original Message-----
>From: Gaston, Jason D
>Sent: Friday, August 31, 2007 10:10 AM
>To: 'Jeff Garzik'
>Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>Subject: RE: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch
for
>Intel Tolapai
>
>This device has both AHCI and RAID modes that use the ahci driver.
Only
>the RAID mode DID's are being added as the PCI class code support will
>cover the AHCI mode. Looking at the Generic, PCI class code support
>section, it uses "board_ahci". I assumed that they should be the same
as
>the generic class code support is working on this platform.
>
>Thanks,
>
>Jason
>
>
>>-----Original Message-----
>>From: Jeff Garzik [mailto:jgarzik@pobox.com]
>>Sent: Friday, August 31, 2007 12:47 AM
>>To: Gaston, Jason D
>>Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>>Subject: Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch
for
>>Intel Tolapai
>>
>>Jason Gaston wrote:
>>> Resend trying to remove 8-bit characters in the email.
>>>
>>> This patch adds the Intel Tolapai RAID controller DID's for SATA
>support.
>>>
>>> Signed-off-by: Jason Gaston <jason.d.gaston@intel.com>
>>>
>>> --- linux-2.6.23-rc4/drivers/ata/ahci.c.orig 2007-08-27
>>18:32:35.000000000 -0700
>>> +++ linux-2.6.23-rc4/drivers/ata/ahci.c 2007-08-28
>16:58:11.000000000 -
>>0700
>>> @@ -411,6 +411,8 @@
>>> { PCI_VDEVICE(INTEL, 0x292f), board_ahci_pi }, /* ICH9M */
>>> { PCI_VDEVICE(INTEL, 0x294d), board_ahci_pi }, /* ICH9 */
>>> { PCI_VDEVICE(INTEL, 0x294e), board_ahci_pi }, /* ICH9M */
>>> + { PCI_VDEVICE(INTEL, 0x502a), board_ahci }, /* Tolapai */
>>> + { PCI_VDEVICE(INTEL, 0x502b), board_ahci }, /* Tolapai */
>>
>>Why did you not use board_ahci_pi? Is the AHCI ports-implemented
>>register unreliable on this platform?
Jeff,
Do I need to change this to board_ahci_pi or is it ok to leave it at
board_ahci, which will be used by the AHCI class code devices?
Thanks,
Jason
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-09-06 17:27 ` Gaston, Jason D
@ 2007-09-07 22:39 ` Jeff Garzik
2007-09-07 23:37 ` Gaston, Jason D
0 siblings, 1 reply; 9+ messages in thread
From: Jeff Garzik @ 2007-09-07 22:39 UTC (permalink / raw)
To: Gaston, Jason D; +Cc: linux-ide, linux-kernel
Gaston, Jason D wrote:
>> -----Original Message-----
>> From: Gaston, Jason D
>> Sent: Friday, August 31, 2007 10:10 AM
>> To: 'Jeff Garzik'
>> Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>> Subject: RE: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch
> for
>> Intel Tolapai
>>
>> This device has both AHCI and RAID modes that use the ahci driver.
> Only
>> the RAID mode DID's are being added as the PCI class code support will
>> cover the AHCI mode. Looking at the Generic, PCI class code support
>> section, it uses "board_ahci". I assumed that they should be the same
> as
>> the generic class code support is working on this platform.
>>
>> Thanks,
>>
>> Jason
>>
>>
>>> -----Original Message-----
>>> From: Jeff Garzik [mailto:jgarzik@pobox.com]
>>> Sent: Friday, August 31, 2007 12:47 AM
>>> To: Gaston, Jason D
>>> Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>>> Subject: Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch
> for
>>> Intel Tolapai
>>>
>>> Jason Gaston wrote:
>>>> Resend trying to remove 8-bit characters in the email.
>>>>
>>>> This patch adds the Intel Tolapai RAID controller DID's for SATA
>> support.
>>>> Signed-off-by: Jason Gaston <jason.d.gaston@intel.com>
>>>>
>>>> --- linux-2.6.23-rc4/drivers/ata/ahci.c.orig 2007-08-27
>>> 18:32:35.000000000 -0700
>>>> +++ linux-2.6.23-rc4/drivers/ata/ahci.c 2007-08-28
>> 16:58:11.000000000 -
>>> 0700
>>>> @@ -411,6 +411,8 @@
>>>> { PCI_VDEVICE(INTEL, 0x292f), board_ahci_pi }, /* ICH9M */
>>>> { PCI_VDEVICE(INTEL, 0x294d), board_ahci_pi }, /* ICH9 */
>>>> { PCI_VDEVICE(INTEL, 0x294e), board_ahci_pi }, /* ICH9M */
>>>> + { PCI_VDEVICE(INTEL, 0x502a), board_ahci }, /* Tolapai */
>>>> + { PCI_VDEVICE(INTEL, 0x502b), board_ahci }, /* Tolapai */
>>> Why did you not use board_ahci_pi? Is the AHCI ports-implemented
>>> register unreliable on this platform?
>
> Jeff,
>
> Do I need to change this to board_ahci_pi or is it ok to leave it at
> board_ahci, which will be used by the AHCI class code devices?
You are the one who needs to answer this question ;-)
Most new Intel AHCI have a sane and reliable Ports-Implemented register
value even across reset, unlike earlier ones or some clones. For those,
we use board_ahci_pi.
If PI is not reliable across reset or if BIOS is absent (yes we care
about that case, when we do our own PCI resume for example), then you
should use board_ahci.
Jeff
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-09-07 22:39 ` Jeff Garzik
@ 2007-09-07 23:37 ` Gaston, Jason D
2007-09-07 23:57 ` Jeff Garzik
0 siblings, 1 reply; 9+ messages in thread
From: Gaston, Jason D @ 2007-09-07 23:37 UTC (permalink / raw)
To: Jeff Garzik; +Cc: linux-ide, linux-kernel
>-----Original Message-----
>From: Jeff Garzik [mailto:jgarzik@pobox.com]
>Sent: Friday, September 07, 2007 3:39 PM
>To: Gaston, Jason D
>Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>Subject: Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch
for
>Intel Tolapai
>
>Gaston, Jason D wrote:
>>> -----Original Message-----
>>> From: Gaston, Jason D
>>> Sent: Friday, August 31, 2007 10:10 AM
>>> To: 'Jeff Garzik'
>>> Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>>> Subject: RE: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch
>> for
>>> Intel Tolapai
>>>
>>> This device has both AHCI and RAID modes that use the ahci driver.
>> Only
>>> the RAID mode DID's are being added as the PCI class code support
will
>>> cover the AHCI mode. Looking at the Generic, PCI class code support
>>> section, it uses "board_ahci". I assumed that they should be the
same
>> as
>>> the generic class code support is working on this platform.
>>>
>>> Thanks,
>>>
>>> Jason
>>>
>>>
>>>> -----Original Message-----
>>>> From: Jeff Garzik [mailto:jgarzik@pobox.com]
>>>> Sent: Friday, August 31, 2007 12:47 AM
>>>> To: Gaston, Jason D
>>>> Cc: linux-ide@vger.kernel.org; linux-kernel@vger.kernel.org
>>>> Subject: Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA
patch
>> for
>>>> Intel Tolapai
>>>>
>>>> Jason Gaston wrote:
>>>>> Resend trying to remove 8-bit characters in the email.
>>>>>
>>>>> This patch adds the Intel Tolapai RAID controller DID's for SATA
>>> support.
>>>>> Signed-off-by: Jason Gaston <jason.d.gaston@intel.com>
>>>>>
>>>>> --- linux-2.6.23-rc4/drivers/ata/ahci.c.orig 2007-08-27
>>>> 18:32:35.000000000 -0700
>>>>> +++ linux-2.6.23-rc4/drivers/ata/ahci.c 2007-08-28
>>> 16:58:11.000000000 -
>>>> 0700
>>>>> @@ -411,6 +411,8 @@
>>>>> { PCI_VDEVICE(INTEL, 0x292f), board_ahci_pi }, /* ICH9M */
>>>>> { PCI_VDEVICE(INTEL, 0x294d), board_ahci_pi }, /* ICH9 */
>>>>> { PCI_VDEVICE(INTEL, 0x294e), board_ahci_pi }, /* ICH9M */
>>>>> + { PCI_VDEVICE(INTEL, 0x502a), board_ahci }, /* Tolapai */
>>>>> + { PCI_VDEVICE(INTEL, 0x502b), board_ahci }, /* Tolapai */
>>>> Why did you not use board_ahci_pi? Is the AHCI ports-implemented
>>>> register unreliable on this platform?
>>
>> Jeff,
>>
>> Do I need to change this to board_ahci_pi or is it ok to leave it at
>> board_ahci, which will be used by the AHCI class code devices?
>
>You are the one who needs to answer this question ;-)
>
>Most new Intel AHCI have a sane and reliable Ports-Implemented register
>value even across reset, unlike earlier ones or some clones. For
those,
>we use board_ahci_pi.
>
>If PI is not reliable across reset or if BIOS is absent (yes we care
>about that case, when we do our own PCI resume for example), then you
>should use board_ahci.
>
> Jeff
At this time, I don't have any way to test those particular DeviceID's
and I know that the AHCI mode DeviceID works by using the class code
support. So, I would like to just leave them at they are, if that is
ok.
Thanks,
Jason
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-09-07 23:37 ` Gaston, Jason D
@ 2007-09-07 23:57 ` Jeff Garzik
2007-09-08 5:51 ` Tejun Heo
0 siblings, 1 reply; 9+ messages in thread
From: Jeff Garzik @ 2007-09-07 23:57 UTC (permalink / raw)
To: Gaston, Jason D; +Cc: linux-ide, linux-kernel
Gaston, Jason D wrote:
> At this time, I don't have any way to test those particular DeviceID's
> and I know that the AHCI mode DeviceID works by using the class code
> support. So, I would like to just leave them at they are, if that is
> ok.
Fine by me... Overall I'll follow "vendor's best advice" here :)
Jeff
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-09-07 23:57 ` Jeff Garzik
@ 2007-09-08 5:51 ` Tejun Heo
0 siblings, 0 replies; 9+ messages in thread
From: Tejun Heo @ 2007-09-08 5:51 UTC (permalink / raw)
To: Jeff Garzik; +Cc: Gaston, Jason D, linux-ide, linux-kernel
Jeff Garzik wrote:
> Gaston, Jason D wrote:
>> At this time, I don't have any way to test those particular DeviceID's
>> and I know that the AHCI mode DeviceID works by using the class code
>> support. So, I would like to just leave them at they are, if that is
>> ok.
>
>
> Fine by me... Overall I'll follow "vendor's best advice" here :)
The distinction between board_ahci and board_ahci_pi has been removed in
#upstream anyway, so it doesn't matter too much anyway.
Thanks.
--
tejun
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai
2007-08-31 4:27 [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai Jason Gaston
2007-08-31 7:47 ` Jeff Garzik
@ 2007-09-20 21:35 ` Jeff Garzik
1 sibling, 0 replies; 9+ messages in thread
From: Jeff Garzik @ 2007-09-20 21:35 UTC (permalink / raw)
To: Jason Gaston; +Cc: linux-ide, linux-kernel
Jason Gaston wrote:
> Resend trying to remove 8-bit characters in the email.
>
> This patch adds the Intel Tolapai RAID controller DID's for SATA support.
>
> Signed-off-by: Jason Gaston <jason.d.gaston@intel.com>
>
> --- linux-2.6.23-rc4/drivers/ata/ahci.c.orig 2007-08-27 18:32:35.000000000 -0700
> +++ linux-2.6.23-rc4/drivers/ata/ahci.c 2007-08-28 16:58:11.000000000 -0700
> @@ -411,6 +411,8 @@
> { PCI_VDEVICE(INTEL, 0x292f), board_ahci_pi }, /* ICH9M */
> { PCI_VDEVICE(INTEL, 0x294d), board_ahci_pi }, /* ICH9 */
> { PCI_VDEVICE(INTEL, 0x294e), board_ahci_pi }, /* ICH9M */
> + { PCI_VDEVICE(INTEL, 0x502a), board_ahci }, /* Tolapai */
> + { PCI_VDEVICE(INTEL, 0x502b), board_ahci }, /* Tolapai */
applied manually, since it conflicted with Tejun's PI patch (not your fault)
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2007-09-20 21:35 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-31 4:27 [PATCH 2.6.23-rc4][reRESEND] ahci: RAID mode SATA patch for Intel Tolapai Jason Gaston
2007-08-31 7:47 ` Jeff Garzik
2007-08-31 17:10 ` Gaston, Jason D
2007-09-06 17:27 ` Gaston, Jason D
2007-09-07 22:39 ` Jeff Garzik
2007-09-07 23:37 ` Gaston, Jason D
2007-09-07 23:57 ` Jeff Garzik
2007-09-08 5:51 ` Tejun Heo
2007-09-20 21:35 ` Jeff Garzik
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).