From: Matteo Croce <rootkit85@yahoo.it>
To: Michal Schmidt <mschmidt@redhat.com>
Subject: Re: [PATCH 0/4] airo fixes
Date: Thu, 28 Jun 2007 15:51:37 +0200
Cc: linux-wireless@vger.kernel.org
MIME-Version: 1.0
Content-Type: Text/Plain;
  charset="iso-8859-1"
Message-Id: <200706281551.37334.rootkit85@yahoo.it>
Sender: linux-wireless-owner@vger.kernel.org

> 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