diff mbox series

wifi: mac80211: Drop WBRF debugging statements

Message ID 20240117030525.539-1-mario.limonciello@amd.com (mailing list archive)
State Accepted
Delegated to: Johannes Berg
Headers show
Series wifi: mac80211: Drop WBRF debugging statements | expand

Commit Message

Mario Limonciello Jan. 17, 2024, 3:05 a.m. UTC 
Due to the way that debugging is used in the mac80211 subsystem
this message ends up being noisier than it needs to be.

As the statement is only useful at a first stage of triage for
BIOS bugs, just drop it.

Cc: Jun Ma <Jun.Ma2@amd.com>
Suggested-by: Kalle Valo <kvalo@kernel.org>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
---
 net/mac80211/wbrf.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Kalle Valo Jan. 18, 2024, 9:16 a.m. UTC | #1 
Mario Limonciello <mario.limonciello@amd.com> writes:

> Due to the way that debugging is used in the mac80211 subsystem
> this message ends up being noisier than it needs to be.
>
> As the statement is only useful at a first stage of triage for
> BIOS bugs, just drop it.
>
> Cc: Jun Ma <Jun.Ma2@amd.com>
> Suggested-by: Kalle Valo <kvalo@kernel.org>
> Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>

Thanks for the patch.

Tested-by: Kalle Valo <kvalo@kernel.org>

It would be good to get this to v6.8 but we cannot take this to wireless
tree yet, only after we have fast forwarded the tree.
diff mbox series

Patch

diff --git a/net/mac80211/wbrf.c b/net/mac80211/wbrf.c
index a05c5b971789..3a8612309137 100644
--- a/net/mac80211/wbrf.c
+++ b/net/mac80211/wbrf.c
@@ -23,8 +23,6 @@  void ieee80211_check_wbrf_support(struct ieee80211_local *local)
 		return;
 
 	local->wbrf_supported = acpi_amd_wbrf_supported_producer(dev);
-	dev_dbg(dev, "WBRF is %s supported\n",
-		local->wbrf_supported ? "" : "not");
 }
 
 static void get_chan_freq_boundary(u32 center_freq, u32 bandwidth, u64 *start, u64 *end)