From: Albert Lee <albertcc@tw.ibm.com>
To: Jonathan Blake Benson <airbatica@verizon.net>
Cc: linux-ide@vger.kernel.org, Jeff Garzik <jgarzik@pobox.com>,
Tejun Heo <htejun@gmail.com>,
Carlos Pardo <Carlos.Pardo@siliconimage.com>,
Doug Maxey <dwm@maxeymade.com>
Subject: Re: libata machine check on Alpha
Date: Thu, 06 Apr 2006 17:17:37 +0800 [thread overview]
Message-ID: <4434DCB1.7010909@tw.ibm.com> (raw)
In-Reply-To: <r4p732p2j7rlui8umeqrq9hothsl79n3bo@4ax.com>
Jonathan Blake Benson wrote:
>
> Albert, enabling the ATAPI_ENABLE_DMADIR workaround did the trick.
> The DVD-ROM drive now works as it should. Thanks again!
>
Hi Jonathan,
Could you please give a try to the attached patch and post the content of word 62?
If word 62 + SiI 3611 looks good, we can turn on the DMADIR automatically. Thanks!
--
albert
--- upstream0/drivers/scsi/libata-core.c 2006-04-06 16:07:32.000000000 +0800
+++ upstream0-show/drivers/scsi/libata-core.c 2006-04-06 17:08:10.000000000 +0800
@@ -1322,6 +1322,9 @@ static int ata_dev_configure(struct ata_
}
dev->cdb_len = (unsigned int) rc;
+ printk(KERN_ERR "ata%u: dev %u word 62 [%x]\n",
+ ap->id, dev->devno, id[62]);
+
/* print device info to dmesg */
if (print_info)
printk(KERN_INFO "ata%u: dev %u ATAPI, max %s\n",
next prev parent reply other threads:[~2006-04-06 9:17 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-03 18:56 libata machine check on Alpha Jonathan Blake Benson
2006-04-04 1:43 ` Tejun Heo
2006-04-04 2:12 ` Albert Lee
2006-04-04 2:15 ` Tejun Heo
2006-04-04 2:49 ` Albert Lee
2006-04-04 3:05 ` Tejun Heo
2006-04-04 8:24 ` Jeff Garzik
2006-04-04 2:57 ` [PATCH] libata: convert ATAPI_ENABLE_DMADIR to module parameter Albert Lee
2006-04-04 3:07 ` Tejun Heo
2006-04-04 12:44 ` Jeff Garzik
2006-04-05 15:48 ` libata machine check on Alpha Jonathan Blake Benson
2006-04-06 9:17 ` Albert Lee [this message]
-- strict thread matches above, loose matches on Subject: below --
2006-04-04 17:12 Carlos Pardo
2006-04-06 2:04 ` Albert Lee
2006-04-06 2:17 ` Jeff Garzik
2006-04-06 2:31 ` Tejun Heo
2006-04-06 6:38 ` Albert Lee
2006-04-06 6:44 ` Doug Maxey
2006-04-06 7:14 ` Albert Lee
2006-04-05 0:14 Jonathan Benson
2006-04-07 2:01 Jonathan Benson
2006-04-07 6:18 ` Albert Lee
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4434DCB1.7010909@tw.ibm.com \
--to=albertcc@tw.ibm.com \
--cc=Carlos.Pardo@siliconimage.com \
--cc=airbatica@verizon.net \
--cc=albertl@mail.com \
--cc=dwm@maxeymade.com \
--cc=htejun@gmail.com \
--cc=jgarzik@pobox.com \
--cc=linux-ide@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.