Re: [PATCH net-next v2 0/3] net: dsa: voidify STP setter and FDB/VLAN add ops

[Florian Fainelli]

2016-04-06 8:55 GMT-07:00 Vivien Didelot <vivien.didelot@xxxxxxxxxxxxxxxxxxxx>:
> Neither the DSA layer nor the bridge code (see br_set_state) really care
> about eventual errors from STP state setters, so make it void.
>
> The DSA layer separates the prepare and commit phases of switchdev in
> two different functions. Logical errors must not happen in commit
> routines, so make them void.
>
> Changes v1 -> v2:
>   - rename port_stp_update to port_stp_state_set
>   - don't change code flow of bcm_sf2_sw_br_set_stp_state
>   - prefer netdev_err over netdev_warn
>
> Vivien Didelot (3):
>   net: dsa: make the STP state function return void
>   net: dsa: make the FDB add function return void
>   net: dsa: make the VLAN add function return void

Reviewed-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
--
Florian