Re: [PATCH net] net/mlx4_core: Change the error print to info print

[Tariq Toukan]

On 5/14/2019 2:03 PM, wangyunjian wrote:
> From: Yunjian Wang <wangyunj...@huawei.com>
> 
> The error print within mlx4_flow_steer_promisc_add() should
> be a info print.
> 
> Fixes: 592e49dda812 ('net/mlx4: Implement promiscuous mode with device 
> managed flow-steering')
> Signed-off-by: Yunjian Wang <wangyunj...@huawei.com>
> ---
>   drivers/net/ethernet/mellanox/mlx4/mcg.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/mellanox/mlx4/mcg.c 
> b/drivers/net/ethernet/mellanox/mlx4/mcg.c
> index ffed2d4..9c48182 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/mcg.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/mcg.c
> @@ -1492,7 +1492,7 @@ int mlx4_flow_steer_promisc_add(struct mlx4_dev *dev, 
> u8 port,
>       rule.port = port;
>       rule.qpn = qpn;
>       INIT_LIST_HEAD(&rule.list);
> -     mlx4_err(dev, "going promisc on %x\n", port);
> +     mlx4_info(dev, "going promisc on %x\n", port);
>   
>       return  mlx4_flow_attach(dev, &rule, regid_p);
>   }
> 

LGTM.
Reviewed-by: Tariq Toukan <tar...@mellanox.com>

Thanks,
Tariq