diff options
author | Tejun Heo <htejun@gmail.com> | 2006-04-11 09:16:45 -0400 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2006-04-11 13:12:46 -0400 |
commit | 2bf2cb26b2512c6a609bb152982c388329bedff6 (patch) | |
tree | 9f1f3ea5000304fd946f34b24d9904c2a1027480 /drivers/scsi/ahci.c | |
parent | db70fef0750e5f8dbb64f9fadb333d2c7caf26a1 (diff) |
[PATCH] libata: kill @verbose from ata_reset_fn_t
@verbose was added to ata_reset_fn_t because AHCI complained during
probing if no device was attached to the port. However, muting
failure message isn't the correct approach. Reset methods are
responsible for detecting no device condition and finishing
successfully. Now that AHCI softreset is fixed, kill @verbose.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/scsi/ahci.c')
-rw-r--r-- | drivers/scsi/ahci.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/drivers/scsi/ahci.c b/drivers/scsi/ahci.c index 20fc0846e0be..0e7fb9bf2cd1 100644 --- a/drivers/scsi/ahci.c +++ b/drivers/scsi/ahci.c | |||
@@ -534,7 +534,7 @@ static int ahci_poll_register(void __iomem *reg, u32 mask, u32 val, | |||
534 | return -1; | 534 | return -1; |
535 | } | 535 | } |
536 | 536 | ||
537 | static int ahci_softreset(struct ata_port *ap, int verbose, unsigned int *class) | 537 | static int ahci_softreset(struct ata_port *ap, unsigned int *class) |
538 | { | 538 | { |
539 | struct ahci_host_priv *hpriv = ap->host_set->private_data; | 539 | struct ahci_host_priv *hpriv = ap->host_set->private_data; |
540 | struct ahci_port_priv *pp = ap->private_data; | 540 | struct ahci_port_priv *pp = ap->private_data; |
@@ -646,22 +646,19 @@ static int ahci_softreset(struct ata_port *ap, int verbose, unsigned int *class) | |||
646 | fail_restart: | 646 | fail_restart: |
647 | ahci_start_engine(ap); | 647 | ahci_start_engine(ap); |
648 | fail: | 648 | fail: |
649 | if (verbose) | 649 | printk(KERN_ERR "ata%u: softreset failed (%s)\n", |
650 | printk(KERN_ERR "ata%u: softreset failed (%s)\n", | 650 | ap->id, reason); |
651 | ap->id, reason); | ||
652 | else | ||
653 | DPRINTK("EXIT, rc=%d reason=\"%s\"\n", rc, reason); | ||
654 | return rc; | 651 | return rc; |
655 | } | 652 | } |
656 | 653 | ||
657 | static int ahci_hardreset(struct ata_port *ap, int verbose, unsigned int *class) | 654 | static int ahci_hardreset(struct ata_port *ap, unsigned int *class) |
658 | { | 655 | { |
659 | int rc; | 656 | int rc; |
660 | 657 | ||
661 | DPRINTK("ENTER\n"); | 658 | DPRINTK("ENTER\n"); |
662 | 659 | ||
663 | ahci_stop_engine(ap); | 660 | ahci_stop_engine(ap); |
664 | rc = sata_std_hardreset(ap, verbose, class); | 661 | rc = sata_std_hardreset(ap, class); |
665 | ahci_start_engine(ap); | 662 | ahci_start_engine(ap); |
666 | 663 | ||
667 | if (rc == 0) | 664 | if (rc == 0) |