memstick: Prevent memstick host from getting runtime suspended during card detection
authorKai-Heng Feng <kai.heng.feng@canonical.com>
Mon, 5 Nov 2018 08:45:04 +0000 (16:45 +0800)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 17 Dec 2018 07:26:24 +0000 (08:26 +0100)
We can use MEMSTICK_POWER_{ON,OFF} along with pm_runtime_{get,put}
helpers to let memstick host support runtime pm.

The rpm count may go down to zero before the memstick host powers on, so
the host can be runtime suspended.

So before doing card detection, increment the rpm count to avoid the
host gets runtime suspended. Balance the rpm count after card detection
is done.

Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
Tested-by: Oleksandr Natalenko <oleksandr@natalenko.name>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/memstick/core/memstick.c

index 76382c858c35435b98e061a7dda49d5dacad9585..1246d69ba187422610fd65f68fba60a2dbd181dd 100644 (file)
@@ -18,6 +18,7 @@
 #include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/module.h>
+#include <linux/pm_runtime.h>
 
 #define DRIVER_NAME "memstick"
 
@@ -436,6 +437,7 @@ static void memstick_check(struct work_struct *work)
        struct memstick_dev *card;
 
        dev_dbg(&host->dev, "memstick_check started\n");
+       pm_runtime_get_noresume(host->dev.parent);
        mutex_lock(&host->lock);
        if (!host->card) {
                if (memstick_power_on(host))
@@ -479,6 +481,7 @@ out_power_off:
                host->set_param(host, MEMSTICK_POWER, MEMSTICK_POWER_OFF);
 
        mutex_unlock(&host->lock);
+       pm_runtime_put(host->dev.parent);
        dev_dbg(&host->dev, "memstick_check finished\n");
 }