Bartlomiej Zolnierkiewicz wrote: >> - if (!ata_dev_enabled(dev) && >> - ((ehc->i.probe_mask & (1 << dev->devno)) && >> - !(ehc->did_probe_mask & (1 << dev->devno)))) { >> - ata_eh_detach_dev(dev); >> - ata_dev_init(dev); >> - ehc->did_probe_mask |= (1 << dev->devno); >> - ehc->i.action |= ATA_EH_SOFTRESET; >> - } >> + /* schedule probe if necessary */ >> + ata_eh_schedule_probe(dev); > > !ata_dev_enabled() got lost, is this intended? Definitely not. Thanks a lot for catching this. Jeff, I'll post an updated version. Thanks. -- tejun - 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