diff options
| author | Jeff Garzik <jgarzik@pobox.com> | 2004-07-09 15:58:35 -0400 |
|---|---|---|
| committer | Jeff Garzik <jgarzik@pobox.com> | 2004-07-09 15:58:35 -0400 |
| commit | ab70628a81cb17fd0e8fbb5fa04e788649f80e6f (patch) | |
| tree | 4f3c9fe91d5115c79c2254443820372b099247f5 | |
| parent | 767769553d675e81311ac8a1086f787bfa11fd9a (diff) | |
[PCI, libata] Fix "combined mode" PCI quirk for ICH6
The hardware vendor, in their infinite wisdom, make the combined
mode configuration register different between ICH5 and ICH6.
Take this into account.
| -rw-r--r-- | drivers/pci/quirks.c | 31 |
1 files changed, 24 insertions, 7 deletions
diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 0ccd6857d508..a92dee84b4bc 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -817,6 +817,7 @@ static void __init quirk_alder_ioapic(struct pci_dev *pdev) static void __init quirk_intel_ide_combined(struct pci_dev *pdev) { u8 prog, comb, tmp; + int ich = 0; /* * Narrow down to Intel SATA PCI devices. @@ -827,8 +828,12 @@ static void __init quirk_intel_ide_combined(struct pci_dev *pdev) case 0x24df: case 0x25a3: case 0x25b0: + ich = 5; + break; case 0x2651: case 0x2652: + case 0x2653: + ich = 6; break; default: /* we do not handle this PCI device */ @@ -839,13 +844,25 @@ static void __init quirk_intel_ide_combined(struct pci_dev *pdev) * Read combined mode register. */ pci_read_config_byte(pdev, 0x90, &tmp); /* combined mode reg */ - tmp &= 0x6; /* interesting bits 2:1, PATA primary/secondary */ - if (tmp == 0x4) /* bits 10x */ - comb = (1 << 0); /* SATA port 0, PATA port 1 */ - else if (tmp == 0x6) /* bits 11x */ - comb = (1 << 2); /* PATA port 0, SATA port 1 */ - else - return; /* not in combined mode */ + + if (ich == 5) { + tmp &= 0x6; /* interesting bits 2:1, PATA primary/secondary */ + if (tmp == 0x4) /* bits 10x */ + comb = (1 << 0); /* SATA port 0, PATA port 1 */ + else if (tmp == 0x6) /* bits 11x */ + comb = (1 << 2); /* PATA port 0, SATA port 1 */ + else + return; /* not in combined mode */ + } else { + WARN_ON(ich != 6); + tmp &= 0x3; /* interesting bits 1:0 */ + if (tmp & (1 << 0)) + comb = (1 << 2); /* PATA port 0, SATA port 1 */ + else if (tmp & (1 << 1)) + comb = (1 << 0); /* SATA port 0, PATA port 1 */ + else + return; /* not in combined mode */ + } /* * Read programming interface register. |