Search Linux Wireless

Re: IWLWIFI should depend on SUSPEND

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, 2012-05-19 at 22:19 +0200, Sebastian Kemper wrote:
> Hello everybody,
> 
> iwlwifi doesn't start without suspend support in the kernel. The last
> thing it says when suspend is missing is this:
> 
> iwlwifi 0000:03:00.0: Failed to register hw (error -22)
> iwlwifi 0000:03:00.0: PCI INT A disabled

I was going to say it's a mac80211 bug due to this code:

        if ((hw->wiphy->wowlan.flags || hw->wiphy->wowlan.n_patterns)
#ifdef CONFIG_PM
            && (!local->ops->suspend || !local->ops->resume)
#endif
            )
                return -EINVAL;

But then I thought about it and think it makes sense, if you don't have
suspend then you can't have WoWLAN, so the driver shouldn't advertise it
in that case, which makes it an iwlwifi bug again, we should ifdef the
wowlan support correctly.

OTOH, we could also just make the wowlan stuff not compile if CONFIG_PM
isn't set by adding an ifdef in cfg80211... That might be better so
others don't run into this trap.

Thoughts?

johannes

--
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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux