Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a896eee3 authored by Vivien Didelot's avatar Vivien Didelot Committed by David S. Miller
Browse files

net: dsa: remove out label in dsa_switch_setup_one 



The "out" label in dsa_switch_setup_one() is useless, thus remove it.

Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarVivien Didelot <vivien.didelot@savoirfairelinux.com>
Reviewed-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 9feb16ae

net/dsa/dsa.c

+13 −27
Original line number Diff line number Diff line
@@ -329,8 +329,7 @@ static int dsa_switch_setup_one(struct dsa_switch *ds, struct device *parent) 
			if (dst->cpu_switch != -1) {

				netdev_err(dst->master_netdev,

					   "multiple cpu ports?!\n");

				ret = -EINVAL;

				goto out;

				return -EINVAL;

			}

			dst->cpu_switch = index;

			dst->cpu_port = i;
@@ -343,10 +342,8 @@ static int dsa_switch_setup_one(struct dsa_switch *ds, struct device *parent) 
		valid_name_found = true;

	}



	if (!valid_name_found && i == DSA_MAX_PORTS) {

		ret = -EINVAL;

		goto out;

	}

	if (!valid_name_found && i == DSA_MAX_PORTS)

		return -EINVAL;



	/* Make the built-in MII bus mask match the number of ports,

	 * switch drivers can override this later
@@ -363,10 +360,8 @@ static int dsa_switch_setup_one(struct dsa_switch *ds, struct device *parent) 


		tag_protocol = ops->get_tag_protocol(ds);

		dst->tag_ops = dsa_resolve_tag_protocol(tag_protocol);

		if (IS_ERR(dst->tag_ops)) {

			ret = PTR_ERR(dst->tag_ops);

			goto out;

		}

		if (IS_ERR(dst->tag_ops))

			return PTR_ERR(dst->tag_ops);



		dst->rcv = dst->tag_ops->rcv;

	}
@@ -378,25 +373,23 @@ static int dsa_switch_setup_one(struct dsa_switch *ds, struct device *parent) 
	 */

	ret = ops->setup(ds);

	if (ret < 0)

		goto out;

		return ret;



	if (ops->set_addr) {

		ret = ops->set_addr(ds, dst->master_netdev->dev_addr);

		if (ret < 0)

			goto out;

			return ret;

	}



	if (!ds->slave_mii_bus && ops->phy_read) {

		ds->slave_mii_bus = devm_mdiobus_alloc(parent);

		if (!ds->slave_mii_bus) {

			ret = -ENOMEM;

			goto out;

		}

		if (!ds->slave_mii_bus)

			return -ENOMEM;

		dsa_slave_mii_bus_init(ds);



		ret = mdiobus_register(ds->slave_mii_bus);

		if (ret < 0)

			goto out;

			return ret;

	}



	/*
@@ -409,20 +402,16 @@ static int dsa_switch_setup_one(struct dsa_switch *ds, struct device *parent) 
			continue;



		ret = dsa_slave_create(ds, parent, i, cd->port_names[i]);

		if (ret < 0) {

		if (ret < 0)

			netdev_err(dst->master_netdev, "[%d]: can't create dsa slave device for port %d(%s): %d\n",

				   index, i, cd->port_names[i], ret);

			ret = 0;

		}

	}



	/* Perform configuration of the CPU and DSA ports */

	ret = dsa_cpu_dsa_setups(ds, parent);

	if (ret < 0) {

	if (ret < 0)

		netdev_err(dst->master_netdev, "[%d] : can't configure CPU and DSA ports\n",

			   index);

		ret = 0;

	}



	ret = dsa_cpu_port_ethtool_setup(ds);

	if (ret)
@@ -453,10 +442,7 @@ static int dsa_switch_setup_one(struct dsa_switch *ds, struct device *parent) 
	}

#endif /* CONFIG_NET_DSA_HWMON */



	return ret;



out:

	return ret;

	return 0;

}



static struct dsa_switch *