Received-SPF: pass (google.com: best guess record for domain of linux-wireless-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org D32CFC447A1
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [RESEND 1/9] ath11k: add wmi helper for turning STA PS on/off
From:   Kalle Valo <kvalo@codeaurora.org>
In-Reply-To: <20191204053713.3064-2-john@phrozen.org>
References: <20191204053713.3064-2-john@phrozen.org>
To:     John Crispin <john@phrozen.org>
Cc:     linux-wireless@vger.kernel.org, ath11k@lists.infradead.org,
        John Crispin <john@phrozen.org>
User-Agent: pwcli/0.0.0-git (https://github.com/kvalo/pwcli/) Python/2.7.12
Message-ID: <0101016ef0e10c73-a684c43b-a9f3-4115-a3c2-142e74aa25ea-000000@us-west-2.amazonses.com>
Date:   Tue, 10 Dec 2019 17:35:11 +0000
Feedback-ID: 1.us-west-2.CZuq2qbDmUIuT3qdvXlRHZZCpfZqZ4GtG9v3VKgRyF0=:AmazonSES
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

John Crispin <john@phrozen.org> wrote:

> Add a WMI call helper to set the powersave mode of a STA interface.
> 
> Signed-off-by: John Crispin <john@phrozen.org>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

There were few new warnings:

drivers/net/wireless/ath/ath11k/debugfs_sta.c: In function 'ath11k_update_per_peer_stats_from_txcompl':
drivers/net/wireless/ath/ath11k/debugfs_sta.c:138:6: warning: variable 'num_tones_in_ru' set but not used [-Wunused-but-set-variable]
drivers/net/wireless/ath/ath11k/debug.c:122: line over 90 characters
drivers/net/wireless/ath/ath11k/dp_tx.c:472: line over 90 characters
drivers/net/wireless/ath/ath11k/wmi.c:5669: char * array declaration might be better as static const

-- 
https://patchwork.kernel.org/patch/11272115/

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