From: Fabio Rossi <rossi.f@inwind.it>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Subject: Re: WARNING: at drivers/base/core.c:130 device_release+0x82/0x90()
Date: Sun, 6 Jun 2010 11:43:56 +0200
Cc: linux-wireless@vger.kernel.org
References: <201006021129.43621.rossi.f@inwind.it> <AANLkTim6tx9plm0tqqms64iUboSRxo9KnTGbHmwKrhQK@mail.gmail.com> <AANLkTimTGPFL_1qAn4YEtYPogOkbZIr_C1YHOtKoHzxw@mail.gmail.com>
In-Reply-To: <AANLkTimTGPFL_1qAn4YEtYPogOkbZIr_C1YHOtKoHzxw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: Text/Plain;
  charset="utf-8"
Message-Id: <201006061143.56259.rossi.f@inwind.it>
Sender: linux-wireless-owner@vger.kernel.org

On Sunday 06 June 2010 11:29:54 Luis R. Rodriguez wrote:

> On Sun, Jun 6, 2010 at 2:28 AM, Luis R. Rodriguez <mcgrof@gmail.com> wrote:
> > On Fri, Jun 4, 2010 at 1:25 AM, Fabio Rossi <rossi.f@inwind.it> wrote:
> >> On Wednesday 02 June 2010 23:42:14 Luis R. Rodriguez wrote:
> >>> On Wed, Jun 2, 2010 at 2:29 AM, Fabio Rossi <rossi.f@inwind.it> wrote:
> >>> > Compiling the kernel without ACPI (CONFIG_ACPI=n) gives the following
> >>> > warning:
> >>> > 
> >>> > cfg80211: Calling CRDA to update world regulatory domain
> >>> > cfg80211: World regulatory domain updated:
> >>> >    (start_freq - end_freq @ bandwidth), (max_antenna_gain, max_eirp)
> >>> >    (2402000 KHz - 2472000 KHz @ 40000 KHz), (300 mBi, 2000 mBm)
> >>> > ath5k 0000:08:01.0: PCI INT A -> GSI 17 (level, low) -> IRQ 17
> >>> > ath5k 0000:08:01.0: registered as 'phy0'
> >>> > ath5k phy0: request_irq failed
> >>> > ------------[ cut here ]------------
> >>> 
> >>> Since traces are printed reversed you will have to read my logic from
> >>> bottom to top:
> >>> 
> >>> Seems we need to implement the dev_release() op for the class of our
> >>> devices.
> >> 
> >> Shall I open a bug in kernel bugzilla to keep trace of this?
> > 
> > Nah something else is up...
> > 
> > struct class ieee80211_class = {
> >        .name = "ieee80211",
> >        .owner = THIS_MODULE,
> >        .dev_release = wiphy_dev_release,
> >        .dev_attrs = ieee80211_dev_attrs,
> > #ifdef CONFIG_HOTPLUG
> >        .dev_uevent = wiphy_uevent,
> > #endif
> >        .suspend = wiphy_suspend,
> >        .resume = wiphy_resume,
> > };
> 
> This is wireless-testing, not compat-wireless right?

Yes, I'm using the latest wireless-testing.git

Fabio