Commit 31117517 authored by Lee Jones's avatar Lee Jones Committed by Kalle Valo

rsi: Fix a myriad of documentation issues

Too many, not enough, misnamed and formatting problems, all resolved.

Fixes the following W=1 kernel build warning(s):

 drivers/net/wireless/rsi/rsi_91x_mgmt.c:24: warning: cannot understand function prototype: 'struct bootup_params boot_params_20 = '
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:487: warning: Excess function parameter 'type' description in 'rsi_mgmt_pkt_to_core'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:538: warning: Function parameter or member 'sta_id' not described in 'rsi_hal_send_sta_notify_frame'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:538: warning: Function parameter or member 'vif' not described in 'rsi_hal_send_sta_notify_frame'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:615: warning: Function parameter or member 'sta_id' not described in 'rsi_send_aggregation_params_frame'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:711: warning: Function parameter or member 'mode' not described in 'rsi_set_vap_capabilities'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:711: warning: Function parameter or member 'mac_addr' not described in 'rsi_set_vap_capabilities'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:711: warning: Function parameter or member 'vap_id' not described in 'rsi_set_vap_capabilities'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:711: warning: Function parameter or member 'vap_status' not described in 'rsi_set_vap_capabilities'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:711: warning: Excess function parameter 'opmode' description in 'rsi_set_vap_capabilities'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:794: warning: Function parameter or member 'sta_id' not described in 'rsi_hal_load_key'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:794: warning: Function parameter or member 'vif' not described in 'rsi_hal_load_key'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1053: warning: Function parameter or member 'curchan' not described in 'rsi_band_check'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1173: warning: Excess function parameter 'channel' description in 'rsi_send_radio_params_update'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1299: warning: Function parameter or member 'sta' not described in 'rsi_send_auto_rate_request'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1299: warning: Function parameter or member 'sta_id' not described in 'rsi_send_auto_rate_request'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1299: warning: Function parameter or member 'vif' not described in 'rsi_send_auto_rate_request'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1459: warning: Function parameter or member 'opmode' not described in 'rsi_inform_bss_status'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1459: warning: Function parameter or member 'addr' not described in 'rsi_inform_bss_status'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1459: warning: Function parameter or member 'sta' not described in 'rsi_inform_bss_status'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1459: warning: Function parameter or member 'sta_id' not described in 'rsi_inform_bss_status'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1459: warning: Function parameter or member 'assoc_cap' not described in 'rsi_inform_bss_status'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1459: warning: Function parameter or member 'vif' not described in 'rsi_inform_bss_status'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1459: warning: Excess function parameter 'bssid' description in 'rsi_inform_bss_status'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1543: warning: Function parameter or member 'common' not described in 'rsi_send_block_unblock_frame'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1543: warning: Function parameter or member 'block_event' not described in 'rsi_send_block_unblock_frame'
 drivers/net/wireless/rsi/rsi_91x_mgmt.c:1587: warning: Excess function parameter 'Return' description in 'rsi_send_rx_filter_frame'

