diff mbox series

[15/28] wireless: mediatek: mt7601u: phy: Fix misnaming when documented function parameter 'dac'

Message ID 20200819072402.3085022-16-lee.jones@linaro.org (mailing list archive)
State Accepted
Commit e6cf87bfe869e0cbe99978b422a43ddd5bccd655
Delegated to: Kalle Valo
Headers show
Series Rid W=1 warnings in Wireless | expand

Commit Message

Lee Jones Aug. 19, 2020, 7:23 a.m. UTC
Fixes the following W=1 kernel build warning(s):

 drivers/net/wireless/mediatek/mt7601u/phy.c:1216: warning: Function parameter or member 'dac' not described in 'mt7601u_set_tx_dac'
 drivers/net/wireless/mediatek/mt7601u/phy.c:1216: warning: Excess function parameter 'path' description in 'mt7601u_set_tx_dac'

Cc: Jakub Kicinski <kubakici@wp.pl>
Cc: Kalle Valo <kvalo@codeaurora.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Matthias Brugger <matthias.bgg@gmail.com>
Cc: Felix Fietkau <nbd@openwrt.org>
Cc: linux-wireless@vger.kernel.org
Cc: netdev@vger.kernel.org
Cc: linux-mediatek@lists.infradead.org
Signed-off-by: Lee Jones <lee.jones@linaro.org>
---
 drivers/net/wireless/mediatek/mt7601u/phy.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jakub Kicinski Aug. 19, 2020, 4:31 p.m. UTC | #1
On Wed, 19 Aug 2020 08:23:49 +0100 Lee Jones wrote:
> Fixes the following W=1 kernel build warning(s):
> 
>  drivers/net/wireless/mediatek/mt7601u/phy.c:1216: warning: Function parameter or member 'dac' not described in 'mt7601u_set_tx_dac'
>  drivers/net/wireless/mediatek/mt7601u/phy.c:1216: warning: Excess function parameter 'path' description in 'mt7601u_set_tx_dac'

Acked-by: Jakub Kicinski <kubakici@wp.pl>
Kalle Valo Sept. 1, 2020, 9:07 a.m. UTC | #2
Lee Jones <lee.jones@linaro.org> wrote:

> Fixes the following W=1 kernel build warning(s):
> 
>  drivers/net/wireless/mediatek/mt7601u/phy.c:1216: warning: Function parameter or member 'dac' not described in 'mt7601u_set_tx_dac'
>  drivers/net/wireless/mediatek/mt7601u/phy.c:1216: warning: Excess function parameter 'path' description in 'mt7601u_set_tx_dac'
> 
> Cc: Jakub Kicinski <kubakici@wp.pl>
> Cc: Kalle Valo <kvalo@codeaurora.org>
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Matthias Brugger <matthias.bgg@gmail.com>
> Cc: Felix Fietkau <nbd@openwrt.org>
> Cc: linux-wireless@vger.kernel.org
> Cc: netdev@vger.kernel.org
> Cc: linux-mediatek@lists.infradead.org
> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> Acked-by: Jakub Kicinski <kubakici@wp.pl>

10 patches applied to wireless-drivers-next.git, thanks.

e6cf87bfe869 mt7601u: phy: Fix misnaming when documented function parameter 'dac'
a8433a92521b rsi: Fix misnamed function parameter 'rx_pkt'
5dfcdc7a520e rsi: Fix a few kerneldoc misdemeanours
311175173c8a rsi: Fix a myriad of documentation issues
9463fd554bb8 rsi: File header comments should not be kernel-doc
7951a3bfa2b1 iwlegacy: 4965: Demote a bunch of nonconformant kernel-doc headers
2de64ca7c9fa brcmfmac: p2p: Deal with set but unused variables
457023556e94 libertas: Fix misnaming for function param 'device'
f030ed4079d0 libertas_tf: Fix function documentation formatting errors
ec511969097f hostap: Remove set but unused variable 'hostscan'
diff mbox series

Patch

diff --git a/drivers/net/wireless/mediatek/mt7601u/phy.c b/drivers/net/wireless/mediatek/mt7601u/phy.c
index d863ab4a66c96..3c4462487ab5c 100644
--- a/drivers/net/wireless/mediatek/mt7601u/phy.c
+++ b/drivers/net/wireless/mediatek/mt7601u/phy.c
@@ -1210,7 +1210,7 @@  void mt7601u_set_rx_path(struct mt7601u_dev *dev, u8 path)
 /**
  * mt7601u_set_tx_dac - set which tx DAC to use
  * @dev:	pointer to adapter structure
- * @path:	DAC index, values are 0-based
+ * @dac:	DAC index, values are 0-based
  */
 void mt7601u_set_tx_dac(struct mt7601u_dev *dev, u8 dac)
 {