mmc: core: Refactor code to register the MMC PM notifier
authorUlf Hansson <ulf.hansson@linaro.org>
Thu, 5 Nov 2015 15:11:12 +0000 (16:11 +0100)
committerUlf Hansson <ulf.hansson@linaro.org>
Tue, 22 Dec 2015 10:32:04 +0000 (11:32 +0100)
Instead of checking for "#ifdef" directly in the code, let's invent a pair
of mmc core functions to deal with register/unregister the MMC PM notifier
block. Implement stubs for these functions when CONFIG_PM_SLEEP is unset,
as in that case the PM notifiers isn't used.

Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/core/core.c
drivers/mmc/core/core.h
drivers/mmc/core/host.c
include/linux/mmc/host.h

index 394963d3fb1891a1fa80daa3997ca747cd7bde36..18a3f2c411365817436794d655175f405bc89902 100644 (file)
@@ -2760,14 +2760,13 @@ int mmc_flush_cache(struct mmc_card *card)
 }
 EXPORT_SYMBOL(mmc_flush_cache);
 
-#ifdef CONFIG_PM
-
+#ifdef CONFIG_PM_SLEEP
 /* Do the card removal on suspend if card is assumed removeable
  * Do that in pm notifier while userspace isn't yet frozen, so we will be able
    to sync the card.
 */
-int mmc_pm_notify(struct notifier_block *notify_block,
-                                       unsigned long mode, void *unused)
+static int mmc_pm_notify(struct notifier_block *notify_block,
+                       unsigned long mode, void *unused)
 {
        struct mmc_host *host = container_of(
                notify_block, struct mmc_host, pm_notify);
@@ -2814,6 +2813,17 @@ int mmc_pm_notify(struct notifier_block *notify_block,
 
        return 0;
 }
+
+void mmc_register_pm_notifier(struct mmc_host *host)
+{
+       host->pm_notify.notifier_call = mmc_pm_notify;
+       register_pm_notifier(&host->pm_notify);
+}
+
+void mmc_unregister_pm_notifier(struct mmc_host *host)
+{
+       unregister_pm_notifier(&host->pm_notify);
+}
 #endif
 
 /**
index 09241e56d62872ec3b4c3b6cc24ef815e29ed3d8..bd00ac2cdc1b60126c779abf5ab2616d72bf26c7 100644 (file)
@@ -90,5 +90,13 @@ int mmc_execute_tuning(struct mmc_card *card);
 int mmc_hs200_to_hs400(struct mmc_card *card);
 int mmc_hs400_to_hs200(struct mmc_card *card);
 
+#ifdef CONFIG_PM_SLEEP
+void mmc_register_pm_notifier(struct mmc_host *host);
+void mmc_unregister_pm_notifier(struct mmc_host *host);
+#else
+static void mmc_register_pm_notifier(struct mmc_host *host) { }
+static void mmc_unregister_pm_notifier(struct mmc_host *host) { }
+#endif
+
 #endif
 
index e2cad924a26f69b127300abeb30e08beafcd0142..0aecd5c00b866571a852c19c91af766591d4025e 100644 (file)
@@ -21,7 +21,6 @@
 #include <linux/export.h>
 #include <linux/leds.h>
 #include <linux/slab.h>
-#include <linux/suspend.h>
 
 #include <linux/mmc/host.h>
 #include <linux/mmc/card.h>
@@ -349,9 +348,6 @@ struct mmc_host *mmc_alloc_host(int extra, struct device *dev)
        spin_lock_init(&host->lock);
        init_waitqueue_head(&host->wq);
        INIT_DELAYED_WORK(&host->detect, mmc_rescan);
-#ifdef CONFIG_PM
-       host->pm_notify.notifier_call = mmc_pm_notify;
-#endif
        setup_timer(&host->retune_timer, mmc_retune_timer, (unsigned long)host);
 
        /*
@@ -396,7 +392,7 @@ int mmc_add_host(struct mmc_host *host)
 #endif
 
        mmc_start_host(host);
-       register_pm_notifier(&host->pm_notify);
+       mmc_register_pm_notifier(host);
 
        return 0;
 }
@@ -413,7 +409,7 @@ EXPORT_SYMBOL(mmc_add_host);
  */
 void mmc_remove_host(struct mmc_host *host)
 {
-       unregister_pm_notifier(&host->pm_notify);
+       mmc_unregister_pm_notifier(host);
        mmc_stop_host(host);
 
 #ifdef CONFIG_DEBUG_FS
index 9b04e717bb869c691d9dfdbec127e5115bb73377..f39888cf501795673b378db1d6bf33534351fd8a 100644 (file)
@@ -212,7 +212,9 @@ struct mmc_host {
        u32                     ocr_avail_sdio; /* SDIO-specific OCR */
        u32                     ocr_avail_sd;   /* SD-specific OCR */
        u32                     ocr_avail_mmc;  /* MMC-specific OCR */
+#ifdef CONFIG_PM_SLEEP
        struct notifier_block   pm_notify;
+#endif
        u32                     max_current_330;
        u32                     max_current_300;
        u32                     max_current_180;
@@ -433,8 +435,6 @@ static inline int mmc_regulator_set_vqmmc(struct mmc_host *mmc,
 
 int mmc_regulator_get_supply(struct mmc_host *mmc);
 
-int mmc_pm_notify(struct notifier_block *notify_block, unsigned long, void *);
-
 static inline int mmc_card_is_removable(struct mmc_host *host)
 {
        return !(host->caps & MMC_CAP_NONREMOVABLE);