Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Sender: kvalo=codeaurora.org@mg.codeaurora.org
DMARC-Filter: OpenDMARC Filter v1.4.1 smtp.codeaurora.org 97F55C4338F
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [PATCH] wcn36xx: Implement Idle Mode Power Save
From:   Kalle Valo <kvalo@codeaurora.org>
In-Reply-To: <20210909153320.2624649-1-bryan.odonoghue@linaro.org>
References: <20210909153320.2624649-1-bryan.odonoghue@linaro.org>
To:     Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Cc:     linux-wireless@vger.kernel.org, wcn36xx@lists.infradead.org,
        loic.poulain@linaro.org, benl@squareup.com,
        bryan.odonoghue@linaro.org
User-Agent: pwcli/0.1.0-git (https://github.com/kvalo/pwcli/) Python/3.7.3
Message-Id: <20210921133444.A7D1EC43616@smtp.codeaurora.org>
Date:   Tue, 21 Sep 2021 13:34:44 +0000 (UTC)
Precedence: bulk

Bryan O'Donoghue <bryan.odonoghue@linaro.org> wrote:

> Idle Mode Power Save (IMPS) is a power saving mechanism which when called
> by wcn36xx will cause the radio hardware to enter power collapse.
> 
> This particular call maps nicely to a simple conjunction/disjunction around
> IEEE80211_CONF_CHANGE_IDLE and IEEE80211_CONF_IDLE.
> 
> Here we enter idle when we are not associated with an AP. The kernel will
> incrementally toggle idle on/off in the process of trying to establish a
> connection, thus saving power until we are connected to the AP again, at
> which point we give way to BMPS if power_save is on.
> 
> We've validated that with IMPS an apq8039 device which has the wcn36xx
> module loaded but, has not authenticated with an AP will get to VMIN on
> suspend and will not without IMPS.
> 
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> Tested-by: Benjamin Li <benl@squareup.com>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

Patch applied to ath-next branch of ath.git, thanks.

0e159d2c0834 wcn36xx: Implement Idle Mode Power Save

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20210909153320.2624649-1-bryan.odonoghue@linaro.org/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches