From: Amit Cohen <amcohen@xxxxxxxxxx>
dev_dbg_ratelimited() is used as part of mlxsw_core_skb_receive(), the
printed info can be achieved using simple tracing. Remove such calls to
clean up the code. A next patch will change 'rx_info' fields, without
these prints, some fields will become unnecessary and can be removed.
Signed-off-by: Amit Cohen <amcohen@xxxxxxxxxx>
Reviewed-by: Ido Schimmel <idosch@xxxxxxxxxx>
Signed-off-by: Petr Machata <petrm@xxxxxxxxxx>
---
drivers/net/ethernet/mellanox/mlxsw/core.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/drivers/net/ethernet/mellanox/mlxsw/core.c b/drivers/net/ethernet/mellanox/mlxsw/core.c
index 2bb2b77351bd..8becb08984a6 100644
--- a/drivers/net/ethernet/mellanox/mlxsw/core.c
+++ b/drivers/net/ethernet/mellanox/mlxsw/core.c
@@ -2948,9 +2948,6 @@ void mlxsw_core_skb_receive(struct mlxsw_core *mlxsw_core, struct sk_buff *skb,
bool found = false;
if (rx_info->is_lag) {
- dev_dbg_ratelimited(mlxsw_core->bus_info->dev, "%s: lag_id = %d, lag_port_index = 0x%x\n",
- __func__, rx_info->u.lag_id,
- rx_info->trap_id);
/* Upper layer does not care if the skb came from LAG or not,
* so just get the local_port for the lag port and push it up.
*/
@@ -2961,9 +2958,6 @@ void mlxsw_core_skb_receive(struct mlxsw_core *mlxsw_core, struct sk_buff *skb,
local_port = rx_info->u.sys_port;
}
- dev_dbg_ratelimited(mlxsw_core->bus_info->dev, "%s: local_port = %d, trap_id = 0x%x\n",
- __func__, local_port, rx_info->trap_id);
-
if ((rx_info->trap_id >= MLXSW_TRAP_ID_MAX) ||
(local_port >= mlxsw_core->max_ports))
goto drop;
--
2.47.0
