Commit 3a3eea20 authored by Jinjie Ruan's avatar Jinjie Ruan Committed by Paolo Abeni
Browse files

net: mv643xx_eth: Simplify with scoped for each OF child loop 



Use scoped for_each_available_child_of_node_scoped() when iterating
over device nodes to make code a bit simpler.

Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Reviewed-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: default avatarJinjie Ruan <ruanjinjie@huawei.com>
Reviewed-by: default avatarFlorian Fainelli <florian.fainelli@broadcom.com>
Signed-off-by: default avatarPaolo Abeni <pabeni@redhat.com>
parent 4078513f

drivers/net/ethernet/marvell/mv643xx_eth.c

+2 −3
Original line number Diff line number Diff line
@@ -2802,7 +2802,7 @@ static int mv643xx_eth_shared_of_add_port(struct platform_device *pdev, 
static int mv643xx_eth_shared_of_probe(struct platform_device *pdev)

{

	struct mv643xx_eth_shared_platform_data *pd;

	struct device_node *pnp, *np = pdev->dev.of_node;

	struct device_node *np = pdev->dev.of_node;

	int ret;



	/* bail out if not registered from DT */
@@ -2816,10 +2816,9 @@ static int mv643xx_eth_shared_of_probe(struct platform_device *pdev) 


	mv643xx_eth_property(np, "tx-checksum-limit", pd->tx_csum_limit);



	for_each_available_child_of_node(np, pnp) {

	for_each_available_child_of_node_scoped(np, pnp) {

		ret = mv643xx_eth_shared_of_add_port(pdev, pnp);

		if (ret) {

			of_node_put(pnp);

			mv643xx_eth_shared_of_remove();

			return ret;

		}