[ added Andrew to cc: ] Hi, On Friday 26 October 2007, Kuan Luo wrote: > From Kuan Luo <kluo@xxxxxxxxxx> > > The patch will avoid ncq function use of sata_nv when swncq is zero by > default. It also completely removes SWNCQ support for MPC61 (something like that should be mentioned in the patch description). Otherwise patch looks fine and mine can be dumped. PS Please always at least Cc: people involved in the debugging of the problem (crediting them in the patch description would be even nicer :). > signed-off-by: Kuan Luo <kluo@xxxxxxxxxx> > --- > > sata_nv.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/ata/sata_nv.c b/drivers/ata/sata_nv.c > index 240a892..e6b6455 100644 > --- a/drivers/ata/sata_nv.c > +++ b/drivers/ata/sata_nv.c > @@ -365,9 +365,9 @@ static const struct pci_device_id nv_pci_tbl[] = { > { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP51_SATA2), > SWNCQ }, > { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_SATA), > SWNCQ }, > { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP55_SATA2), > SWNCQ }, > - { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA), > SWNCQ }, > - { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA2), > SWNCQ }, > - { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA3), > SWNCQ }, > + { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA), > GENERIC }, > + { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA2), > GENERIC }, > + { PCI_VDEVICE(NVIDIA, PCI_DEVICE_ID_NVIDIA_NFORCE_MCP61_SATA3), > GENERIC }, > > { } /* terminate list */ > }; > @@ -2383,6 +2383,9 @@ static int nv_init_one (struct pci_dev *pdev, > const struct pci_device_id *ent) > type = ADMA; > } > > + if (type == SWNCQ && !swncq_enabled) > + type = GENERIC; > + > ppi[0] = &nv_port_info[type]; > rc = ata_pci_prepare_sff_host(pdev, ppi, &host); > if (rc) - 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