On Thu, May 12, 2011 at 11:10 PM, Luciano Coelho <coelho@xxxxxx> wrote: > 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? > sure. Eliad. -- 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