staging: mt7621-mmc: Remove unused field enable_cd_eirq from msdc_hw
authorChristian Lütke-Stetzkamp <christian@lkamp.de>
Wed, 18 Apr 2018 15:27:22 +0000 (17:27 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 23 Apr 2018 13:37:48 +0000 (15:37 +0200)
The enable_cd_eirq() field of msdc_hw is never set and only once
checked for not being zero, so it is removed.

Signed-off-by: Christian Lütke-Stetzkamp <christian@lkamp.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/mt7621-mmc/board.h
drivers/staging/mt7621-mmc/sd.c

index 833d8770fc4637fea0d63d4d36679348332cc13c..ce33d599fe29c979adf48a0e207773e4420f0702 100644 (file)
@@ -74,7 +74,6 @@ struct msdc_hw {
        void (*disable_sdio_eirq)(void);
 
        /* external cd irq operations */
-       void (*enable_cd_eirq)(void);
        void (*disable_cd_eirq)(void);
 };
 
index 61f2e79460f58786c5e8b48be8abb60a178d6f7b..f97b3d4526e8d44e6540bf03eeb159d59c26d10c 100644 (file)
@@ -2453,23 +2453,19 @@ static void msdc_enable_cd_irq(struct msdc_host *host, int enable)
        N_MSG(CFG, "CD IRQ Eanable(%d)", enable);
 
        if (enable) {
-               if (hw->enable_cd_eirq) { /* not set, never enter */
-                       hw->enable_cd_eirq();
-               } else {
-                       /* card detection circuit relies on the core power so that the core power
-                        * shouldn't be turned off. Here adds a reference count to keep
-                        * the core power alive.
-                        */
-                       //msdc_vcore_on(host); //did in msdc_init_hw()
+               /* card detection circuit relies on the core power so that the core power
+                * shouldn't be turned off. Here adds a reference count to keep
+                * the core power alive.
+                */
+               //msdc_vcore_on(host); //did in msdc_init_hw()
 
-                       if (hw->config_gpio_pin) /* NULL */
-                               hw->config_gpio_pin(MSDC_CD_PIN, GPIO_PULL_UP);
+               if (hw->config_gpio_pin) /* NULL */
+                       hw->config_gpio_pin(MSDC_CD_PIN, GPIO_PULL_UP);
 
-                       sdr_set_field(MSDC_PS, MSDC_PS_CDDEBOUNCE, DEFAULT_DEBOUNCE);
-                       sdr_set_bits(MSDC_PS, MSDC_PS_CDEN);
-                       sdr_set_bits(MSDC_INTEN, MSDC_INTEN_CDSC);
-                       sdr_set_bits(SDC_CFG, SDC_CFG_INSWKUP);  /* not in document! Fix me */
-               }
+               sdr_set_field(MSDC_PS, MSDC_PS_CDDEBOUNCE, DEFAULT_DEBOUNCE);
+               sdr_set_bits(MSDC_PS, MSDC_PS_CDEN);
+               sdr_set_bits(MSDC_INTEN, MSDC_INTEN_CDSC);
+               sdr_set_bits(SDC_CFG, SDC_CFG_INSWKUP);  /* not in document! Fix me */
        } else {
                if (hw->disable_cd_eirq) {
                        hw->disable_cd_eirq();