Re: [PATCH] libertas: fix return codes in if_sdio_suspend()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wednesday, June 27, 2018 09:54 AM, Ulf Hansson wrote:
On 26 June 2018 at 22:50, Daniel Mack <daniel@xxxxxxxxxx> wrote:
On Tuesday, June 26, 2018 10:48 PM, Daniel Mack wrote:

@@ -1321,7 +1334,7 @@ static int if_sdio_suspend(struct device *dev)
         if (ret)
                 return ret;
   -     ret = lbs_suspend(card->priv);
+       ret = lbs_suspend(priv);
         if (ret)
                 return ret;
   @@ -1336,6 +1349,9 @@ static int if_sdio_resume(struct device *dev)
         dev_info(dev, "%s: resume: we're back\n", sdio_func_id(func));
   +     if (card->priv->power_up_on_resume)
+               if_sdio_power_on(card);
+

To guarantee firmware is loaded, don't you need the below as well?

wait_event(card->pwron_waitq, priv->fw_ready);

Hmm, yes. I should probably just be calling into if_sdio_power_restore().

I'll test that and resend the patch.


Thanks,
Daniel
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux