From: Ioana Ciornei Date: Fri, 25 Sep 2020 14:44:19 +0000 (+0300) Subject: dpaa2-mac: do not check for both child and parent DTS nodes X-Git-Tag: howlett/maple_spf/20210104~792^2~143^2~2 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=98179709c95ecd60274668a20365a0cd0b3726eb;p=users%2Fjedix%2Flinux-maple.git dpaa2-mac: do not check for both child and parent DTS nodes There is no need to check if both the MDIO controller node and its child node, the PCS device, are available since there is no chance that the child node would be enabled when the parent it's not. Suggested-by: Andrew Lunn Signed-off-by: Ioana Ciornei Reviewed-by: Andrew Lunn Signed-off-by: David S. Miller --- diff --git a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c index 6ff64dd1cf274..cdd1acd0117eb 100644 --- a/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c +++ b/drivers/net/ethernet/freescale/dpaa2/dpaa2-mac.c @@ -267,8 +267,7 @@ static int dpaa2_pcs_create(struct dpaa2_mac *mac, return 0; } - if (!of_device_is_available(node) || - !of_device_is_available(node->parent)) { + if (!of_device_is_available(node)) { netdev_err(mac->net_dev, "pcs-handle node not available\n"); return -ENODEV; }