Tue, Mar 28, 2017 at 12:48:34AM CEST, da...@davemloft.net wrote:
>
>Please fix up these warnings and resubmit:
>
>drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c: In function
>‘mlxsw_sp_rif_counter_free’:
>drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c:208:2: warning:
>‘p_counter_index’ may be used uninitialized in this function
>[-Wmaybe-uninitialized]
> mlxsw_sp_rif_counter_edit(mlxsw_sp, rif->rif_index,
> ^
>drivers/net/ethernet/mellanox/mlxsw/spectrum_dpipe.c: In function
>‘mlxsw_sp_table_erif_entries_dump’:
>drivers/net/ethernet/mellanox/mlxsw/spectrum_dpipe.c:220:9: warning: missing
>braces around initializer [-Wmissing-braces]
> struct devlink_dpipe_value match_value = {0}, action_value = {0};
I know about this warning. I believe that it is a gcc bug. It happens
for gcc 4, but gcc 6 is silent, therefore I choose to ignore this. Do
you still want me to fix it?
> ^
>drivers/net/ethernet/mellanox/mlxsw/spectrum_dpipe.c:220:9: warning: (near
>initialization for ‘match_value.<anonymous>’) [-Wmissing-braces]
>drivers/net/ethernet/mellanox/mlxsw/spectrum_dpipe.c:220:9: warning: missing
>braces around initializer [-Wmissing-braces]
>drivers/net/ethernet/mellanox/mlxsw/spectrum_dpipe.c:220:9: warning: (near
>initialization for ‘action_value.<anonymous>’) [-Wmissing-braces]
