On Wed 10 Feb 2021 at 10:51, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > Hello Vlad Buslov, > > The patch 84ae9c1f29c0: "net/mlx5e: E-Switch, Maintain vhca_id to > vport_num mapping" from Sep 23, 2020, leads to the following static > checker warning: > > drivers/net/ethernet/mellanox/mlx5/core/vport.c:1170 mlx5_vport_get_other_func_cap() > warn: odd binop '0x0 & 0x1' > > drivers/net/ethernet/mellanox/mlx5/core/vport.c > 1168 int mlx5_vport_get_other_func_cap(struct mlx5_core_dev *dev, u16 function_id, void *out) > 1169 { > 1170 u16 opmod = (MLX5_CAP_GENERAL << 1) | (HCA_CAP_OPMOD_GET_MAX & 0x01); > > HCA_CAP_OPMOD_GET_MAX is zero. The 0x01 is a magical number. Hi Dan, Thanks for reporting and sorry for the late reply. Adding Saeed, we will handle the issue. Regards, Vlad [...]