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

["John W. Linville" <linville@tuxdriver.com>]

On Sat, Mar 21, 2009 at 08:59:43AM +0100, Johannes Berg wrote:
> 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.

I fixed this in wireless-testing.  Feel free to change it as you like it.

-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.