net: dsa: Removed unneeded of_node_put in felix_parse_ports_node
authorLu Hongfei <luhongfei@vivo.com>
Mon, 10 Jul 2023 03:18:59 +0000 (11:18 +0800)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 11 Jul 2023 08:53:14 +0000 (10:53 +0200)
Remove unnecessary of_node_put from the continue path to prevent
child node from being released twice, which could avoid resource
leak or other unexpected issues.

Signed-off-by: Lu Hongfei <luhongfei@vivo.com>
Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Fixes: de879a016a94 ("net: dsa: felix: add functionality when not all ports are supported")
Link: https://lore.kernel.org/r/20230710031859.36784-1-luhongfei@vivo.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/dsa/ocelot/felix.c

index dee43caee19e3223202be27314e752352d93907c..8da46d284e35649a7b68a4148593e2a03aa5d8c7 100644 (file)
@@ -1286,7 +1286,6 @@ static int felix_parse_ports_node(struct felix *felix,
                if (err < 0) {
                        dev_info(dev, "Unsupported PHY mode %s on port %d\n",
                                 phy_modes(phy_mode), port);
-                       of_node_put(child);
 
                        /* Leave port_phy_modes[port] = 0, which is also
                         * PHY_INTERFACE_MODE_NA. This will perform a