> >-    if (!of_node && dev->parent->of_node)
> >-            of_node = dev->parent->of_node;
> >+    /* The Micrel driver has a deprecated option to place phy OF
> >+     * properties in the MAC node. Walk up the tree of devices to
> >+     * find a device with an OF node.
> >+     */
> >+    dev_walker = &phydev->dev;
> >+    do {
> >+            of_node = dev_walker->of_node;
> >+            dev_walker = dev_walker->parent;
> >+
> >+    } while (!of_node && dev_walker);
> 
> Looks good to me, can we also issuing à warning so people get a chance to 
> update their Device Tree?

I did wounder about that.

But i think it might be better to wait until there are updated dts
files. No point telling users they need to update when there are no
updated DT files to use.

If Dinh updates the SoCFPGA .dts files then i think it makes sense to
add the warning.

    Andrew
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to