git: d6616705231c - main - [net80211] Quieten the logging from ieee80211_vht_get_vhtflags()

From: Adrian Chadd <adrian_at_FreeBSD.org>
Date: Mon, 08 Sep 2025 00:12:54 UTC
The branch main has been updated by adrian:

URL: https://cgit.FreeBSD.org/src/commit/?id=d6616705231ce5a90068136f1aae05c143e8c0eb

commit d6616705231ce5a90068136f1aae05c143e8c0eb
Author:     Adrian Chadd <adrian@FreeBSD.org>
AuthorDate: 2025-08-25 00:50:01 +0000
Commit:     Adrian Chadd <adrian@FreeBSD.org>
CommitDate: 2025-09-08 00:11:00 +0000

    [net80211] Quieten the logging from ieee80211_vht_get_vhtflags()
    
    The commit in Fixes: introduced logging the output bits from
    ieee80211_vht_get_vhtflags().  This ends up causing quite a lot
    of logging when net80211 is doing things like processing
    received beacons.
    
    So just remove the logging; if it's needed again then a developer
    can add it back to that location, or just use dtrace to capture
    the return value.
    
    Fixes:  4bf049bfeefd9
    Differential Revision: https://reviews.freebsd.org/D52142
    Reviewed by:    bz
---
 sys/net80211/ieee80211_ht.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/net80211/ieee80211_ht.c b/sys/net80211/ieee80211_ht.c
index 018927a6dad4..3af56a228295 100644
--- a/sys/net80211/ieee80211_ht.c
+++ b/sys/net80211/ieee80211_ht.c
@@ -1933,7 +1933,7 @@ ieee80211_vht_get_vhtflags(struct ieee80211_node *ni, uint32_t htflags)
 {
 #define	_RETURN_CHAN_BITS(_cb)						\
 do {									\
-	IEEE80211_NOTE(ni->ni_vap, IEEE80211_MSG_11N, ni,		\
+	if (0) IEEE80211_NOTE(ni->ni_vap, IEEE80211_MSG_11N, ni,	\
 	    "%s:%d: selected %b", __func__, __LINE__,			\
 	    (_cb), IEEE80211_CHAN_BITS);				\
 	return (_cb);							\