sfc: stop setting dev_port
authorBert Kenward <bkenward@solarflare.com>
Tue, 10 Jan 2017 16:24:31 +0000 (16:24 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 10 Jan 2017 19:16:17 +0000 (14:16 -0500)
Setting dev_port changes the device names allocated by systemd. Any devices
with a dev_port >0 will (in default distro configurations) have a suffix of
"d<port-number>" appended.

This is not something done by other drivers, and causes confusion for users.

Fixes: 8be41320f346 ("sfc: Add code to export port_num in netdev->dev_port")
Signed-off-by: Edward Cree <ecree@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sfc/ef10.c

index 4f3c965f5997bcf485d5fac6cb2d898cb2e9d34f..208e0048709b82723d839f98a560936a921bd00c 100644 (file)
@@ -547,7 +547,6 @@ static DEVICE_ATTR(primary_flag, 0444, efx_ef10_show_primary_flag, NULL);
 static int efx_ef10_probe(struct efx_nic *efx)
 {
        struct efx_ef10_nic_data *nic_data;
-       struct net_device *net_dev = efx->net_dev;
        int i, rc;
 
        /* We can have one VI for each 8K region.  However, until we
@@ -637,7 +636,6 @@ static int efx_ef10_probe(struct efx_nic *efx)
        if (rc < 0)
                goto fail5;
        efx->port_num = rc;
-       net_dev->dev_port = rc;
 
        rc = efx->type->get_mac_address(efx, efx->net_dev->perm_addr);
        if (rc)