net: dsa: make the FDB add function return void

The switchdev design implies that a software error should not happen in
the commit phase since it must have been previously reported in the
prepare phase. If an hardware error occurs during the commit phase,
there is nothing switchdev can do about it.

The DSA layer separates port_fdb_prepare and port_fdb_add for simplicity
and convenience. If an hardware error occurs during the commit phase,
there is no need to report it outside the DSA driver itself.

Make the DSA port_fdb_add routine return void for explicitness.

Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 8 insertions, 8 deletions

diff --git a/net/dsa/slave.c b/net/dsa/slave.c
index 088215c3642f..90bc7442c44f 100644
--- a/net/dsa/slave.c
+++ b/net/dsa/slave.c
@@ -256,17 +256,17 @@ static int dsa_slave_port_fdb_add(struct net_device *dev,
 {
         struct dsa_slave_priv *p = netdev_priv(dev);
         struct dsa_switch *ds = p->parent;
-        int ret;
 
-        if (!ds->drv->port_fdb_prepare || !ds->drv->port_fdb_add)
-                return -EOPNOTSUPP;
+        if (switchdev_trans_ph_prepare(trans)) {
+                if (!ds->drv->port_fdb_prepare || !ds->drv->port_fdb_add)
+                        return -EOPNOTSUPP;
 
-        if (switchdev_trans_ph_prepare(trans))
-                ret = ds->drv->port_fdb_prepare(ds, p->port, fdb, trans);
-        else
-                ret = ds->drv->port_fdb_add(ds, p->port, fdb, trans);
+                return ds->drv->port_fdb_prepare(ds, p->port, fdb, trans);
+        }
 
-        return ret;
+        ds->drv->port_fdb_add(ds, p->port, fdb, trans);
+
+        return 0;
 }
 
 static int dsa_slave_port_fdb_del(struct net_device *dev,