Re: sis5513.c patch

From: Bartlomiej Zolnierkiewicz
Date: Wed Jun 08 2005 - 02:28:50 EST


On 6/8/05, Andrew Hutchings <info@xxxxxxxxxxxx> wrote:
> Hi,

Hi,

> I'm not sure if a similar patch has been submitted or not, but here is a
> patch to get DMA working on ASUS K8S-MX with a SiS 760GX/SiS 965L
> chipset combo.

This patch is incorrect, it adds PCI ID of SiS IDE controller (this ID
is common for almost all SiS IDE controllers and is already present in
sis5513_pci_tbl[]) to the table of SiS Host PCI IDs. As a result driver
will try to use ATA_133 on all _unknown_ IDE controllers. You need
to add PCI ID of the Host chipset (lspci should reveal it) instead.

Thanks,
Bartlomiej
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/