lkml.org 
[lkml]   [2010]   [Oct]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] wireless/wl1271: remove redundant if-statement v2
From
Date
On Mon, 2010-10-25 at 15:30 +0200, ext Nicolas Kaiser wrote:
> wl1271_ps_elp_sleep() is void and cannot return a value.
>
> Signed-off-by: Nicolas Kaiser <nikai@nikai.net>
> ---
> Sorry for the poor wording of the initial description.
>
> drivers/net/wireless/wl12xx/wl1271_event.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/net/wireless/wl12xx/wl1271_event.c b/drivers/net/wireless/wl12xx/wl1271_event.c
> index 7b3f503..38ccef7 100644
> --- a/drivers/net/wireless/wl12xx/wl1271_event.c
> +++ b/drivers/net/wireless/wl12xx/wl1271_event.c
> @@ -134,8 +134,6 @@ static int wl1271_event_ps_report(struct wl1271 *wl,
>
> /* go to extremely low power mode */
> wl1271_ps_elp_sleep(wl);
> - if (ret < 0)
> - break;
> break;
> case EVENT_EXIT_POWER_SAVE_FAIL:
> wl1271_debug(DEBUG_PSM, "PSM exit failed");

Reviewed-by: Luciano Coelho <luciano.coelho@nokia.com>

Thanks! Applied to the wl12xx tree.

--
Cheers,
Luca.



\
 
 \ /
  Last update: 2010-10-26 12:31    [W:0.041 / U:1.168 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site