ahci: Add MCP79 support to AHCI driver

PATCH Applied Peer Chen 4 participants 7 messages Sep 21, 2007 – Oct 2, 2007 libata

lore

Share & tools

lore

Peer Chen

author

2007-09-21 06:06 UTC #0 lore raw

PATCH 1/1 ahci: Add MCP79 support to AHCI driver

Add the MCP79 support to ahci driver.
The patch base on kernel 2.6.23-rc7

Signed-off-by Peer Chen <[email protected]>

---

drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +12 -0

@@ -472,6 +472,18 @@ static const struct pci_device_id ahci_p
 	{ PCI_VDEVICE(NVIDIA, 0x0ad9), board_ahci },		/* MCP77 */
 	{ PCI_VDEVICE(NVIDIA, 0x0ada), board_ahci },		/* MCP77 */
 	{ PCI_VDEVICE(NVIDIA, 0x0adb), board_ahci },		/* MCP77 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab4), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab5), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab6), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab7), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab8), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab9), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0aba), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abb), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abc), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abd), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abe), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abf), board_ahci },		/* MCP79 */
 
 	/* SiS */
 	{ PCI_VDEVICE(SI, 0x1184), board_ahci }, /* SiS 966 */

Alan Cox

2007-09-21 10:32 UTC #1 lore raw

Quoted 2 lines · On Fri, 21 Sep 2007 14:06:10 +0800 "Peer Chen" <[email protected]> wrote:

Add the MCP79 support to ahci driver.
The patch base on kernel 2.6.23-rc7

Is this actually needed or as they are standard ahci devices will the class match get them anyway ?

Peer Chen

author

2007-09-24 02:02 UTC #2 lore raw

Yes,it's necessary because our RAID controller also use the ahci driver.

------------------
Peer Chen
2007-09-24

-------------------------------------------------------------
·¢¼þÈË£ºAlan Cox
·¢ËÍÈÕÆÚ£º2007-09-21 18:32:09
ÊÕ¼þÈË£ºPeer Chen
³ËÍ£ºlinux-kernel; linux-ide; akpm; jeff Ö÷Ìâ£ºRe: [PATCH] ahci: Add MCP79 support to AHCI driver

Quoted 2 lines · On Fri, 21 Sep 2007 14:06:10 +0800 "Peer Chen" <[email protected]> wrote:

Add the MCP79 support to ahci driver.
The patch base on kernel 2.6.23-rc7

Is this actually needed or as they are standard ahci devices will the class match get them anyway ?

Peer Chen

author

2007-09-24 02:16 UTC #3 lore raw

In reply to Alan Cox

Code change, remove some Device IDs.

Signed-off-by Peer Chen <[email protected]>

---

drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +8 -0

@@ -472,6 +472,14 @@ static const struct pci_device_id ahci_p
 	{ PCI_VDEVICE(NVIDIA, 0x0ad9), board_ahci },		/* MCP77 */
 	{ PCI_VDEVICE(NVIDIA, 0x0ada), board_ahci },		/* MCP77 */
 	{ PCI_VDEVICE(NVIDIA, 0x0adb), board_ahci },		/* MCP77 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab8), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab9), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0aba), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abb), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abc), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abd), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abe), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abf), board_ahci },		/* MCP79 */
 
 	/* SiS */
 	{ PCI_VDEVICE(SI, 0x1184), board_ahci }, /* SiS 966 */

------------------
Peer Chen
2007-09-24

Quoted 2 lines · On Fri, 21 Sep 2007 14:06:10 +0800 "Peer Chen" <[email protected]> wrote:

Add the MCP79 support to ahci driver.
The patch base on kernel 2.6.23-rc7

Is this actually needed or as they are standard ahci devices will the class match get them anyway ?

Sergei Shtylyov

2007-09-24 12:53 UTC #4 lore raw

Hello.

Quoted 8 lines · Peer Chen

Code change, remove some Device IDs.
Signed-off-by Peer Chen <[email protected]>
---
drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +0 -0
@@ -472,6 +472,14 @@ static const struct pci_device_id ahci_p
 	{ PCI_VDEVICE(NVIDIA, 0x0ad9), board_ahci },		/* MCP77 */

drivers/ata/ahci.c static const struct pci_device_id ahci_p · line 482

Quoted 10 lines

drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +8 -0

 	{ PCI_VDEVICE(NVIDIA, 0x0ada), board_ahci },		/* MCP77 */
 	{ PCI_VDEVICE(NVIDIA, 0x0adb), board_ahci },		/* MCP77 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab8), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab9), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0aba), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abb), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abc), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abd), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abe), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abf), board_ahci },		/* MCP79 */

I wonder whether all of those 8 IDs belong to AHCI controllers...

MBR, Sergei

Peer Chen

author

2007-09-25 02:58 UTC #5 lore raw

Yes,they all belong to AHCI controllers, 4 of them use ahci class code and others use RAID class code.

------------------
Peer Chen
2007-09-25

-------------------------------------------------------------
·¢¼þÈË£ºSergei Shtylyov
·¢ËÍÈÕÆÚ£º2007-09-24 20:53:44
ÊÕ¼þÈË£ºPeer Chen
³ËÍ£ºAlan Cox; linux-kernel; linux-ide; akpm; jeff Ö÷Ìâ£ºRe: [PATCH] ahci: Add MCP79 support to AHCI driver

Hello.

Quoted 8 lines · Peer Chen

Code change, remove some Device IDs.
Signed-off-by Peer Chen <[email protected]>
---
drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +0 -0
@@ -472,6 +472,14 @@ static const struct pci_device_id ahci_p
 	{ PCI_VDEVICE(NVIDIA, 0x0ad9), board_ahci },		/* MCP77 */

drivers/ata/ahci.c static const struct pci_device_id ahci_p · line 482

Quoted 10 lines

drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +8 -0

 	{ PCI_VDEVICE(NVIDIA, 0x0ada), board_ahci },		/* MCP77 */
 	{ PCI_VDEVICE(NVIDIA, 0x0adb), board_ahci },		/* MCP77 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab8), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab9), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0aba), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abb), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abc), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abd), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abe), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abf), board_ahci },		/* MCP79 */

I wonder whether all of those 8 IDs belong to AHCI controllers...

MBR, Sergei

Jeff Garzik

Applied

2007-10-02 15:31 UTC #6 lore raw

In reply to Peer Chen

Quoted 8 lines · Peer Chen

Code change, remove some Device IDs.
Signed-off-by Peer Chen <[email protected]>
---
drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +0 -0
@@ -472,6 +472,14 @@ static const struct pci_device_id ahci_p
 	{ PCI_VDEVICE(NVIDIA, 0x0ad9), board_ahci },		/* MCP77 */

drivers/ata/ahci.c static const struct pci_device_id ahci_p · line 482

Quoted 10 lines

drivers/ata/ahci.c renamed from drivers/ata/ahci.c.orig +8 -0

 	{ PCI_VDEVICE(NVIDIA, 0x0ada), board_ahci },		/* MCP77 */
 	{ PCI_VDEVICE(NVIDIA, 0x0adb), board_ahci },		/* MCP77 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab8), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0ab9), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0aba), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abb), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abc), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abd), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abe), board_ahci },		/* MCP79 */
+	{ PCI_VDEVICE(NVIDIA, 0x0abf), board_ahci },		/* MCP79 */

applied