![]() | |||||||||||||
Messages in this thread |
On Sat, 14 Jul 2007 04:17:28 -0400 Jeff Garzik <jeff@garzik.org> wrote: > > Review definitely helps here... Without knowing what the precise semantics are that is hard. Every SFF_DMA case may not be supporting DMA depending upon the configuration so it should probably be set by ata_pci_init_one for all the normal SFF devices. Thus I would say - Drop every user of ata_pci_init_one from the patch in ata_pci_init_one set it to SFF in ata_pci_init_bmdma set it to SFF + DMA on success and while we are at it this would nicely fix your TODO in ata_pci_init_bmdma by testing the class when copying the modes over. Alan - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2007-07-14 10:53 [from the cache] ©2003-2008 | |||||||||||||