Re: sis190

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

Francois Romieu wrote:
> Andrew Hutchings <info@a-wing.co.uk> :
> [...]
> 
>>Tried it, it didn't detect the sis190 in this board so I changed the 
>>PCI_ID lines to:
>>static struct pci_device_id sis190_pci_tbl[] __devinitdata = {
>>   { 0x1039, 0x0190, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0 },
>>   { 0,},
>>};
> 
> 
> /me scratches head
> 
> The figures look the same. I must be tired.
> 

I thought so, but for some reason it makes a difference.

> 
>>This then detected it but caused a soft lockup on modprobe with a dump 
>>which I have attached here.  I have also attched lspci -vvv and -xxx for 
>>you.
> 
> 
> Right, there was a bug. I am not sure the fix will really fix though.
> 
> See the patch of the day:
> 
> http://www.fr.zoreil.com/people/francois/misc/20050610-2.6.12-rc-sis190-test.patch
> 
> Please add a dmesg as well if something goes wrong.
> 

Something went wrong on build.  Getting 'syntax error before '}' token' 
on every line there is _msleep(1);

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 311: transient bus protocol violation