* [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).