Thread (11 messages) flat view 11 messages, 3 authors, 2021-10-05

Re: [PATCH net-next 1/4] mlx4: replace mlx4_mac_to_u64() with ether_addr_to_u64()

From: Tariq Toukan <hidden>
Date: 2021-10-05 09:55:52
Also in: linux-rdma


On 10/4/2021 10:14 PM, Jakub Kicinski wrote:
mlx4_mac_to_u64() predates and opencodes ether_addr_to_u64().
It doesn't make the argument constant so it'll be problematic
when dev->dev_addr becomes a const. Convert to the generic helper.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>
---
  drivers/infiniband/hw/mlx4/main.c              |  2 +-
  drivers/infiniband/hw/mlx4/qp.c                |  2 +-
  drivers/net/ethernet/mellanox/mlx4/cmd.c       |  2 +-
  drivers/net/ethernet/mellanox/mlx4/en_netdev.c | 16 ++++++++--------
  drivers/net/ethernet/mellanox/mlx4/fw.c        |  2 +-
  include/linux/mlx4/driver.h                    | 12 ------------
  6 files changed, 12 insertions(+), 24 deletions(-)
Reviewed-by: Tariq Toukan <tariqt@nvidia.com>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help