Return-path: Received: from s3.sipsolutions.net ([144.76.63.242]:58210 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935299AbeFRUqR (ORCPT ); Mon, 18 Jun 2018 16:46:17 -0400 Message-ID: <1529354772.3092.41.camel@sipsolutions.net> (sfid-20180618_224640_892827_764076DF) Subject: Re: [PATCH] cfg80211: Add cumulative channel survey dump support. From: Johannes Berg To: Ben Greear , vnaralas@codeaurora.org Cc: linux-wireless@vger.kernel.org, linux-wireless-owner@vger.kernel.org Date: Mon, 18 Jun 2018 22:46:12 +0200 In-Reply-To: <7afee956-9f91-37f5-52d2-7cdc0686f213@candelatech.com> References: <1526980556-26707-1-git-send-email-vnaralas@codeaurora.org> <1527069282.3759.16.camel@sipsolutions.net> <7afee956-9f91-37f5-52d2-7cdc0686f213@candelatech.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, 2018-05-31 at 08:52 -0700, Ben Greear wrote: > > On 05/31/2018 02:06 AM, vnaralas@codeaurora.org wrote: > > On 2018-05-23 15:24, Johannes Berg wrote: > > > On Tue, 2018-05-22 at 14:45 +0530, Venkateswara Naralasetty wrote: > > > > This patch provides support to send accumulated survey data to > > > > user if low level drivers provides non-accumulated survey data. > > > > > > I think the commit log should say what you need this for? > > > > > > It's simultaneously a new flag, and a lot of code, but it's not clear > > > what the point is? > > > > > > johannes > > > > I will sent next version of patch with updated commit log. > > > > Providing you the earlier discussion of this patch to give quick overview about this patch. > > https://patchwork.kernel.org/patch/9701459/ > > It is simple to fix the firmware, it just has several bugs related to clearing the > accumulator as well as the 'real' values. If you can find a QCA firmware engineer > that will accept patches I can show them how to fix this easily. I recently > fixed this in my wave-1 firmware. I posted patches to the ath10k driver to > support this a week or so ago, but not sure if Kalle will apply them. That would be nicer even, but I guess we still have to worry about older firmware? Perhaps not for survey support? Dunno. Obviously less code is better, but I can't really say how easy it would be to get firmware updates rolled out to people who'd need them ... > And, it is likely that even if you don't use the 'clear' option, you are going to > get wraps in the firmware and that will just be other harder to debug bugs. You have to use clear if you don't accumulate completely in fw, no? johannes