| Message ID | 20230919095205.24949-1-colin.i.king@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 6c0da8406382d39ec06ad54b0d4935bd7d63612c |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [next] wifi: cfg80211: make read-only array centers_80mhz static const | expand |
Hello: This patch was applied to netdev/net-next.git (main) by David S. Miller <davem@davemloft.net>: On Tue, 19 Sep 2023 10:52:05 +0100 you wrote: > Don't populate the read-only array lanes on the stack, instead make > it static const. > > Signed-off-by: Colin Ian King <colin.i.king@gmail.com> > --- > net/mac80211/tdls.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Here is the summary with links: - [next] wifi: cfg80211: make read-only array centers_80mhz static const https://git.kernel.org/netdev/net-next/c/6c0da8406382 You are awesome, thank you!
diff --git a/net/mac80211/tdls.c b/net/mac80211/tdls.c index ba14f570cda7..f3fd66d30b84 100644 --- a/net/mac80211/tdls.c +++ b/net/mac80211/tdls.c @@ -309,7 +309,7 @@ ieee80211_tdls_chandef_vht_upgrade(struct ieee80211_sub_if_data *sdata, struct sta_info *sta) { /* IEEE802.11ac-2013 Table E-4 */ - u16 centers_80mhz[] = { 5210, 5290, 5530, 5610, 5690, 5775 }; + static const u16 centers_80mhz[] = { 5210, 5290, 5530, 5610, 5690, 5775 }; struct cfg80211_chan_def uc = sta->tdls_chandef; enum nl80211_chan_width max_width = ieee80211_sta_cap_chan_bw(&sta->deflink);
Don't populate the read-only array lanes on the stack, instead make it static const. Signed-off-by: Colin Ian King <colin.i.king@gmail.com> --- net/mac80211/tdls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)