cvs commit: src/sys/dev/ipw if_ipw.c

Sam Leffler sam at errno.com
Sun May 22 23:45:15 GMT 2005


Nate Lawson wrote:
> Damien Bergamini wrote:
> 
>>> Why?  A good freebsd commit msg explains why you make a change.
>>
>>
>>
>> Basically because the Linux driver does it and because some users
>> reported that this fixes issues with ACPI.  Remember that I have
>> zero documentation for this chipset except the Linux driver itself
>> which is very poorly commented.  And Intel refuses to answer to
>> any technical question regarding the hardware.
> 
> 
> It's not an ACPI issue.  You would have the problem with any 
> suspend/resume (i.e., APM).
> 

I think he just meant "issues seen when ACPI is enabled".  It appears 
this change originated in one of the Linux ethernet drivers (can't 
recall) and has been blindly propagated to other drivers because it 
works. I applied it when certain 5211 parts started to get PCI bus dma 
abort errors. Disabling retries stopped it but I was never able to get a 
logic analyzer in place to see exactly what was happening (this was a 
cardbus card).

	Sam


More information about the cvs-src mailing list