staging: mt7621-mmc: Remove old references to tasklet
authorChristian Lütke-Stetzkamp <christian@lkamp.de>
Tue, 24 Apr 2018 18:01:28 +0000 (20:01 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 25 Apr 2018 13:36:20 +0000 (15:36 +0200)
In the current code there are '#if 0' out references on using a
tasklet instead of delayed_work. Removing these improves readability.

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

index ce14881d8fdaf701766ead7a12913e63f5738f09..ae9b36a0bd4576c48a050d909235bb7f48d97210 100644 (file)
@@ -931,7 +931,6 @@ struct msdc_host {
 
        int                         irq;            /* host interrupt */
 
-       struct tasklet_struct       card_tasklet;
 #if 0
        struct work_struct              card_workqueue;
 #else
index 9476728c925c626decd62dc776f61309cbcf0429..ae70f9d0b34630d657af57684259ce6741c06754 100644 (file)
@@ -485,16 +485,10 @@ static void msdc_set_timeout(struct msdc_host *host, u32 ns, u32 clks)
              ns, clks, timeout + 1);
 }
 
-#if 0
-static void msdc_tasklet_card(unsigned long arg)
-{
-       struct msdc_host *host = (struct msdc_host *)arg;
-#else
 static void msdc_tasklet_card(struct work_struct *work)
 {
        struct msdc_host *host = (struct msdc_host *)container_of(work,
                                struct msdc_host, card_delaywork.work);
-#endif
        void __iomem *base = host->base;
        u32 inserted;
        u32 status = 0;
@@ -2210,11 +2204,7 @@ static irqreturn_t msdc_irq(int irq, void *dev_id)
                if (host->mmc->caps & MMC_CAP_NEEDS_POLL)
                        return IRQ_HANDLED;
                IRQ_MSG("MSDC_INT_CDSC irq<0x%.8x>", intsts);
-#if 0 /* ---/+++ by chhung: fix slot mechanical bounce issue */
-               tasklet_hi_schedule(&host->card_tasklet);
-#else
                schedule_delayed_work(&host->card_delaywork, HZ);
-#endif
                /* tuning when plug card ? */
        }
 
@@ -2617,11 +2607,7 @@ static int msdc_drv_probe(struct platform_device *pdev)
        }
        msdc_init_gpd_bd(host, &host->dma);
 
-#if 0
-       tasklet_init(&host->card_tasklet, msdc_tasklet_card, (ulong)host);
-#else
        INIT_DELAYED_WORK(&host->card_delaywork, msdc_tasklet_card);
-#endif
        spin_lock_init(&host->lock);
        msdc_init_hw(host);
 
@@ -2655,12 +2641,7 @@ static int msdc_drv_probe(struct platform_device *pdev)
 release:
        platform_set_drvdata(pdev, NULL);
        msdc_deinit_hw(host);
-
-#if 0
-       tasklet_kill(&host->card_tasklet);
-#else
        cancel_delayed_work_sync(&host->card_delaywork);
-#endif
 
 release_mem:
        if (host->dma.gpd)
@@ -2693,11 +2674,7 @@ static int msdc_drv_remove(struct platform_device *pdev)
        mmc_remove_host(host->mmc);
        msdc_deinit_hw(host);
 
-#if 0
-       tasklet_kill(&host->card_tasklet);
-#else
        cancel_delayed_work_sync(&host->card_delaywork);
-#endif
 
        dma_free_coherent(&pdev->dev, MAX_GPD_NUM * sizeof(struct gpd),
                          host->dma.gpd, host->dma.gpd_addr);