Tue, Dec 19, 2017 at 05:09:06PM CET, da...@davemloft.net wrote:
>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.
Yes. Thanks
