Re: sis5513.c patch

[Andrew Hutchings <info@a-wing.co.uk>]

Bartlomiej Zolnierkiewicz wrote:
> On 6/8/05, Andrew Hutchings <info@a-wing.co.uk> wrote:
>
> > Hi,
>
>
> Hi,

Hi again,

> > 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.

Unless I am reading the following wrong 5513 is the PCI ID:

00:02.5 Class 0101: 1039:5513 (rev 01) (prog-if 80 [Master])
	Subsystem: 1043:8139
	Flags: bus master, medium devsel, latency 128
	I/O ports at <unassigned>
	I/O ports at <unassigned>
	I/O ports at <unassigned>
	I/O ports at <unassigned>
	I/O ports at ffa0 [size=16]
	Capabilities: [58] Power Management version 2

> Thanks,
> Bartlomiej

Regards
Andrew
--
Andrew Hutchings (A-Wing)
Linux Guru - Netserve Consultants Ltd. - http://www.domaincity.co.uk/
Admin - North Wales Linux User Group - http://www.nwlug.org.uk/
BOFH excuse 424: operation failed because: there is no message for this 
error (#1014)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/