Cc: Amitkumar Karwar <amitkarwar@gmail.com>
Cc: Siva Rebbagondla <siva8118@gmail.com>
Cc: Kalle Valo <kvalo@codeaurora.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: linux-wireless@vger.kernel.org
Cc: netdev@vger.kernel.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200819072402.3085022-21-lee.jones@linaro.org
parent 5dfcdc7a
/** /*
* Copyright (c) 2014 Redpine Signals Inc. * Copyright (c) 2014 Redpine Signals Inc.
* *
* Permission to use, copy, modify, and/or distribute this software for any * Permission to use, copy, modify, and/or distribute this software for any
...@@ -477,7 +477,6 @@ static int rsi_load_radio_caps(struct rsi_common *common) ...@@ -477,7 +477,6 @@ static int rsi_load_radio_caps(struct rsi_common *common)
* @common: Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @msg: Pointer to received packet. * @msg: Pointer to received packet.
* @msg_len: Length of the received packet. * @msg_len: Length of the received packet.
* @type: Type of received packet.
* *
* Return: 0 on success, -1 on failure. * Return: 0 on success, -1 on failure.
*/ */
...@@ -528,6 +527,8 @@ static int rsi_mgmt_pkt_to_core(struct rsi_common *common, ...@@ -528,6 +527,8 @@ static int rsi_mgmt_pkt_to_core(struct rsi_common *common,
* @bssid: bssid. * @bssid: bssid.
* @qos_enable: Qos is enabled. * @qos_enable: Qos is enabled.
* @aid: Aid (unique for all STA). * @aid: Aid (unique for all STA).
* @sta_id: station id.
* @vif: Pointer to the ieee80211_vif structure.
* *
* Return: status: 0 on success, corresponding negative error code on failure. * Return: status: 0 on success, corresponding negative error code on failure.
*/ */
...@@ -603,6 +604,7 @@ int rsi_hal_send_sta_notify_frame(struct rsi_common *common, enum opmode opmode, ...@@ -603,6 +604,7 @@ int rsi_hal_send_sta_notify_frame(struct rsi_common *common, enum opmode opmode,
* @ssn: ssn. * @ssn: ssn.
* @buf_size: buffer size. * @buf_size: buffer size.
* @event: notification about station connection. * @event: notification about station connection.
* @sta_id: station id.
* *
* Return: 0 on success, corresponding negative error code on failure. * Return: 0 on success, corresponding negative error code on failure.
*/ */
...@@ -699,7 +701,7 @@ static int rsi_program_bb_rf(struct rsi_common *common) ...@@ -699,7 +701,7 @@ static int rsi_program_bb_rf(struct rsi_common *common)
/** /**
* rsi_set_vap_capabilities() - This function send vap capability to firmware. * rsi_set_vap_capabilities() - This function send vap capability to firmware.
* @common: Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @opmode: Operating mode of device. * @mode: Operating mode of device.
* *
* Return: 0 on success, corresponding negative error code on failure. * Return: 0 on success, corresponding negative error code on failure.
*/ */
...@@ -780,6 +782,8 @@ int rsi_set_vap_capabilities(struct rsi_common *common, ...@@ -780,6 +782,8 @@ int rsi_set_vap_capabilities(struct rsi_common *common,
* @key_type: Type of key: GROUP/PAIRWISE. * @key_type: Type of key: GROUP/PAIRWISE.
* @key_id: Key index. * @key_id: Key index.
* @cipher: Type of cipher used. * @cipher: Type of cipher used.
* @sta_id: Station id.
* @vif: Pointer to the ieee80211_vif structure.
* *
* Return: 0 on success, -1 on failure. * Return: 0 on success, -1 on failure.
*/ */
...@@ -1045,6 +1049,7 @@ static int rsi_send_reset_mac(struct rsi_common *common) ...@@ -1045,6 +1049,7 @@ static int rsi_send_reset_mac(struct rsi_common *common)
/** /**
* rsi_band_check() - This function programs the band * rsi_band_check() - This function programs the band
* @common: Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @curchan: Pointer to the current channel structure.
* *
* Return: 0 on success, corresponding error code on failure. * Return: 0 on success, corresponding error code on failure.
*/ */
...@@ -1165,7 +1170,6 @@ int rsi_set_channel(struct rsi_common *common, ...@@ -1165,7 +1170,6 @@ int rsi_set_channel(struct rsi_common *common,
* rsi_send_radio_params_update() - This function sends the radio * rsi_send_radio_params_update() - This function sends the radio
* parameters update to device * parameters update to device
* @common: Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @channel: Channel value to be set.
* *
* Return: 0 on success, corresponding error code on failure. * Return: 0 on success, corresponding error code on failure.
*/ */
...@@ -1289,6 +1293,9 @@ static bool rsi_map_rates(u16 rate, int *offset) ...@@ -1289,6 +1293,9 @@ static bool rsi_map_rates(u16 rate, int *offset)
* rsi_send_auto_rate_request() - This function is to set rates for connection * rsi_send_auto_rate_request() - This function is to set rates for connection
* and send autorate request to firmware. * and send autorate request to firmware.
* @common: Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @sta: mac80211 station.
* @sta_id: station id.
* @vif: Pointer to the ieee80211_vif structure.
* *
* Return: 0 on success, corresponding error code on failure. * Return: 0 on success, corresponding error code on failure.
*/ */
...@@ -1439,10 +1446,15 @@ static int rsi_send_auto_rate_request(struct rsi_common *common, ...@@ -1439,10 +1446,15 @@ static int rsi_send_auto_rate_request(struct rsi_common *common,
* help of sta notify params by sending an internal * help of sta notify params by sending an internal
* management frame to firmware. * management frame to firmware.
* @common: Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @opmode: Operating mode of device.
* @status: Bss status type. * @status: Bss status type.
* @bssid: Bssid. * @addr: Address of the register.
* @qos_enable: Qos is enabled. * @qos_enable: Qos is enabled.
* @aid: Aid (unique for all STAs). * @aid: Aid (unique for all STAs).
* @sta: mac80211 station.
* @sta_id: station id.
* @assoc_cap: capabilities.
* @vif: Pointer to the ieee80211_vif structure.
* *
* Return: None. * Return: None.
*/ */
...@@ -1535,9 +1547,9 @@ static int rsi_eeprom_read(struct rsi_common *common) ...@@ -1535,9 +1547,9 @@ static int rsi_eeprom_read(struct rsi_common *common)
* This function sends a frame to block/unblock * This function sends a frame to block/unblock
* data queues in the firmware * data queues in the firmware
* *
* @param common Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @param block event - block if true, unblock if false * @block_event: Event block if true, unblock if false
* @return 0 on success, -1 on failure. * returns 0 on success, -1 on failure.
*/ */
int rsi_send_block_unblock_frame(struct rsi_common *common, bool block_event) int rsi_send_block_unblock_frame(struct rsi_common *common, bool block_event)
{ {
...@@ -1581,7 +1593,7 @@ int rsi_send_block_unblock_frame(struct rsi_common *common, bool block_event) ...@@ -1581,7 +1593,7 @@ int rsi_send_block_unblock_frame(struct rsi_common *common, bool block_event)
* @common: Pointer to the driver private structure. * @common: Pointer to the driver private structure.
* @rx_filter_word: Flags of filter packets * @rx_filter_word: Flags of filter packets
* *
* @Return: 0 on success, -1 on failure. * Returns 0 on success, -1 on failure.
*/ */
int rsi_send_rx_filter_frame(struct rsi_common *common, u16 rx_filter_word) int rsi_send_rx_filter_frame(struct rsi_common *common, u16 rx_filter_word)
{ {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment