[PATCH] libata: clear only affected flags during ata_dev_configure()
authorTejun Heo <htejun@gmail.com>
Sun, 2 Apr 2006 09:51:53 +0000 (18:51 +0900)
committerJeff Garzik <jeff@garzik.org>
Sun, 2 Apr 2006 14:09:19 +0000 (10:09 -0400)
ata_dev_configure() should not clear dynamic device flags determined
elsewhere.  Lower eight bits are reserved for feature flags, define
ATA_DFLAG_CFG_MASK and clear only those bits before configuring
device.  Without this patch, ATA_DFLAG_PIO gets turned off during
revalidation making PIO mode unuseable.

Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/scsi/libata-core.c
include/linux/libata.h

index 53226b16355cd4d014cd0a2f427a3789dc9dadbd..985283cbcbdf2c4832d8909bc931f961c3024ae5 100644 (file)
@@ -1241,7 +1241,7 @@ static int ata_dev_configure(struct ata_port *ap, struct ata_device *dev,
                       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;
index 890262f44d0a78728410ed0ffd34ffb3e21e1e60..cbbc821fe22c6a2213b8900f9bc3c53ab207a19a 100644 (file)
@@ -122,6 +122,7 @@ enum {
        /* struct ata_device stuff */
        ATA_DFLAG_LBA           = (1 << 0), /* device supports LBA */
        ATA_DFLAG_LBA48         = (1 << 1), /* device supports LBA48 */
+       ATA_DFLAG_CFG_MASK      = (1 << 8) - 1,
 
        ATA_DFLAG_PIO           = (1 << 8), /* device currently in PIO mode */