Messages in this thread | | | Date | Tue, 31 Oct 2023 13:53:25 +0100 | From | Andrew Lunn <> | Subject | Re: [PATCH net-next v2 8/9] microchip: lan865x: add driver support for Microchip's LAN865X MACPHY |
| |
> Ah ok, then it is supposed to be like below, isn't it? > > static int lan865x_set_mac_address(struct net_device *netdev, void *addr) > { > struct sockaddr *address = addr; > int ret; > > if (netif_running(netdev)) > return -EBUSY; > > ret = lan865x_set_hw_macaddr(netdev); > if (ret) > return ret; > > eth_hw_addr_set(netdev, address->sa_data); > > return 0; > }
Yes, that is better. In practice, its probably not an issue, setting the MAC address will never fail, but it is good to get right, just in case.
Andrew
| |