Return-path: Received: from webhosting01.bon.m2soft.com ([195.38.20.32]:44144 "EHLO webhosting01.bon.m2soft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750962Ab0JYNbT (ORCPT ); Mon, 25 Oct 2010 09:31:19 -0400 Date: Mon, 25 Oct 2010 15:30:03 +0200 From: Nicolas Kaiser To: Luciano Coelho Cc: Larry Finger , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] wireless/wl1271: remove redundant if-statement v2 Message-ID: <20101025153003.3aa7c168@absol.kitzblitz> In-Reply-To: <1288012245.21130.24.camel@chilepepper> References: <20101025000744.17f7086e@absol.kitzblitz> <4CC4B210.2010608@lwfinger.net> <1288012245.21130.24.camel@chilepepper> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-wireless-owner@vger.kernel.org List-ID: wl1271_ps_elp_sleep() is void and cannot return a value. Signed-off-by: Nicolas Kaiser --- 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"); -- 1.7.2.2