Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751744AbcDFPz1 (ORCPT ); Wed, 6 Apr 2016 11:55:27 -0400 Received: from mail.savoirfairelinux.com ([208.88.110.44]:56886 "EHLO mail.savoirfairelinux.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751266AbcDFPzY (ORCPT ); Wed, 6 Apr 2016 11:55:24 -0400 From: Vivien Didelot To: netdev@vger.kernel.org Cc: linux-kernel@vger.kernel.org, kernel@savoirfairelinux.com, "David S. Miller" , Andrew Lunn , Florian Fainelli , Jiri Pirko , Scott Feldman , Vivien Didelot Subject: [PATCH net-next v2 0/3] net: dsa: voidify STP setter and FDB/VLAN add ops Date: Wed, 6 Apr 2016 11:55:02 -0400 Message-Id: <1459958105-1090-1-git-send-email-vivien.didelot@savoirfairelinux.com> X-Mailer: git-send-email 2.8.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1155 Lines: 29 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 Documentation/networking/dsa/dsa.txt | 2 +- drivers/net/dsa/bcm_sf2.c | 25 +++++++-------- drivers/net/dsa/mv88e6171.c | 2 +- drivers/net/dsa/mv88e6352.c | 2 +- drivers/net/dsa/mv88e6xxx.c | 44 +++++++++++---------------- drivers/net/dsa/mv88e6xxx.h | 14 ++++----- include/net/dsa.h | 8 ++--- net/dsa/slave.c | 59 ++++++++++++++++-------------------- 8 files changed, 69 insertions(+), 87 deletions(-) -- 2.8.0