Thread (92 messages) 92 messages, 11 authors, 2018-10-04

Re: [PATCH v2 14/29] net: simplify eth_platform_get_mac_address()

From: Andy Shevchenko <hidden>
Date: 2018-08-10 14:39:25
Also in: linux-arm-kernel, linux-i2c, linux-omap, lkml, netdev

On Fri, Aug 10, 2018 at 11:05 AM, Bartosz Golaszewski [off-list ref] wrote:
From: Bartosz Golaszewski <redacted>

We don't need to use pci_device_to_OF_node() - we can retrieve
dev->of_node directly even for pci devices.
        struct device_node *dp;

-       if (dev_is_pci(dev))
-               dp = pci_device_to_OF_node(to_pci_dev(dev));
-       else
-               dp = dev->of_node;
-
+       dp = dev->of_node;
        addr = NULL;
        if (dp)
                addr = of_get_mac_address(dp);
Looking more at this I could even propose to change all above by

addr = device_get_mac_address(dev, mac_addr, ETH_ALEN);

Thoughts?

-- 
With Best Regards,
Andy Shevchenko
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help