Re: [PATCH net-next 1/3] net: dsa: make the STP state function return void

From: Vivien Didelot
Date: Tue Apr 05 2016 - 23:16:57 EST


Hi Andrew,

Andrew Lunn <andrew@xxxxxxx> writes:

>> -- port_stp_update: bridge layer function invoked when a given switch port STP
>> +- port_stp_state: bridge layer function invoked when a given switch port STP
>
> port_stp_state_set might be a better name, to make it clear it is
> setting the state, not getting the current state, etc. Most of the
> other functions are _add, _prepare, _join, _leave, so _set would fit
> the pattern.

I agree, I'm changing that.

> Changing to a void makes sense.

Thanks,
Vivien