diff options
| author | Biju Das <biju.das.jz@bp.renesas.com> | 2025-02-05 15:42:22 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2025-02-07 16:43:55 +0300 |
| commit | 46df19a8dfdf48c9ef6fe1fc66e912eabb0213ed (patch) | |
| tree | a0c0ccf8fb0a95cf93540fb47bf5250334a18902 | |
| parent | 8d3bbe4355aded32961b9009b31de6d41b7352e9 (diff) | |
| download | linux-46df19a8dfdf48c9ef6fe1fc66e912eabb0213ed.tar.xz | |
net: dsa: rzn1_a5psw: Use of_get_available_child_by_name()
Simplify a5psw_probe() by using of_get_available_child_by_name().
While at it, move of_node_put(mdio) inside the if block to avoid code
duplication.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/dsa/rzn1_a5psw.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/net/dsa/rzn1_a5psw.c b/drivers/net/dsa/rzn1_a5psw.c index 66974379334a..31ea8130a495 100644 --- a/drivers/net/dsa/rzn1_a5psw.c +++ b/drivers/net/dsa/rzn1_a5psw.c @@ -1248,18 +1248,16 @@ static int a5psw_probe(struct platform_device *pdev) if (ret) goto clk_disable; - mdio = of_get_child_by_name(dev->of_node, "mdio"); - if (of_device_is_available(mdio)) { + mdio = of_get_available_child_by_name(dev->of_node, "mdio"); + if (mdio) { ret = a5psw_probe_mdio(a5psw, mdio); + of_node_put(mdio); if (ret) { - of_node_put(mdio); dev_err(dev, "Failed to register MDIO: %d\n", ret); goto hclk_disable; } } - of_node_put(mdio); - ds = &a5psw->ds; ds->dev = dev; ds->num_ports = A5PSW_PORTS_NUM; |