[v2,00/24] Rid W=1 warnings in net

Message ID 20210517044535.21473-1-shenyang39@huawei.com
Headers show
Series
  • Rid W=1 warnings in net
Related show

Message

Yang Shen May 17, 2021, 4:45 a.m.
This is 1 of 2 sets to fully clean drivers/net.

Changes since v1:
 - add some warnings missed in v1
 - move the patches about drivers/net/wireless into another series

Yang Shen (24):
  net: arc: Demote non-compliant kernel-doc headers
  net: atheros: atl1c: Fix wrong function name in comments
  net: atheros: atl1e: Fix wrong function name in comments
  net: atheros: atl1x: Fix wrong function name in comments
  net: broadcom: bnx2x: Fix wrong function name in comments
  net: brocade: bna: Fix wrong function name in comments
  net: cadence: Demote non-compliant kernel-doc headers
  net: calxeda: Fix wrong function name in comments
  net: chelsio: cxgb3: Fix wrong function name in comments
  net: chelsio: cxgb4: Fix wrong function name in comments
  net: chelsio: cxgb4vf: Fix wrong function name in comments
  net: huawei: hinic: Fix wrong function name in comments
  net: micrel: Fix wrong function name in comments
  net: microchip: Demote non-compliant kernel-doc headers
  net: neterion: Fix wrong function name in comments
  net: neterion: vxge: Fix wrong function name in comments
  net: netronome: nfp: Fix wrong function name in comments
  net: calxeda: Fix wrong function name in comments
  net: samsung: sxgbe: Fix wrong function name in comments
  net: socionext: Demote non-compliant kernel-doc headers
  net: ti: Fix wrong struct name in comments
  net: via: Fix wrong function name in comments
  net: phy: Demote non-compliant kernel-doc headers
  net: hisilicon: hns: Fix wrong function name in comments

 drivers/net/ethernet/arc/emac_rockchip.c         |  2 +-
 drivers/net/ethernet/atheros/atl1c/atl1c_main.c  |  6 +++---
 drivers/net/ethernet/atheros/atl1e/atl1e_main.c  |  4 ++--
 drivers/net/ethernet/atheros/atlx/atl1.c         |  2 +-
 drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c |  2 +-
 drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c   |  4 ++--
 drivers/net/ethernet/brocade/bna/bfa_cee.c       |  2 +-
 drivers/net/ethernet/cadence/macb_pci.c          |  2 +-
 drivers/net/ethernet/cadence/macb_ptp.c          |  2 +-
 drivers/net/ethernet/calxeda/xgmac.c             |  8 ++++----
 drivers/net/ethernet/chelsio/cxgb3/sge.c         |  4 ++--
 drivers/net/ethernet/chelsio/cxgb4/cxgb4_ptp.c   |  2 +-
 drivers/net/ethernet/chelsio/cxgb4/t4_hw.c       |  4 ++--
 drivers/net/ethernet/chelsio/cxgb4vf/sge.c       |  2 +-
 .../net/ethernet/hisilicon/hns/hns_dsaf_mac.c    |  6 +++---
 .../net/ethernet/hisilicon/hns/hns_dsaf_main.c   | 16 ++++++++--------
 .../net/ethernet/hisilicon/hns/hns_dsaf_misc.c   |  4 ++--
 .../net/ethernet/hisilicon/hns/hns_dsaf_ppe.c    |  8 ++++----
 .../net/ethernet/hisilicon/hns/hns_dsaf_rcb.c    |  2 +-
 .../net/ethernet/hisilicon/hns/hns_dsaf_xgmac.c  |  4 ++--
 drivers/net/ethernet/hisilicon/hns/hns_enet.c    |  2 +-
 drivers/net/ethernet/hisilicon/hns/hns_ethtool.c |  6 +++---
 .../net/ethernet/huawei/hinic/hinic_hw_cmdq.c    |  2 +-
 drivers/net/ethernet/huawei/hinic/hinic_hw_dev.c |  8 ++++----
 drivers/net/ethernet/huawei/hinic/hinic_hw_if.c  |  2 +-
 drivers/net/ethernet/huawei/hinic/hinic_hw_qp.c  |  2 +-
 drivers/net/ethernet/huawei/hinic/hinic_main.c   |  4 ++--
 drivers/net/ethernet/huawei/hinic/hinic_tx.c     |  2 +-
 drivers/net/ethernet/micrel/ksz884x.c            |  6 +++---
 drivers/net/ethernet/microchip/encx24j600.c      |  2 +-
 drivers/net/ethernet/neterion/s2io.c             |  4 ++--
 drivers/net/ethernet/neterion/vxge/vxge-config.c |  2 +-
 drivers/net/ethernet/neterion/vxge/vxge-main.c   |  4 ++--
 drivers/net/ethernet/netronome/nfp/ccm_mbox.c    |  2 +-
 .../ethernet/netronome/nfp/flower/tunnel_conf.c  |  2 +-
 .../ethernet/netronome/nfp/nfpcore/nfp_nffw.c    |  2 +-
 .../ethernet/qlogic/qlcnic/qlcnic_83xx_init.c    |  2 +-
 .../ethernet/qlogic/qlcnic/qlcnic_83xx_vnic.c    |  2 +-
 drivers/net/ethernet/samsung/sxgbe/sxgbe_main.c  |  4 ++--
 drivers/net/ethernet/socionext/sni_ave.c         |  2 +-
 drivers/net/ethernet/ti/cpsw_ale.c               |  2 +-
 drivers/net/ethernet/via/via-velocity.c          |  6 +++---
 drivers/net/phy/adin.c                           |  2 +-
 drivers/net/phy/rockchip.c                       |  2 +-
 44 files changed, 80 insertions(+), 80 deletions(-)

