sfc: don't use netif_info et al before net_device is registered
authorHeiner Kallweit <hkallweit1@gmail.com>
Thu, 13 May 2021 21:29:12 +0000 (23:29 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 14 May 2021 21:59:59 +0000 (14:59 -0700)
Using netif_info() before the net_device is registered results in ugly
messages like the following:
sfc 0000:01:00.1 (unnamed net_device) (uninitialized): Solarflare NIC detected
Therefore use pci_info() et al until net_device is registered.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sfc/efx.c

index c746ca7235f1cb18a863b6334ba1a7f95fff589c..4fd9903ffe9862fb65a3ce194d6918e54fe1baeb 100644 (file)
@@ -722,8 +722,7 @@ static int efx_register_netdev(struct efx_nic *efx)
        efx->state = STATE_READY;
        smp_mb(); /* ensure we change state before checking reset_pending */
        if (efx->reset_pending) {
-               netif_err(efx, probe, efx->net_dev,
-                         "aborting probe due to scheduled reset\n");
+               pci_err(efx->pci_dev, "aborting probe due to scheduled reset\n");
                rc = -EIO;
                goto fail_locked;
        }
@@ -990,8 +989,7 @@ static int efx_pci_probe_main(struct efx_nic *efx)
        rc = efx->type->init(efx);
        up_write(&efx->filter_sem);
        if (rc) {
-               netif_err(efx, probe, efx->net_dev,
-                         "failed to initialise NIC\n");
+               pci_err(efx->pci_dev, "failed to initialise NIC\n");
                goto fail3;
        }
 
@@ -1038,8 +1036,8 @@ static int efx_pci_probe_post_io(struct efx_nic *efx)
        if (efx->type->sriov_init) {
                rc = efx->type->sriov_init(efx);
                if (rc)
-                       netif_err(efx, probe, efx->net_dev,
-                                 "SR-IOV can't be enabled rc %d\n", rc);
+                       pci_err(efx->pci_dev, "SR-IOV can't be enabled rc %d\n",
+                               rc);
        }
 
        /* Determine netdevice features */
@@ -1106,8 +1104,7 @@ static int efx_pci_probe(struct pci_dev *pci_dev,
        if (rc)
                goto fail1;
 
-       netif_info(efx, probe, efx->net_dev,
-                  "Solarflare NIC detected\n");
+       pci_info(pci_dev, "Solarflare NIC detected\n");
 
        if (!efx->type->is_vf)
                efx_probe_vpd_strings(efx);