Re: [PATCH v2] mac80211: resume properly, add suspend/resume test

[Johannes Berg <johannes@sipsolutions.net>]

[-- Attachment #1: Type: text/plain, Size: 871 bytes --]

On Fri, 2009-03-20 at 18:03 +0200, Jouni Malinen wrote:
> On Fri, Mar 13, 2009 at 11:44:18AM +0100, Johannes Berg wrote:
> 
> > --- wireless-testing.orig/net/mac80211/debugfs.c	2009-03-13 11:36:57.000000000 +0100
> > +++ wireless-testing/net/mac80211/debugfs.c	2009-03-13 11:41:53.000000000 +0100
> > +static ssize_t reset_write(struct file *file, const char __user *user_buf,
> > +			   size_t count, loff_t *ppos)
> 
> > +	__ieee80211_suspend(&local->hw);
> > +	__ieee80211_resume(&local->hw);
> 
> This seems to have broken build if CONFIG_PM is not defined: pm.c, with
> these functions, is not included.

Indeed, this file should depend on CONFIG_PM too I suppose, will fix
that. OTOH, a future patch I have will require _resume to always be
present, so I'm inclined to just always put the code in. I'll come up
with a fix in any case.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]