libertas CS: convert to asynchronous firmware loading
authorDaniel Drake <dsd@laptop.org>
Wed, 18 Apr 2012 19:09:44 +0000 (20:09 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 23 Apr 2012 19:34:06 +0000 (15:34 -0400)
Signed-off-by: Daniel Drake <dsd@laptop.org>
Tested-by: Dan Williams <dcbw@redhat.com>
Acked-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/libertas/if_cs.c

index cee50528522b863e7c19b8bdf3be6673519f3c74..16beaf39dc534a167f4ee783ce913903fd174eb8 100644 (file)
@@ -738,6 +738,50 @@ done:
        return ret;
 }
 
+static void if_cs_prog_firmware(struct lbs_private *priv, int ret,
+                                const struct firmware *helper,
+                                const struct firmware *mainfw)
+{
+       struct if_cs_card *card = priv->card;
+
+       if (ret) {
+               pr_err("failed to find firmware (%d)\n", ret);
+               return;
+       }
+
+       /* Load the firmware */
+       ret = if_cs_prog_helper(card, helper);
+       if (ret == 0 && (card->model != MODEL_8305))
+               ret = if_cs_prog_real(card, mainfw);
+       if (ret)
+               goto out;
+
+       /* Now actually get the IRQ */
+       ret = request_irq(card->p_dev->irq, if_cs_interrupt,
+               IRQF_SHARED, DRV_NAME, card);
+       if (ret) {
+               pr_err("error in request_irq\n");
+               goto out;
+       }
+
+       /*
+        * Clear any interrupt cause that happened while sending
+        * firmware/initializing card
+        */
+       if_cs_write16(card, IF_CS_CARD_INT_CAUSE, IF_CS_BIT_MASK);
+       if_cs_enable_ints(card);
+
+       /* And finally bring the card up */
+       priv->fw_ready = 1;
+       if (lbs_start_card(priv) != 0) {
+               pr_err("could not activate card\n");
+               free_irq(card->p_dev->irq, card);
+       }
+
+out:
+       release_firmware(helper);
+       release_firmware(mainfw);
+}
 
 
 /********************************************************************/
@@ -809,8 +853,6 @@ static int if_cs_probe(struct pcmcia_device *p_dev)
        unsigned int prod_id;
        struct lbs_private *priv;
        struct if_cs_card *card;
-       const struct firmware *helper = NULL;
-       const struct firmware *mainfw = NULL;
 
        lbs_deb_enter(LBS_DEB_CS);
 
@@ -890,20 +932,6 @@ static int if_cs_probe(struct pcmcia_device *p_dev)
                goto out2;
        }
 
-       ret = lbs_get_firmware(&p_dev->dev, card->model, &fw_table[0],
-                               &helper, &mainfw);
-       if (ret) {
-               pr_err("failed to find firmware (%d)\n", ret);
-               goto out2;
-       }
-
-       /* Load the firmware early, before calling into libertas.ko */
-       ret = if_cs_prog_helper(card, helper);
-       if (ret == 0 && (card->model != MODEL_8305))
-               ret = if_cs_prog_real(card, mainfw);
-       if (ret)
-               goto out2;
-
        /* Make this card known to the libertas driver */
        priv = lbs_add_card(card, &p_dev->dev);
        if (!priv) {
@@ -911,37 +939,22 @@ static int if_cs_probe(struct pcmcia_device *p_dev)
                goto out2;
        }
 
-       /* Finish setting up fields in lbs_private */
+       /* Set up fields in lbs_private */
        card->priv = priv;
        priv->card = card;
        priv->hw_host_to_card = if_cs_host_to_card;
        priv->enter_deep_sleep = NULL;
        priv->exit_deep_sleep = NULL;
        priv->reset_deep_sleep_wakeup = NULL;
-       priv->fw_ready = 1;
 
-       /* Now actually get the IRQ */
-       ret = request_irq(p_dev->irq, if_cs_interrupt,
-               IRQF_SHARED, DRV_NAME, card);
+       /* Get firmware */
+       ret = lbs_get_firmware_async(priv, &p_dev->dev, card->model, fw_table,
+                                    if_cs_prog_firmware);
        if (ret) {
-               pr_err("error in request_irq\n");
-               goto out3;
-       }
-
-       /*
-        * Clear any interrupt cause that happened while sending
-        * firmware/initializing card
-        */
-       if_cs_write16(card, IF_CS_CARD_INT_CAUSE, IF_CS_BIT_MASK);
-       if_cs_enable_ints(card);
-
-       /* And finally bring the card up */
-       if (lbs_start_card(priv) != 0) {
-               pr_err("could not activate card\n");
+               pr_err("failed to find firmware (%d)\n", ret);
                goto out3;
        }
 
-       ret = 0;
        goto out;
 
 out3:
@@ -951,9 +964,6 @@ out2:
 out1:
        pcmcia_disable_device(p_dev);
 out:
-       release_firmware(helper);
-       release_firmware(mainfw);
-
        lbs_deb_leave_args(LBS_DEB_CS, "ret %d", ret);
        return ret;
 }