> On Wed, 2007-06-27 at 23:14 +0200, Michal Schmidt wrote: > Hello, > > this is a series of four patches for the airo driver: > > 1/4: airo: don't use the interface name so much before registration > 2/4: airo: delay parts of initialization until the netdev is up > 3/4: airo: disable the PCI device when unloading module > 4/4: airo: start with radio off > > The first two fix the race condition Matteo Croce was experiencing. > The third one fixes a bug I noticed during testing. > The last one saves some power when the card is not used. I have applied your 4 patches on both a vanilla 2.6.21.5 kernel and a 2.6.22-rc6 wireless-dev one, and i found that only patches 1/4 and 2/4 are good. the 3/4 one gives this bogus output: airo(): Probing for PCI adapters PCI: Setting latency timer of device 0000:05:08.0 to 64 airo(eth%d): cmd:111 status:7f11 rsp0:2 rsp1:0 rsp2:0 airo(eth%d): Doing fast bap_reads airo(): WPA is supported. airo(eth0): MAC enabled 0:d:29:4f:c:be airo(): Finished probing for PCI adapters and the 4/4 definitely broke all: airo(): Probing for PCI adapters PCI: Setting latency timer of device 0000:05:08.0 to 64 airo(eth%d): cmd:111 status:7f11 rsp0:2 rsp1:0 rsp2:0 airo(eth%d): Doing fast bap_reads airo(eth%d): Bad MAC enable reason = 2, rid = 0, offset = 0 airo(eth%d): MAC could not be enabled airo(): Finished probing for PCI adapters - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
