diff options
| author | Tejun Heo <htejun@gmail.com> | 2006-04-02 05:51:53 -0400 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-04-02 10:09:19 -0400 |
| commit | ea1dd4e13010eb9dd5ffb4bfabbb472bc238bebb (patch) | |
| tree | 2c55fb5671c1dab0c99d84d46c09fd03ebdc5f53 /drivers | |
| parent | 198e0fed9e59461fc1890dd8b75ec72d14638873 (diff) | |
[PATCH] libata: clear only affected flags during ata_dev_configure()
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>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/scsi/libata-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c index 53226b16355c..985283cbcbdf 100644 --- a/drivers/scsi/libata-core.c +++ b/drivers/scsi/libata-core.c | |||
| @@ -1241,7 +1241,7 @@ static int ata_dev_configure(struct ata_port *ap, struct ata_device *dev, | |||
| 1241 | id[84], id[85], id[86], id[87], id[88]); | 1241 | id[84], id[85], id[86], id[87], id[88]); |
| 1242 | 1242 | ||
| 1243 | /* initialize to-be-configured parameters */ | 1243 | /* initialize to-be-configured parameters */ |
| 1244 | dev->flags = 0; | 1244 | dev->flags &= ~ATA_DFLAG_CFG_MASK; |
| 1245 | dev->max_sectors = 0; | 1245 | dev->max_sectors = 0; |
| 1246 | dev->cdb_len = 0; | 1246 | dev->cdb_len = 0; |
| 1247 | dev->n_sectors = 0; | 1247 | dev->n_sectors = 0; |