> Can you please add comments to explain it? Also, it would be nice if > you do ap->ctl = ap->last_ctl and turn off ATA_TFLAG_DEVICE to avoid > loading them yet again. I don't think you can turn off ATA_TFLAG_DEVICE, you have no guarantee it won't be referred to again anywhere else in the stack - tf->flags isn't owned by the driver so its asking for trouble to do that. Alan -- 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