driver: net: ldpaa_eth: Set MAC address during interface open
PATCH v3 1/2 driver: net: ldpaa_eth: Set MAC address during interface open
Currently ldpaa ethernet driver rely on DPL file to statically configure mac address for the DPNIs. It is not a correct approach.
Add support setting MAC address from env variable or Random MAC address.
Signed-off-by Prabhakar Kushwaha <[email protected]>
---
Changs for v2: Sending as it is
Changs for v3: Sending as it is
drivers/net/ldpaa_eth/ldpaa_eth.c | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-)
drivers/net/ldpaa_eth/ldpaa_eth.c +3 -12
@@ -220,7 +220,6 @@ static int ldpaa_eth_open(struct eth_device *net_dev, bd_t *bd) {struct ldpaa_eth_priv *priv = (struct ldpaa_eth_priv *)net_dev->priv;struct dpni_queue_attr rx_queue_attr; -uint8_t mac_addr[6 ];int err;if (net_dev->state ==ETH_STATE_ACTIVE )
@@ -240,21 +239,13 @@ static int ldpaa_eth_open(struct eth_device *net_dev, bd_t *bd)if (err) goto err_bind; - err =dpni_get_primary_mac_addr (dflt_mc_io,MC_CMD_NO_FLAGS , - priv->dpni_handle, mac_addr); + err =dpni_add_mac_addr (dflt_mc_io,MC_CMD_NO_FLAGS , + priv->dpni_handle, net_dev->enetaddr);if (err) { -printf ("dpni_get_primary_mac_addr() failed\n" ); +printf ("dpni_add_mac_addr() failed\n" );return err; } -memcpy (net_dev->enetaddr, mac_addr,0x6 ); - -/* setup the MAC address */ -if (net_dev->enetaddr[0 ] &0x01 ) { -printf ("%s: MacAddress is multcast address\n" , __func__); -return 1 ; - } -#ifdef CONFIG_PHYLIB /* TODO Check this path */ err =phy_startup (priv->phydev);
Signature · 1 line
1.9.1
PATCH v3 1/2 driver: net: ldpaa_eth: Set MAC address during interface open
Quoted 10 lines · Prabhakar Kushwaha
Currently ldpaa ethernet driver rely on DPL file to statically configure mac address for the DPNIs. It is not a correct approach.
Add support setting MAC address from env variable or Random MAC address.
Signed-off-by Prabhakar Kushwaha <[email protected]>---
Changs for v2: Sending as it is
Changs for v3: Sending as it is
Applied to u-boot-fsl-qoriq. Awaiting upstream. Thanks.
York