dmaengine: s3c24xx: remove unused ‘cdata’
authorVinod Koul <vinod.koul@intel.com>
Fri, 9 Dec 2016 09:54:12 +0000 (15:24 +0530)
committerVinod Koul <vinod.koul@intel.com>
Mon, 12 Dec 2016 16:55:22 +0000 (22:25 +0530)
In s3c24xx_dma_get_phy(), variable ‘cdata’ is initialized but never used, which
leads to warning with W=1. Removing this make pdata unused.

drivers/dma/s3c24xx-dma.c: In function ‘s3c24xx_dma_get_phy’:
drivers/dma/s3c24xx-dma.c:293:30: warning: variable ‘cdata’ set but not used [-Wunused-but-set-variable]
  struct s3c24xx_dma_channel *cdata;

So remove both of them.

Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/s3c24xx-dma.c

index 3c579ab..f04c470 100644 (file)
@@ -289,16 +289,11 @@ static
 struct s3c24xx_dma_phy *s3c24xx_dma_get_phy(struct s3c24xx_dma_chan *s3cchan)
 {
        struct s3c24xx_dma_engine *s3cdma = s3cchan->host;
-       const struct s3c24xx_dma_platdata *pdata = s3cdma->pdata;
-       struct s3c24xx_dma_channel *cdata;
        struct s3c24xx_dma_phy *phy = NULL;
        unsigned long flags;
        int i;
        int ret;
 
-       if (s3cchan->slave)
-               cdata = &pdata->channels[s3cchan->id];
-
        for (i = 0; i < s3cdma->pdata->num_phy_channels; i++) {
                phy = &s3cdma->phy_chans[i];