linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jonathan Dieter <jdieter@gmail.com>
To: jeff@garzik.org, linux-ide@vger.kernel.org
Cc: fedora-devel-list@redhat.com
Subject: Kernel fix for ICH7M controller
Date: Sun, 03 Sep 2006 17:26:56 +0300	[thread overview]
Message-ID: <44FAE630.20801@gmail.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 991 bytes --]

The ata_piix module doesn't detect that my laptop's disk controller 
(Intel Corporation 82801GBM/GHM - ICH7 Family) is configured to have 
PATA drives in the first two channels.  I looked at Intel's 
documentation and they say that my controller isn't supposed to support 
this configuration, but it's the default setup on this laptop, there's 
no way to change it in the BIOS, and it works in Windows, so I'm 
assuming this is a documentation error.

This patch changes one line in ata_piix.c so that it sees this 
configuration as valid (and therefore recognizes my drives).  I've 
tested it on my laptop and the module is now working perfectly (whereas 
the kernel was using the old IDE driver previously with no DMA enabled).

I'm cc'ing fedora-devel-list because I'm not even sure where this patch 
is supposed to go and I would like it to make it into FC6, as I really 
don't want to have to jump through hoop after hoop just to get FC6 
installed on my hard drive.

Thanks,
Jonathan


[-- Attachment #2: linux-2.6-fix-ich7m-with-pata-on-channel-one.patch --]
[-- Type: text/x-patch, Size: 577 bytes --]

--- linux-2.6.17/drivers/scsi/ata_piix.c	2006-09-03 16:37:16.000000000 +0300
+++ linux-2.6.17-new/drivers/scsi/ata_piix.c	2006-09-03 16:39:14.000000000 +0300
@@ -326,15 +326,15 @@
 static const struct piix_map_db ich6m_map_db = {
 	.mask = 0x3,
 	.port_enable = 0x5,
 	.present_shift = 4,
 	.map = {
 		/* PM   PS   SM   SS       MAP */
 		{  P0,  P2,  RV,  RV }, /* 00b */
-		{  RV,  RV,  RV,  RV },
+		{ IDE, IDE,  P1,  P3 }, /* 01b */
 		{  P0,  P2, IDE, IDE }, /* 10b */
 		{  RV,  RV,  RV,  RV },
 	},
 };
 
 static const struct piix_map_db ich8_map_db = {
 	.mask = 0x3,

             reply	other threads:[~2006-09-03 14:27 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-09-03 14:26 Jonathan Dieter [this message]
2006-09-04  2:47 ` Kernel fix for ICH7M controller Tejun Heo
2006-09-04  5:45   ` Jeff Garzik
2006-09-04  7:34     ` Tejun Heo
2006-09-04  6:53   ` Jonathan Dieter
2006-09-04  8:15     ` Tejun Heo
2006-09-04  9:04       ` Jonathan Dieter
2006-09-10 19:54       ` Jonathan Dieter
2006-09-10 21:29         ` [PATCH] ata_piix: add map 01b for ICH7M Tejun Heo

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=44FAE630.20801@gmail.com \
    --to=jdieter@gmail.com \
    --cc=fedora-devel-list@redhat.com \
    --cc=jeff@garzik.org \
    --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 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).