--
2.17.1

Comments

patchwork-bot+netdevbpf@kernel.org May 17, 2021, 9:20 p.m. | #1
Hello:

This series was applied to netdev/net-next.git (refs/heads/master):

On Mon, 17 May 2021 12:45:11 +0800 you wrote:
> This is 1 of 2 sets to fully clean drivers/net.
> 
> Changes since v1:
>  - add some warnings missed in v1
>  - move the patches about drivers/net/wireless into another series
> 
> Yang Shen (24):
>   net: arc: Demote non-compliant kernel-doc headers
>   net: atheros: atl1c: Fix wrong function name in comments
>   net: atheros: atl1e: Fix wrong function name in comments
>   net: atheros: atl1x: Fix wrong function name in comments
>   net: broadcom: bnx2x: Fix wrong function name in comments
>   net: brocade: bna: Fix wrong function name in comments
>   net: cadence: Demote non-compliant kernel-doc headers
>   net: calxeda: Fix wrong function name in comments
>   net: chelsio: cxgb3: Fix wrong function name in comments
>   net: chelsio: cxgb4: Fix wrong function name in comments
>   net: chelsio: cxgb4vf: Fix wrong function name in comments
>   net: huawei: hinic: Fix wrong function name in comments
>   net: micrel: Fix wrong function name in comments
>   net: microchip: Demote non-compliant kernel-doc headers
>   net: neterion: Fix wrong function name in comments
>   net: neterion: vxge: Fix wrong function name in comments
>   net: netronome: nfp: Fix wrong function name in comments
>   net: calxeda: Fix wrong function name in comments
>   net: samsung: sxgbe: Fix wrong function name in comments
>   net: socionext: Demote non-compliant kernel-doc headers
>   net: ti: Fix wrong struct name in comments
>   net: via: Fix wrong function name in comments
>   net: phy: Demote non-compliant kernel-doc headers
>   net: hisilicon: hns: Fix wrong function name in comments
> 
> [...]

