lkml.org 
[lkml]   [2022]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 19/22] wnc36xx: Replace comments with C99 initializers
From
On 3/26/2022 9:59 AM, Benjamin Stürz wrote:
> This replaces comments with C99's designated
> initializers because the kernel supports them now.
>
> Signed-off-by: Benjamin Stürz <benni@stuerz.xyz>
> ---
> drivers/net/wireless/ath/wcn36xx/main.c | 122 ++++++++++++------------
> 1 file changed, 61 insertions(+), 61 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/wcn36xx/main.c b/drivers/net/wireless/ath/wcn36xx/main.c
> index 95ea7d040d8c..0fed64bd37b4 100644
> --- a/drivers/net/wireless/ath/wcn36xx/main.c
> +++ b/drivers/net/wireless/ath/wcn36xx/main.c
> @@ -193,67 +193,67 @@ static inline u8 get_sta_index(struct ieee80211_vif *vif,
> }
>
> static const char * const wcn36xx_caps_names[] = {
> - "MCC", /* 0 */
> - "P2P", /* 1 */
> - "DOT11AC", /* 2 */
> - "SLM_SESSIONIZATION", /* 3 */
> - "DOT11AC_OPMODE", /* 4 */
> - "SAP32STA", /* 5 */
> - "TDLS", /* 6 */
> - "P2P_GO_NOA_DECOUPLE_INIT_SCAN",/* 7 */
> - "WLANACTIVE_OFFLOAD", /* 8 */
> - "BEACON_OFFLOAD", /* 9 */
> - "SCAN_OFFLOAD", /* 10 */
> - "ROAM_OFFLOAD", /* 11 */
> - "BCN_MISS_OFFLOAD", /* 12 */
> - "STA_POWERSAVE", /* 13 */
> - "STA_ADVANCED_PWRSAVE", /* 14 */
> - "AP_UAPSD", /* 15 */
> - "AP_DFS", /* 16 */
> - "BLOCKACK", /* 17 */
> - "PHY_ERR", /* 18 */
> - "BCN_FILTER", /* 19 */
> - "RTT", /* 20 */
> - "RATECTRL", /* 21 */
> - "WOW", /* 22 */
> - "WLAN_ROAM_SCAN_OFFLOAD", /* 23 */
> - "SPECULATIVE_PS_POLL", /* 24 */
> - "SCAN_SCH", /* 25 */
> - "IBSS_HEARTBEAT_OFFLOAD", /* 26 */
> - "WLAN_SCAN_OFFLOAD", /* 27 */
> - "WLAN_PERIODIC_TX_PTRN", /* 28 */
> - "ADVANCE_TDLS", /* 29 */
> - "BATCH_SCAN", /* 30 */
> - "FW_IN_TX_PATH", /* 31 */
> - "EXTENDED_NSOFFLOAD_SLOT", /* 32 */
> - "CH_SWITCH_V1", /* 33 */
> - "HT40_OBSS_SCAN", /* 34 */
> - "UPDATE_CHANNEL_LIST", /* 35 */
> - "WLAN_MCADDR_FLT", /* 36 */
> - "WLAN_CH144", /* 37 */
> - "NAN", /* 38 */
> - "TDLS_SCAN_COEXISTENCE", /* 39 */
> - "LINK_LAYER_STATS_MEAS", /* 40 */
> - "MU_MIMO", /* 41 */
> - "EXTENDED_SCAN", /* 42 */
> - "DYNAMIC_WMM_PS", /* 43 */
> - "MAC_SPOOFED_SCAN", /* 44 */
> - "BMU_ERROR_GENERIC_RECOVERY", /* 45 */
> - "DISA", /* 46 */
> - "FW_STATS", /* 47 */
> - "WPS_PRBRSP_TMPL", /* 48 */
> - "BCN_IE_FLT_DELTA", /* 49 */
> - "TDLS_OFF_CHANNEL", /* 51 */
> - "RTT3", /* 52 */
> - "MGMT_FRAME_LOGGING", /* 53 */
> - "ENHANCED_TXBD_COMPLETION", /* 54 */
> - "LOGGING_ENHANCEMENT", /* 55 */
> - "EXT_SCAN_ENHANCED", /* 56 */
> - "MEMORY_DUMP_SUPPORTED", /* 57 */
> - "PER_PKT_STATS_SUPPORTED", /* 58 */
> - "EXT_LL_STAT", /* 60 */
> - "WIFI_CONFIG", /* 61 */
> - "ANTENNA_DIVERSITY_SELECTION", /* 62 */
> + [0] = "MCC",
> + [1] = "P2P",
> + [2] = "DOT11AC",
> + [3] = "SLM_SESSIONIZATION",
> + [4] = "DOT11AC_OPMODE",
> + [5] = "SAP32STA",
> + [6] = "TDLS",
> + [7] = "P2P_GO_NOA_DECOUPLE_INIT_SCAN",
> + [8] = "WLANACTIVE_OFFLOAD",
> + [9] = "BEACON_OFFLOAD",
> + [10] = "SCAN_OFFLOAD",
> + [11] = "ROAM_OFFLOAD",
> + [12] = "BCN_MISS_OFFLOAD",
> + [13] = "STA_POWERSAVE",
> + [14] = "STA_ADVANCED_PWRSAVE",
> + [15] = "AP_UAPSD",
> + [16] = "AP_DFS",
> + [17] = "BLOCKACK",
> + [18] = "PHY_ERR",
> + [19] = "BCN_FILTER",
> + [20] = "RTT",
> + [21] = "RATECTRL",
> + [22] = "WOW",
> + [23] = "WLAN_ROAM_SCAN_OFFLOAD",
> + [24] = "SPECULATIVE_PS_POLL",
> + [25] = "SCAN_SCH",
> + [26] = "IBSS_HEARTBEAT_OFFLOAD",
> + [27] = "WLAN_SCAN_OFFLOAD",
> + [28] = "WLAN_PERIODIC_TX_PTRN",
> + [29] = "ADVANCE_TDLS",
> + [30] = "BATCH_SCAN",
> + [31] = "FW_IN_TX_PATH",
> + [32] = "EXTENDED_NSOFFLOAD_SLOT",
> + [33] = "CH_SWITCH_V1",
> + [34] = "HT40_OBSS_SCAN",
> + [35] = "UPDATE_CHANNEL_LIST",
> + [36] = "WLAN_MCADDR_FLT",
> + [37] = "WLAN_CH144",
> + [38] = "NAN",
> + [39] = "TDLS_SCAN_COEXISTENCE",
> + [40] = "LINK_LAYER_STATS_MEAS",
> + [41] = "MU_MIMO",
> + [42] = "EXTENDED_SCAN",
> + [43] = "DYNAMIC_WMM_PS",
> + [44] = "MAC_SPOOFED_SCAN",
> + [45] = "BMU_ERROR_GENERIC_RECOVERY",
> + [46] = "DISA",
> + [47] = "FW_STATS",
> + [48] = "WPS_PRBRSP_TMPL",
> + [49] = "BCN_IE_FLT_DELTA",
> + [51] = "TDLS_OFF_CHANNEL",
> + [52] = "RTT3",
> + [53] = "MGMT_FRAME_LOGGING",
> + [54] = "ENHANCED_TXBD_COMPLETION",
> + [55] = "LOGGING_ENHANCEMENT",
> + [56] = "EXT_SCAN_ENHANCED",
> + [57] = "MEMORY_DUMP_SUPPORTED",
> + [58] = "PER_PKT_STATS_SUPPORTED",
> + [60] = "EXT_LL_STAT",
> + [61] = "WIFI_CONFIG",
> + [62] = "ANTENNA_DIVERSITY_SELECTION",
> };
>
> static const char *wcn36xx_get_cap_name(enum place_holder_in_cap_bitmap x)

I know there has been much discussion on this series. For this specific
patch this would be a great change if you use the actual enumerations
from enum place_holder_in_cap_bitmap as the index values,
i.e.
[MCC] = "MCC",
etc.

So a v2 for this patch would be appreciated



\
 
 \ /
  Last update: 2022-03-28 18:19    [W:0.402 / U:1.828 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site