On Wed, 2011-05-11 at 11:54 +0300, Eliad Peller wrote: > Signal when psm was entered successfully, if the completion > variable is being set. > > Signed-off-by: Eliad Peller <eliad@xxxxxxxxxx> > --- > drivers/net/wireless/wl12xx/event.c | 7 +++++++ > drivers/net/wireless/wl12xx/wl12xx.h | 1 + > 2 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/drivers/net/wireless/wl12xx/event.c b/drivers/net/wireless/wl12xx/event.c > index ae69330..21e52ed 100644 > --- a/drivers/net/wireless/wl12xx/event.c > +++ b/drivers/net/wireless/wl12xx/event.c > @@ -135,6 +135,13 @@ static int wl1271_event_ps_report(struct wl1271 *wl, > > /* enable beacon early termination */ > ret = wl1271_acx_bet_enable(wl, true); > + if (ret < 0) > + break; > + > + if (wl->ps_compl) { > + complete(wl->ps_compl); > + wl->ps_compl = NULL; > + } > break; > default: > break; > diff --git a/drivers/net/wireless/wl12xx/wl12xx.h b/drivers/net/wireless/wl12xx/wl12xx.h > index daf941d..cf08a9d 100644 > --- a/drivers/net/wireless/wl12xx/wl12xx.h > +++ b/drivers/net/wireless/wl12xx/wl12xx.h > @@ -512,6 +512,7 @@ struct wl1271 { > unsigned int rx_filter; > > struct completion *elp_compl; > + struct completion *ps_compl; > struct delayed_work elp_work; > struct delayed_work pspoll_work; > This patch doesn't make sense by itself, without the next one. Can you squash them? -- Cheers, Luca. -- 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