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

Haoyue Xu posted 7 patches 3 years, 6 months ago
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(-)
[PATCH net-next 0/7] net: ll_temac: Cleanup for clearing static warnings
Posted by Haoyue Xu 3 years, 6 months ago
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(-)

-- 
2.30.0
Re: [PATCH net-next 0/7] net: ll_temac: Cleanup for clearing static warnings
Posted by Harini Katakam 3 years, 6 months ago
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
Re: [PATCH net-next 0/7] net: ll_temac: Cleanup for clearing static warnings
Posted by patchwork-bot+netdevbpf@kernel.org 3 years, 6 months ago
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!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html