On Sun, Apr 02, 2006 at 09:55:44AM +0900, Tejun Heo wrote: > Jeff Garzik wrote: > >Tejun Heo wrote: > >>Implement ata_down_xfermask_limit(). This function manipulates > >>@dev->pio/mwdma/udma_mask such that the next lower transfer mode is > >>selected. This will be used by ata_bus_probe() rewrite and later by > >>EH speeding down. > >> > >>Signed-off-by: Tejun Heo <htejun@xxxxxxxxx> > > > >ACK, though I share Alan's concerns, even with your explanation. We'll > >see where this leads... > > > >Not applied because it will cause a compile warning. > > > > I'll write fuller description of synchronization model in the head > message of new EH framework patchset. Sorry about the compile warning. > Jeff, the compile warning is about ata_down_xfermask_limit() being unused, which is intended. This patch implements the function. Later changes will use it. I'll re-post this patch as is in the next take. -- tejun - : 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