On Mon, 13 Oct 2008 16:17:02 +0900 Tejun Heo <htejun@xxxxxxxxx> wrote: > Hello, > > Wang Jian wrote: > > +static void pata_platform_postreset(struct ata_link *link, unsigned int *classes) > > +{ > > + struct ata_port *ap = link->ap; > > + struct ata_device *dev; > > + u8 select = ATA_DEVICE_OBS; > > + > > + /* Call default callback first */ > > + ata_sff_postreset(link, classes); > > + > > + if (!(ap->flags & ATA_FLAG_8BIT_DATA)) > > + return; > > + > > + /* Set 8-bit mode. We know we can do that */ Not really - the 8bit transfer command is CFA specific not ATA or ATAPI, in addition it is not needed with most hardware that is running in 8bit modes. -- 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