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.

Reply via email to