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

Bartosz Golaszewski
2018-08-10T18:48:19Z

2018-08-10 16:39 GMT+02:00 Andy Shevchenko <andy.shevchenko@gmail.com>:
> On Fri, Aug 10, 2018 at 11:05 AM, Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>> From: Bartosz Golaszewski <bgolaszewski@baylibre.com>
>>
>> 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

Indeed seems like it's even more generalized. Thanks for spotting that.

Bart