From: Joe Perches <j...@perches.com> Date: Sun, 17 Dec 2017 08:25:00 -0800
> On Sun, 2017-12-17 at 17:15 +0100, Jiri Pirko wrote: >> From: Arkadi Sharshevsky <arka...@mellanox.com> >> >> Add "spectrum" string prefix macro for error strings. > [] >> diff --git a/drivers/net/ethernet/mellanox/mlxsw/spectrum.c >> b/drivers/net/ethernet/mellanox/mlxsw/spectrum.c > [] >> @@ -4168,13 +4168,11 @@ mlxsw_sp_master_lag_check(struct mlxsw_sp *mlxsw_sp, >> u16 lag_id; >> >> if (mlxsw_sp_lag_index_get(mlxsw_sp, lag_dev, &lag_id) != 0) { >> - NL_SET_ERR_MSG(extack, >> - "spectrum: Exceeded number of supported LAG >> devices"); >> + NL_SET_ERR_MSG(extack, MLXSW_SP_PREFIX "Exceeded number of >> supported LAG devices"); > > Perhaps use NL_SET_ERR_MSG_MOD instead. Yeah that probably makes sense.