diff options
| author | Tejun Heo <htejun@gmail.com> | 2006-04-11 22:26:29 +0900 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-04-11 13:22:17 -0400 |
| commit | 51713d359ae274fa4dd4b199ba3a6b0c21ef99e0 (patch) | |
| tree | c233c1f8848ed921cabd4f8591085c24ccb0cdc3 /drivers/scsi/libata-core.c | |
| parent | ec573755fcd7975aae6b0d536dbcd74a6eed029c (diff) | |
| download | lwn-51713d359ae274fa4dd4b199ba3a6b0c21ef99e0.tar.gz lwn-51713d359ae274fa4dd4b199ba3a6b0c21ef99e0.zip | |
[PATCH] libata: cosmetic update to ata_bus_probe()
Move ata_set_mode() failure handling outside of ap->ops->set_mode if
clause such that it can handle ap->ops->set_mode failures after it's
updated.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/scsi/libata-core.c')
| -rw-r--r-- | drivers/scsi/libata-core.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c index 9194e4a6a55a..588578dc6caf 100644 --- a/drivers/scsi/libata-core.c +++ b/drivers/scsi/libata-core.c @@ -1441,12 +1441,12 @@ static int ata_bus_probe(struct ata_port *ap) break; } rc = 0; - } else { + } else rc = ata_set_mode(ap, &dev); - if (rc) { - down_xfermask = 1; - goto fail; - } + + if (rc) { + down_xfermask = 1; + goto fail; } for (i = 0; i < ATA_MAX_DEVICES; i++) |