From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: [PATCH 2/5] sata_sil24: better error message from softreset Date: Fri, 7 Apr 2006 17:02:47 +0900 Message-ID: <1144396967527-git-send-email-htejun@gmail.com> References: <11443969673485-git-send-email-htejun@gmail.com> Reply-To: Tejun Heo Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT Return-path: Received: from wproxy.gmail.com ([64.233.184.237]:28115 "EHLO wproxy.gmail.com") by vger.kernel.org with ESMTP id S932336AbWDGICy (ORCPT ); Fri, 7 Apr 2006 04:02:54 -0400 Received: by wproxy.gmail.com with SMTP id i28so312986wra for ; Fri, 07 Apr 2006 01:02:53 -0700 (PDT) In-Reply-To: <11443969673485-git-send-email-htejun@gmail.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: jgarzik@pobox.com, linux-ide@vger.kernel.org Cc: Tejun Heo Improve softreset error message. Signed-off-by: Tejun Heo --- drivers/scsi/sata_sil24.c | 12 ++++++++++-- 1 files changed, 10 insertions(+), 2 deletions(-) 7b901b3ebd190cbab8424b599221da795375f892 diff --git a/drivers/scsi/sata_sil24.c b/drivers/scsi/sata_sil24.c index 512ee6f..c75c228 100644 --- a/drivers/scsi/sata_sil24.c +++ b/drivers/scsi/sata_sil24.c @@ -435,6 +435,7 @@ static int sil24_softreset(struct ata_po dma_addr_t paddr = pp->cmd_block_dma; unsigned long timeout = jiffies + ATA_TMOUT_BOOT; u32 irq_enable, irq_stat; + const char *reason; DPRINTK("ENTER\n"); @@ -474,8 +475,11 @@ static int sil24_softreset(struct ata_po writel(irq_enable, port + PORT_IRQ_ENABLE_SET); if (!(irq_stat & PORT_IRQ_COMPLETE)) { - printk(KERN_ERR "ata%u: softreset failed (timeout)\n", ap->id); - return -EIO; + if (irq_stat & PORT_IRQ_ERROR) + reason = "SRST command error"; + else + reason = "timeout"; + goto err; } sil24_update_tf(ap); @@ -487,6 +491,10 @@ static int sil24_softreset(struct ata_po out: DPRINTK("EXIT, class=%u\n", *class); return 0; + + err: + printk(KERN_ERR "ata%u: softreset failed (%s)\n", ap->id, reason); + return -EIO; } static int sil24_hardreset(struct ata_port *ap, unsigned int *class) -- 1.2.4