mbox series

[net-next,0/7] net: ll_temac: Cleanup for clearing static warnings

Message ID 20220917103843.526877-1-xuhaoyue1@hisilicon.com (mailing list archive)
Headers show
Series net: ll_temac: Cleanup for clearing static warnings | expand

Message

Haoyue Xu Sept. 17, 2022, 10:38 a.m. UTC 
Most static warnings are detected by Checkpatch.pl, mainly about:
(1) #1: About the comments.
(2) #2: About function name in a string.
(3) #3: About the open parenthesis.
(4) #4: About the else branch.
(6) #6: About trailing statements.
(7) #5,#7: About blank lines and spaces.

Haoyue Xu (1):
  net: ll_temac: Cleanup for function name in a string

huangjunxian (6):
  net: ll_temac: fix the format of block comments
  net: ll_temac: axienet: align with open parenthesis
  net: ll_temac: delete unnecessary else branch
  net: ll_temac: fix the missing spaces around '='
  net: ll_temac: move trailing statements to next line
  net: ll_temac: axienet: delete unnecessary blank lines and spaces

 drivers/net/ethernet/xilinx/ll_temac.h        | 181 +++++++++---------
 drivers/net/ethernet/xilinx/ll_temac_main.c   |  65 ++++---
 drivers/net/ethernet/xilinx/ll_temac_mdio.c   |   6 +-
 drivers/net/ethernet/xilinx/xilinx_axienet.h  |   2 +-
 .../net/ethernet/xilinx/xilinx_axienet_main.c |   6 +-
 .../net/ethernet/xilinx/xilinx_axienet_mdio.c |   2 +-
 6 files changed, 139 insertions(+), 123 deletions(-)

Comments

Harini Katakam Sept. 20, 2022, 5:30 a.m. UTC | #1 
On Sat, Sep 17, 2022 at 4:10 PM Haoyue Xu <xuhaoyue1@hisilicon.com> wrote:
>
> Most static warnings are detected by Checkpatch.pl, mainly about:
> (1) #1: About the comments.
> (2) #2: About function name in a string.
> (3) #3: About the open parenthesis.
> (4) #4: About the else branch.
> (6) #6: About trailing statements.
> (7) #5,#7: About blank lines and spaces.

Thank you. For the series:
Reviewed-by: Harini Katakam <harini.katakam@amd.com>

Regards,
Harini
patchwork-bot+netdevbpf@kernel.org Sept. 22, 2022, 1:40 a.m. UTC | #2 
Hello:

This series was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Sat, 17 Sep 2022 18:38:36 +0800 you wrote:
> Most static warnings are detected by Checkpatch.pl, mainly about:
> (1) #1: About the comments.
> (2) #2: About function name in a string.
> (3) #3: About the open parenthesis.
> (4) #4: About the else branch.
> (6) #6: About trailing statements.
> (7) #5,#7: About blank lines and spaces.
> 
> [...]

Here is the summary with links:
  - [net-next,1/7] net: ll_temac: fix the format of block comments
    https://git.kernel.org/netdev/net-next/c/75124116520b
  - [net-next,2/7] net: ll_temac: Cleanup for function name in a string
    https://git.kernel.org/netdev/net-next/c/653de988eb58
  - [net-next,3/7] net: ll_temac: axienet: align with open parenthesis
    https://git.kernel.org/netdev/net-next/c/42f5d4d0e0d9
  - [net-next,4/7] net: ll_temac: delete unnecessary else branch
    https://git.kernel.org/netdev/net-next/c/7dfd0dcc5e72
  - [net-next,5/7] net: ll_temac: fix the missing spaces around '='
    https://git.kernel.org/netdev/net-next/c/a9f1ef7034b8
  - [net-next,6/7] net: ll_temac: move trailing statements to next line
    https://git.kernel.org/netdev/net-next/c/a0a850976801
  - [net-next,7/7] net: ll_temac: axienet: delete unnecessary blank lines and spaces
    https://git.kernel.org/netdev/net-next/c/7fe85bb3af4e

You are awesome, thank you!