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;
Message-ID: <d720e6c495aeb8cb6d17cf44b8c15d96de48b61d.camel@sipsolutions.net>
Subject: Re: [PATCH 5/9] cfg80211: report frequency in KHz for management RX
From:   Johannes Berg <johannes@sipsolutions.net>
To:     Thomas Pedersen <thomas@adapt-ip.com>
Cc:     linux-wireless <linux-wireless@vger.kernel.org>
Date:   Fri, 24 Apr 2020 12:11:21 +0200
In-Reply-To: <20200402011810.22947-6-thomas@adapt-ip.com>
References: <20200402011810.22947-1-thomas@adapt-ip.com>
         <20200402011810.22947-6-thomas@adapt-ip.com>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.34.4 (3.34.4-1.fc31) 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

On Wed, 2020-04-01 at 18:18 -0700, Thomas Pedersen wrote:
> Expect and convert drivers to report the RX frequency to
> cfg80211_rx_mgmt() and cfg80211_report_obss_beacon() in
> KHz

I would prefer to have some inlines for this, and add
cfg80211_rx_mgmt_khz() or something.

You've had to change a lot of drivers, but I got conflicts, and you also
missed some (which may have added the code since you created the
patches, or you prepared against mac80211-next and that code was waiting
elsewhere, etc.)

I've applied patches 1-4, but please respin the rest.

johannes