mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-30 06:10:56 +00:00
wl3501_cs: Fix a bunch of formatting issues related to function docs
[ Upstream commit2307d0bc9d
] Fixes the following W=1 kernel build warning(s): In file included from drivers/net/wireless/wl3501_cs.c:57: drivers/net/wireless/wl3501_cs.c:143: warning: Function parameter or member 'reg_domain' not described in 'iw_valid_channel' drivers/net/wireless/wl3501_cs.c:143: warning: Function parameter or member 'channel' not described in 'iw_valid_channel' drivers/net/wireless/wl3501_cs.c:162: warning: Function parameter or member 'reg_domain' not described in 'iw_default_channel' drivers/net/wireless/wl3501_cs.c:248: warning: Function parameter or member 'this' not described in 'wl3501_set_to_wla' drivers/net/wireless/wl3501_cs.c:270: warning: Function parameter or member 'this' not described in 'wl3501_get_from_wla' drivers/net/wireless/wl3501_cs.c:467: warning: Function parameter or member 'this' not described in 'wl3501_send_pkt' drivers/net/wireless/wl3501_cs.c:467: warning: Function parameter or member 'data' not described in 'wl3501_send_pkt' drivers/net/wireless/wl3501_cs.c:467: warning: Function parameter or member 'len' not described in 'wl3501_send_pkt' drivers/net/wireless/wl3501_cs.c:729: warning: Function parameter or member 'this' not described in 'wl3501_block_interrupt' drivers/net/wireless/wl3501_cs.c:746: warning: Function parameter or member 'this' not described in 'wl3501_unblock_interrupt' drivers/net/wireless/wl3501_cs.c:1124: warning: Function parameter or member 'irq' not described in 'wl3501_interrupt' drivers/net/wireless/wl3501_cs.c:1124: warning: Function parameter or member 'dev_id' not described in 'wl3501_interrupt' drivers/net/wireless/wl3501_cs.c:1257: warning: Function parameter or member 'dev' not described in 'wl3501_reset' drivers/net/wireless/wl3501_cs.c:1420: warning: Function parameter or member 'link' not described in 'wl3501_detach' Cc: Kalle Valo <kvalo@codeaurora.org> Cc: "David S. Miller" <davem@davemloft.net> Cc: Jakub Kicinski <kuba@kernel.org> Cc: Fox Chen <mhchen@golf.ccl.itri.org.tw> Cc: de Melo <acme@conectiva.com.br> Cc: Gustavo Niemeyer <niemeyer@conectiva.com> Cc: linux-wireless@vger.kernel.org Cc: netdev@vger.kernel.org Signed-off-by: Lee Jones <lee.jones@linaro.org> Signed-off-by: Kalle Valo <kvalo@codeaurora.org> Link: https://lore.kernel.org/r/20200826093401.1458456-21-lee.jones@linaro.org Stable-dep-of:391af06a02
("wifi: wl3501_cs: Fix an error handling path in wl3501_probe()") Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
419b03c6d9
commit
5c9e57d44b
1 changed files with 12 additions and 10 deletions
|
@ -133,8 +133,8 @@ static const struct {
|
|||
|
||||
/**
|
||||
* iw_valid_channel - validate channel in regulatory domain
|
||||
* @reg_comain - regulatory domain
|
||||
* @channel - channel to validate
|
||||
* @reg_comain: regulatory domain
|
||||
* @channel: channel to validate
|
||||
*
|
||||
* Returns 0 if invalid in the specified regulatory domain, non-zero if valid.
|
||||
*/
|
||||
|
@ -153,7 +153,7 @@ static int iw_valid_channel(int reg_domain, int channel)
|
|||
|
||||
/**
|
||||
* iw_default_channel - get default channel for a regulatory domain
|
||||
* @reg_comain - regulatory domain
|
||||
* @reg_domain: regulatory domain
|
||||
*
|
||||
* Returns the default channel for a regulatory domain
|
||||
*/
|
||||
|
@ -236,6 +236,7 @@ static int wl3501_get_flash_mac_addr(struct wl3501_card *this)
|
|||
|
||||
/**
|
||||
* wl3501_set_to_wla - Move 'size' bytes from PC to card
|
||||
* @this: Card
|
||||
* @dest: Card addressing space
|
||||
* @src: PC addressing space
|
||||
* @size: Bytes to move
|
||||
|
@ -258,6 +259,7 @@ static void wl3501_set_to_wla(struct wl3501_card *this, u16 dest, void *src,
|
|||
|
||||
/**
|
||||
* wl3501_get_from_wla - Move 'size' bytes from card to PC
|
||||
* @this: Card
|
||||
* @src: Card addressing space
|
||||
* @dest: PC addressing space
|
||||
* @size: Bytes to move
|
||||
|
@ -454,7 +456,7 @@ static int wl3501_pwr_mgmt(struct wl3501_card *this, int suspend)
|
|||
|
||||
/**
|
||||
* wl3501_send_pkt - Send a packet.
|
||||
* @this - card
|
||||
* @this: Card
|
||||
*
|
||||
* Send a packet.
|
||||
*
|
||||
|
@ -722,7 +724,7 @@ static void wl3501_mgmt_scan_confirm(struct wl3501_card *this, u16 addr)
|
|||
|
||||
/**
|
||||
* wl3501_block_interrupt - Mask interrupt from SUTRO
|
||||
* @this - card
|
||||
* @this: Card
|
||||
*
|
||||
* Mask interrupt from SUTRO. (i.e. SUTRO cannot interrupt the HOST)
|
||||
* Return: 1 if interrupt is originally enabled
|
||||
|
@ -739,7 +741,7 @@ static int wl3501_block_interrupt(struct wl3501_card *this)
|
|||
|
||||
/**
|
||||
* wl3501_unblock_interrupt - Enable interrupt from SUTRO
|
||||
* @this - card
|
||||
* @this: Card
|
||||
*
|
||||
* Enable interrupt from SUTRO. (i.e. SUTRO can interrupt the HOST)
|
||||
* Return: 1 if interrupt is originally enabled
|
||||
|
@ -1113,8 +1115,8 @@ static inline void wl3501_ack_interrupt(struct wl3501_card *this)
|
|||
|
||||
/**
|
||||
* wl3501_interrupt - Hardware interrupt from card.
|
||||
* @irq - Interrupt number
|
||||
* @dev_id - net_device
|
||||
* @irq: Interrupt number
|
||||
* @dev_id: net_device
|
||||
*
|
||||
* We must acknowledge the interrupt as soon as possible, and block the
|
||||
* interrupt from the same card immediately to prevent re-entry.
|
||||
|
@ -1252,7 +1254,7 @@ static int wl3501_close(struct net_device *dev)
|
|||
|
||||
/**
|
||||
* wl3501_reset - Reset the SUTRO.
|
||||
* @dev - network device
|
||||
* @dev: network device
|
||||
*
|
||||
* It is almost the same as wl3501_open(). In fact, we may just wl3501_close()
|
||||
* and wl3501_open() again, but I wouldn't like to free_irq() when the driver
|
||||
|
@ -1415,7 +1417,7 @@ static struct iw_statistics *wl3501_get_wireless_stats(struct net_device *dev)
|
|||
|
||||
/**
|
||||
* wl3501_detach - deletes a driver "instance"
|
||||
* @link - FILL_IN
|
||||
* @link: FILL_IN
|
||||
*
|
||||
* This deletes a driver "instance". The device is de-registered with Card
|
||||
* Services. If it has been released, all local data structures are freed.
|
||||
|
|
Loading…
Reference in a new issue