Skip to content

Remove duplicated devlink registration check

Leon Romanovsky netdev 1 version Applied

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

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 0 repliesApplied 3/3

Changes in v1 from the cover letter

* Added two new patches that remove registration field from mlx5 and ti drivers.
v0: https://lore.kernel.org/lkml/ed7bbb1e4c51dd58e6035a058e93d16f883b09ce.1627215829.git.leonro@nvidia.com

Patches in v1 All 3 applied

#Subject and filesSizeRepliesApplied
1/3net: ti: am65-cpsw-nuss: fix wrong devlink release orderdrivers/net/ethernet/ti/am65-cpsw-nuss.c+17 -170 repliesApplied
2/3net/mlx5: Don't rely on always true registered fielddrivers/net/ethernet/mellanox/mlx5/core/en/devlink.c drivers/net/ethernet/mellanox/mlx5/core/en_main.c+5 -160 repliesApplied
3/3devlink: Remove duplicated registration checkinclude/net/devlink.h net/core/devlink.c+11 -120 repliesApplied

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH net-next v1 0/3] Remove duplicated devlink registration check

From: Leon Romanovsky <[email protected]>

Changelog:
v1:
* Added two new patches that remove registration field from mlx5 and ti drivers.
v0: https://lore.kernel.org/lkml/ed7bbb1e4c51dd58e6035a058e93d16f883b09ce.1627215829.git.leonro@nvidia.com

--------------------------------------------------------------------

Both registered flag and devlink pointer are set at the same time and indicate the same thing - devlink/devlink_port are ready. Instead of checking ->registered use devlink pointer as an indication.

Thanks

Leon Romanovsky (3):
net: ti: am65-cpsw-nuss: fix wrong devlink release order net/mlx5: Don't rely on always true registered field
devlink: Remove duplicated registration check

 .../ethernet/mellanox/mlx5/core/en/devlink.c  | 10 ++----
 .../net/ethernet/mellanox/mlx5/core/en_main.c | 11 ++----
 drivers/net/ethernet/ti/am65-cpsw-nuss.c      | 34 +++++++++----------
 include/net/devlink.h                         |  4 +--
 net/core/devlink.c                            | 19 ++++++-----
 5 files changed, 33 insertions(+), 45 deletions(-)
Signature · 1 line
2.31.1