Here is the summary with links:
  - [v2,01/24] net: arc: Demote non-compliant kernel-doc headers
    https://git.kernel.org/netdev/net-next/c/1d7f7ecadc5a
  - [v2,02/24] net: atheros: atl1c: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/8965c1c535b1
  - [v2,03/24] net: atheros: atl1e: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/b43e1554a7cf
  - [v2,04/24] net: atheros: atl1x: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/c706c75aaee2
  - [v2,05/24] net: broadcom: bnx2x: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/76d85049173b
  - [v2,06/24] net: brocade: bna: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/5a02bf4fefd5
  - [v2,07/24] net: cadence: Demote non-compliant kernel-doc headers
    https://git.kernel.org/netdev/net-next/c/c1167cee462d
  - [v2,08/24] net: calxeda: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/2e45d961a6a8
  - [v2,09/24] net: chelsio: cxgb3: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/aeed744a49ba
  - [v2,10/24] net: chelsio: cxgb4: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/e0333b1bb174
  - [v2,11/24] net: chelsio: cxgb4vf: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/1eb00ff517f4
  - [v2,12/24] net: huawei: hinic: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/d6174870c0f1
  - [v2,13/24] net: micrel: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/229fd41f6447
  - [v2,14/24] net: microchip: Demote non-compliant kernel-doc headers
    https://git.kernel.org/netdev/net-next/c/331a3219d3b6
  - [v2,15/24] net: neterion: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/dc432f5acb86
  - [v2,16/24] net: neterion: vxge: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/146c91e2bc9a
  - [v2,17/24] net: netronome: nfp: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/a507b1644524
  - [v2,18/24] net: calxeda: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/9f2e6fb63413
  - [v2,19/24] net: samsung: sxgbe: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/61633d71a71c
  - [v2,20/24] net: socionext: Demote non-compliant kernel-doc headers
    https://git.kernel.org/netdev/net-next/c/40d9fca8b3fd
  - [v2,21/24] net: ti: Fix wrong struct name in comments
    https://git.kernel.org/netdev/net-next/c/85ead77dc3d5
  - [v2,22/24] net: via: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/03055a25213b
  - [v2,23/24] net: phy: Demote non-compliant kernel-doc headers
    https://git.kernel.org/netdev/net-next/c/1f2d109e8363
  - [v2,24/24] net: hisilicon: hns: Fix wrong function name in comments
    https://git.kernel.org/netdev/net-next/c/5a9594cf1d14

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
Sudarsana Reddy Kalluru May 19, 2021, 9:30 a.m. | #2
> -----Original Message-----

> From: Yang Shen <shenyang39@huawei.com>

> Sent: Monday, May 17, 2021 10:15 AM

> To: davem@davemloft.net; kuba@kernel.org

> Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Yang Shen

> <shenyang39@huawei.com>; Ariel Elior <aelior@marvell.com>; Sudarsana

> Reddy Kalluru <skalluru@marvell.com>; GR-everest-linux-l2 <GR-everest-

> linux-l2@marvell.com>

> Subject: [EXT] [PATCH v2 05/24] net: broadcom: bnx2x: Fix wrong function

> name in comments

> 

> External Email

> 

> ----------------------------------------------------------------------

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

> 

>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c:13595: warning:

> expecting prototype for bnx2x_get_num_none_def_sbs(). Prototype was for

> bnx2x_get_num_non_def_sbs() instead

>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c:4165: warning:

> expecting prototype for atomic_add_ifless(). Prototype was for

> __atomic_add_ifless() instead

>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c:4193: warning:

> expecting prototype for atomic_dec_ifmoe(). Prototype was for

> __atomic_dec_ifmoe() instead

> 

> Cc: Ariel Elior <aelior@marvell.com>

> Cc: Sudarsana Kalluru <skalluru@marvell.com>

> Cc: GR-everest-linux-l2@marvell.com

> Signed-off-by: Yang Shen <shenyang39@huawei.com>

> ---

>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c | 2 +-

>  drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c   | 4 ++--

>  2 files changed, 3 insertions(+), 3 deletions(-)

> 

> diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c

> b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c

> index 281b1c2e04a7..2acbc73dcd18 100644

> --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c

> +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c

> @@ -13586,7 +13586,7 @@ static int bnx2x_set_qm_cid_count(struct bnx2x

> *bp)  }

> 

>  /**

> - * bnx2x_get_num_none_def_sbs - return the number of none default SBs

> + * bnx2x_get_num_non_def_sbs - return the number of none default SBs

>   * @pdev: pci device

>   * @cnic_cnt: count

>   *

> diff --git a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c

> b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c

> index 6cd1523ad9e5..542c69822649 100644

> --- a/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c

> +++ b/drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c

> @@ -4152,7 +4152,7 @@ void bnx2x_init_mcast_obj(struct bnx2x *bp,

>  /*************************** Credit handling

> **********************************/

> 

>  /**

> - * atomic_add_ifless - add if the result is less than a given value.

> + * __atomic_add_ifless - add if the result is less than a given value.

>   *

>   * @v:	pointer of type atomic_t

>   * @a:	the amount to add to v...

> @@ -4180,7 +4180,7 @@ static inline bool __atomic_add_ifless(atomic_t *v,

> int a, int u)  }

> 

>  /**

> - * atomic_dec_ifmoe - dec if the result is more or equal than a given value.

> + * __atomic_dec_ifmoe - dec if the result is more or equal than a given

> value.

>   *

>   * @v:	pointer of type atomic_t

>   * @a:	the amount to dec from v...

> --

> 2.17.1


Thanks for the changes.

Acked-by: Sudarsana Reddy Kalluru <skalluru@marvell.com>