From: Tejun Heo <htejun@gmail.com>
To: Jeff Garzik <jgarzik@pobox.com>,
kaos@ocs.com.au, stevenm@umd.edu, linux-ide@vger.kernel.org
Subject: [PATCH] ata_piix: ignore PCS on ICH5
Date: Tue, 22 Aug 2006 21:06:46 +0900 [thread overview]
Message-ID: <20060822120646.GA21866@htj.dyndns.org> (raw)
There have been a number of reports regarding some ICH5s failing to
detect devices since the PCS handling update. Analysis shows that
these problems are caused by bogus PCS values from those controllers.
Before the PCS update, the driver didn't honor PCS regs exactly and
probed them in many cases PCS reports no device. Now that PCS is
honored exactly, these hardware problems are visible.
This patch makes ICH5 ignore PCS.
Signed-off-by: Tejun Heo <htejun@gmail.com>
---
Keith Owens, Steve and Andre, please test this patch and report how
this works for you guys. If possible, please test with several
different setups. Some ICHs (7 and 8) suffer from long boot delays
due to ghost device if PCS is not honored and we want to make sure
that ICH5 doesn't have that problem.
ICH7 and 8's choke if the master device is ATAPI and the slave slot is
empty, so testing this case on ICH5 would be especially helpful.
e.g. If your ata_piix reports [P0 P1 IDE IDE], connect SATA ATAPI
device to SATA port 0 and don't connect anything to P1, boot and see
if things work okay.
This patch is against libata-dev#upstream-fixes and should apply to
2.6.18-rc4 and the current linus #master.
Thanks.
diff --git a/drivers/scsi/ata_piix.c b/drivers/scsi/ata_piix.c
index 01b3530..3b98f18 100644
--- a/drivers/scsi/ata_piix.c
+++ b/drivers/scsi/ata_piix.c
@@ -390,7 +390,8 @@ #endif
/* ich5_sata */
{
.sht = &piix_sht,
- .host_flags = ATA_FLAG_SATA | PIIX_FLAG_CHECKINTR,
+ .host_flags = ATA_FLAG_SATA | PIIX_FLAG_CHECKINTR |
+ PIIX_FLAG_IGNORE_PCS,
.pio_mask = 0x1f, /* pio0-4 */
.mwdma_mask = 0x07, /* mwdma0-2 */
.udma_mask = 0x7f, /* udma0-6 */
next reply other threads:[~2006-08-22 12:06 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-08-22 12:06 Tejun Heo [this message]
2006-08-23 2:04 ` [PATCH] ata_piix: ignore PCS on ICH5 Keith Owens
2006-08-24 6:52 ` Jeff Garzik
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=20060822120646.GA21866@htj.dyndns.org \
--to=htejun@gmail.com \
--cc=jgarzik@pobox.com \
--cc=kaos@ocs.com.au \
--cc=linux-ide@vger.kernel.org \
--cc=stevenm@umd.edu \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).