Received-SPF: pass (google.com: domain of linux-kernel+bounces-145371-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Mon, 15 Apr 2024 15:35:10 +0100
From: Simon Horman <horms@kernel.org>
To: Alexander Lobakin <aleksander.lobakin@intel.com>
Cc: "David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	David Ahern <dsahern@kernel.org>,
	Xuan Zhuo <xuanzhuo@linux.alibaba.com>,
	Andrew Lunn <andrew@lunn.ch>,
	nex.sw.ncis.osdt.itp.upstreaming@intel.com, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH RFC net-next 5/7] netdev_features: convert
 NETIF_F_HIGHDMA to priv_flag IFF_HIGHDMA
Message-ID: <20240415143510.GA3545293@kernel.org>
References: <20240405133731.1010128-1-aleksander.lobakin@intel.com>
 <20240405133731.1010128-6-aleksander.lobakin@intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240405133731.1010128-6-aleksander.lobakin@intel.com>

On Fri, Apr 05, 2024 at 03:37:29PM +0200, Alexander Lobakin wrote:
> The ability to send bufferst from the high memory is rather an
> attribute, not a feature. It can't be toggled via Ethtool and
> is constant for the whole interface lifetime.
> Convert it to a private flag, carrying necessary inheritance for
> lower interfaces.
> 
> Signed-off-by: Alexander Lobakin <aleksander.lobakin@intel.com>

..

> index f57c29de3a91..78e749f6b91f 100644
> --- a/include/net/mac80211.h
> +++ b/include/net/mac80211.h
> @@ -3007,6 +3007,7 @@ struct ieee80211_hw {
>  		s16 accuracy;
>  	} radiotap_timestamp;
>  	netdev_features_t netdev_features;
> +	u64 netdev_priv_flags;
>  	u8 uapsd_queues;
>  	u8 uapsd_max_sp_len;
>  	u8 max_nan_de_entries;

nit: please add netdev_features to the kernel-doc for struct ieee80211_hw.

Flagged by ./scripts/kernel-doc -none

..