sfc: Make efx_mcdi_{init,fini}() call efx_mcdi_drv_attach()
authorBen Hutchings <bhutchings@solarflare.com>
Thu, 29 Aug 2013 18:04:03 +0000 (19:04 +0100)
committerBen Hutchings <bhutchings@solarflare.com>
Thu, 29 Aug 2013 18:06:34 +0000 (19:06 +0100)
This should be done during MCDI initialisation for any NIC.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
drivers/net/ethernet/sfc/mcdi.c
drivers/net/ethernet/sfc/mcdi.h
drivers/net/ethernet/sfc/siena.c

index 8150781b41eb70d35852a5a94572a3f3c094c9fc..d8a20f514f80fe92e9ae92dc689043c5512d0325 100644 (file)
@@ -48,6 +48,8 @@ struct efx_mcdi_async_param {
 };
 
 static void efx_mcdi_timeout_async(unsigned long context);
+static int efx_mcdi_drv_attach(struct efx_nic *efx, bool driver_operating,
+                              bool *was_attached_out);
 
 static inline struct efx_mcdi_iface *efx_mcdi(struct efx_nic *efx)
 {
@@ -58,6 +60,8 @@ static inline struct efx_mcdi_iface *efx_mcdi(struct efx_nic *efx)
 int efx_mcdi_init(struct efx_nic *efx)
 {
        struct efx_mcdi_iface *mcdi;
+       bool already_attached;
+       int rc;
 
        efx->mcdi = kzalloc(sizeof(*efx->mcdi), GFP_KERNEL);
        if (!efx->mcdi)
@@ -78,12 +82,37 @@ int efx_mcdi_init(struct efx_nic *efx)
        mcdi->new_epoch = true;
 
        /* Recover from a failed assertion before probing */
-       return efx_mcdi_handle_assertion(efx);
+       rc = efx_mcdi_handle_assertion(efx);
+       if (rc)
+               return rc;
+
+       /* Let the MC (and BMC, if this is a LOM) know that the driver
+        * is loaded. We should do this before we reset the NIC.
+        */
+       rc = efx_mcdi_drv_attach(efx, true, &already_attached);
+       if (rc) {
+               netif_err(efx, probe, efx->net_dev,
+                         "Unable to register driver with MCPU\n");
+               return rc;
+       }
+       if (already_attached)
+               /* Not a fatal error */
+               netif_err(efx, probe, efx->net_dev,
+                         "Host already registered with MCPU\n");
+
+       return 0;
 }
 
 void efx_mcdi_fini(struct efx_nic *efx)
 {
-       BUG_ON(efx->mcdi && efx->mcdi->iface.state != MCDI_STATE_QUIESCENT);
+       if (!efx->mcdi)
+               return;
+
+       BUG_ON(efx->mcdi->iface.state != MCDI_STATE_QUIESCENT);
+
+       /* Relinquish the device (back to the BMC, if this is a LOM) */
+       efx_mcdi_drv_attach(efx, false, NULL);
+
        kfree(efx->mcdi);
 }
 
@@ -889,8 +918,8 @@ fail:
        buf[0] = 0;
 }
 
-int efx_mcdi_drv_attach(struct efx_nic *efx, bool driver_operating,
-                       bool *was_attached)
+static int efx_mcdi_drv_attach(struct efx_nic *efx, bool driver_operating,
+                              bool *was_attached)
 {
        MCDI_DECLARE_BUF(inbuf, MC_CMD_DRV_ATTACH_IN_LEN);
        MCDI_DECLARE_BUF(outbuf, MC_CMD_DRV_ATTACH_OUT_LEN);
index e37cf1d6ed46a55baad8caecbad82f095d3d6936..0ca00a631986323f9a278cda2fe22bd85ae9d253 100644 (file)
@@ -273,8 +273,6 @@ extern void efx_mcdi_sensor_event(struct efx_nic *efx, efx_qword_t *ev);
        EFX_QWORD_FIELD(_ev, MCDI_EVENT_ ## _field)
 
 extern void efx_mcdi_print_fwver(struct efx_nic *efx, char *buf, size_t len);
-extern int efx_mcdi_drv_attach(struct efx_nic *efx, bool driver_operating,
-                              bool *was_attached_out);
 extern int efx_mcdi_get_board_cfg(struct efx_nic *efx, u8 *mac_address,
                                  u16 *fw_subtype_list, u32 *capabilities);
 extern int efx_mcdi_log_ctrl(struct efx_nic *efx, bool evq, bool uart,
index 1500405b3a5540d5b2f7b5a0a2603ee458821f10..e8eef63a55eef3d18f51582a820a5f91e4c2385f 100644 (file)
@@ -196,7 +196,6 @@ static unsigned int siena_mem_map_size(struct efx_nic *efx)
 static int siena_probe_nic(struct efx_nic *efx)
 {
        struct siena_nic_data *nic_data;
-       bool already_attached = false;
        efx_oword_t reg;
        int rc;
 
@@ -222,19 +221,6 @@ static int siena_probe_nic(struct efx_nic *efx)
        if (rc)
                goto fail1;
 
-       /* Let the BMC know that the driver is now in charge of link and
-        * filter settings. We must do this before we reset the NIC */
-       rc = efx_mcdi_drv_attach(efx, true, &already_attached);
-       if (rc) {
-               netif_err(efx, probe, efx->net_dev,
-                         "Unable to register driver with MCPU\n");
-               goto fail2;
-       }
-       if (already_attached)
-               /* Not a fatal error */
-               netif_err(efx, probe, efx->net_dev,
-                         "Host already registered with MCPU\n");
-
        /* Now we can reset the NIC */
        rc = efx_mcdi_reset(efx, RESET_TYPE_ALL);
        if (rc) {
@@ -281,8 +267,6 @@ fail5:
        efx_nic_free_buffer(efx, &efx->irq_status);
 fail4:
 fail3:
-       efx_mcdi_drv_attach(efx, false, NULL);
-fail2:
        efx_mcdi_fini(efx);
 fail1:
        kfree(efx->nic_data);
@@ -371,14 +355,11 @@ static void siena_remove_nic(struct efx_nic *efx)
 
        efx_mcdi_reset(efx, RESET_TYPE_ALL);
 
-       /* Relinquish the device back to the BMC */
-       efx_mcdi_drv_attach(efx, false, NULL);
+       efx_mcdi_fini(efx);
 
        /* Tear down the private nic state */
        kfree(efx->nic_data);
        efx->nic_data = NULL;
-
-       efx_mcdi_fini(efx);
 }
 
 #define SIENA_DMA_STAT(ext_name, mcdi_name)                    \