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;
Message-ID: <93ad9a422ea97bd4cca1c07f6088ba9025adbb34.camel@sipsolutions.net>
Subject: Re: [PATCH v3 1/2] mac80211_hwsim: enable parent TSF feature
From:   Johannes Berg <johannes@sipsolutions.net>
To:     James Prestwood <prestwoj@gmail.com>,
        linux-wireless@vger.kernel.org
Date:   Fri, 13 Dec 2019 10:53:17 +0100
In-Reply-To: <20191122171401.8788-1-prestwoj@gmail.com> (sfid-20191122_181704_319217_3F437F0C)
References: <20191122171401.8788-1-prestwoj@gmail.com>
         (sfid-20191122_181704_319217_3F437F0C)
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.34.2 (3.34.2-1.fc31) 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

On Fri, 2019-11-22 at 09:14 -0800, James Prestwood wrote:
> Support for this is already in nl80211, and some code paths in
> mac80211_hwsim already set the rx_status to allow the parent TSF
> value to be sent to user space.

This really should come *after* the second patch, not before.

I'd have reordered it, but now that you need to fix the second patch
anyway it'll be easier if you also reorder and reword this.

johannes