Return-path: Received: from mail-ob0-f175.google.com ([209.85.214.175]:35267 "EHLO mail-ob0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932707AbbDORKP (ORCPT ); Wed, 15 Apr 2015 13:10:15 -0400 Received: by obbfy7 with SMTP id fy7so27479555obb.2 for ; Wed, 15 Apr 2015 10:10:14 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: References: <1428911141-6534-1-git-send-email-janusz.dziedzic@tieto.com> <1428911141-6534-2-git-send-email-janusz.dziedzic@tieto.com> Date: Wed, 15 Apr 2015 10:10:09 -0700 Message-ID: (sfid-20150415_191026_628515_81CB5C80) Subject: Re: [RFC 2/2] ath10k: don't disable PS when not connected From: YanBo To: Janusz Dziedzic Cc: "ath10k@lists.infradead.org" , linux-wireless Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, Apr 14, 2015 at 10:00 PM, Janusz Dziedzic wrote: > On 15 April 2015 at 00:45, YanBo wrote: >> On Mon, Apr 13, 2015 at 12:45 AM, Janusz Dziedzic >> wrote: >>> Don't disable PS while we are not connected. >>> In other case we will get higher power consumption. >>> >>> Signed-off-by: Janusz Dziedzic >>> --- >>> drivers/net/wireless/ath/ath10k/mac.c | 8 +++++++- >>> 1 file changed, 7 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c >>> index 52c5b1f..b896dd4 100644 >>> --- a/drivers/net/wireless/ath/ath10k/mac.c >>> +++ b/drivers/net/wireless/ath/ath10k/mac.c >>> @@ -1730,7 +1730,13 @@ static int ath10k_mac_vif_setup_ps(struct ath10k_vif *arvif) >>> enable_ps = false; >>> } >>> >>> - if (enable_ps) { >>> + if (!arvif->is_started) { >>> + /* enable power save mode while not connected, >>> + * in other case after iface up we will get >>> + * higher power consumption - firmware design >>> + */ >>> + psmode = WMI_STA_PS_MODE_ENABLED; >>> + } else if (enable_ps) { >>> psmode = WMI_STA_PS_MODE_ENABLED; >>> param = WMI_STA_PS_PARAM_INACTIVITY_TIME; >>> >>> -- >> >> What the expectation behavior after we enable the >> WMI_STA_PS_MODE_ENABLED at Idle status? >> Is there any effect for TX or RX chain after set it? >> > > First I think that WMI_STA_PS_MODE_ENABLED is important only when we > are connected. > But, I see current consumption drop in my test environtment from 88mA > to 33mA when: > 1) load driver, iface up > 2) disconnect network, iface down, iface up > So, seems WMI_STA_PS_MODE_ENABLED do something more in FW (not only > standard PS enable/disable when > connected to AP). Probably someone from FW team should answer that, if > that is a feature or a bug. > I guess the chip enter Idle mode power save after set this, did you run the the test on QCA61XX or QCA 98XX? Can it still auto scan to get all the APs around it, cause for IMPS mode it will only looks in its existing profile list and sends a probe request, with SSID specified in profile for what I know BR /Yanbo