Re: sis5513.c patch

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

Bartlomiej Zolnierkiewicz wrote:
> [ Andrew, please remove this patch from -mm queue. ]
> 
> On 6/8/05, Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> wrote:
> 
>>On 6/8/05, Andrew Hutchings <info@a-wing.co.uk> wrote:
>>
>>>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.
> 
> 
> 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)