Re: sis5513.c patch

[Andrew Hutchings]

Bartlomiej Zolnierkiewicz wrote:
> [ Andrew, please remove this patch from -mm queue. ]
>
> On 6/8/05, Bartlomiej Zolnierkiewicz <bzolnier@xxxxxxxxx> wrote:
>
> > On 6/8/05, Andrew Hutchings <info@xxxxxxxxxxxx> wrote:
> >
> > > Bartlomiej Zolnierkiewicz wrote:
> > >
> > > > On 6/8/05, Andrew Hutchings <info@xxxxxxxxxxxx> 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.
>
>
> Second look into sis5513.c and another problem turns out - patch breaks 
> support for IDE controllers integrated into 961 and 961B South Bridges
> (ATA_133 is used instead of ATA_100 and ATA133a).
>
> For unknown Host Bridges driver checks for presence of 961/961B/962/963
> South Bridges by checking true device ID (please see sis5513.c for details) 
> and assigns 'chipset_family' accordingly (ATA_100/ATA_133a or ATA_133).
>
> You have 965L South Bridge so probably it has newer true device ID 
> and  may also require different programming sequence.

Ah, you see this is what happens when I enter the scary world of driver 
patching, I'll will stick to patching/developing apps in future I think ;)

> > Could you please send full lspci -vvv output?
>
>
> and lspci -xxx
>
> Bartlomiej

I'm just installing another one of those boards now ready to go into our 
racks later today, I'll give you the output from that shortly.
--
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 24: network packets travelling uphill (use a carrier pigeon)
-
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/