ata_dev_configure() should not clear dynamic device flags determined elsewhere. Only mask out ATA_DFLAG_CFG_MASK before configuring. Without this patch, ATA_DFLAG_PIO gets turned off during revalidation making PIO mode unuseable. Signed-off-by: Tejun Heo <htejun@xxxxxxxxx> --- drivers/scsi/libata-core.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) b5318e739bed15eb68a16b4d61bcd7d7c21ea8fa diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c index 0df994d..0bc81c3 100644 --- a/drivers/scsi/libata-core.c +++ b/drivers/scsi/libata-core.c @@ -1241,7 +1241,7 @@ static int ata_dev_configure(struct ata_ id[84], id[85], id[86], id[87], id[88]); /* initialize to-be-configured parameters */ - dev->flags = 0; + dev->flags &= ~ATA_DFLAG_CFG_MASK; dev->max_sectors = 0; dev->cdb_len = 0; dev->n_sectors = 0; -- 1.2.4 - : 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