diff options
author | Johannes Berg <johannes.berg@intel.com> | 2024-01-29 19:34:39 +0100 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2024-02-08 12:58:38 +0100 |
commit | 9bf7079bc2271321fac467cae981c44e495b76b9 (patch) | |
tree | f7c21ec872ca5c29d0cf8805ce2fcafc1ecb0f19 | |
parent | wifi: mac80211: simplify non-chanctx drivers (diff) | |
download | linux-9bf7079bc2271321fac467cae981c44e495b76b9.tar.xz linux-9bf7079bc2271321fac467cae981c44e495b76b9.zip |
wifi: mac80211: chan: chandef is non-NULL for reserved
The last caller of this with a NULL argument was related to
the non-chanctx code, so we can now remove this odd logic.
Reviewed-by: Miriam Rachel Korenblit <miriam.rachel.korenblit@intel.com>
Link: https://msgid.link/20240129194108.bad8ec1e76c8.I12287452f42c54baf75821e75491cf6d021af20a@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
-rw-r--r-- | net/mac80211/chan.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/mac80211/chan.c b/net/mac80211/chan.c index cf6297ffaef3..6b82c79cf7a6 100644 --- a/net/mac80211/chan.c +++ b/net/mac80211/chan.c @@ -90,11 +90,11 @@ ieee80211_chanctx_reserved_chandef(struct ieee80211_local *local, lockdep_assert_wiphy(local->hw.wiphy); + if (WARN_ON(!compat)) + return NULL; + list_for_each_entry(link, &ctx->reserved_links, reserved_chanctx_list) { - if (!compat) - compat = &link->reserved_chandef; - compat = cfg80211_chandef_compatible(&link->reserved_chandef, compat); if (!compat) |