]> Pileus Git - ~andy/linux/commitdiff
dmaengine: mmp_pdma: fix mismerge
authorArnd Bergmann <arnd@arndb.de>
Tue, 28 Jan 2014 20:45:38 +0000 (21:45 +0100)
committerVinod Koul <vinod.koul@intel.com>
Wed, 29 Jan 2014 12:08:57 +0000 (17:38 +0530)
The merge between 2b7f65b11d87f "mmp_pdma: Style neatening" and
8010dad55a0ab0 "dma: add dma_get_any_slave_channel(), for use in of_xlate()"
caused a build error by leaving obsolete code in place:

  mmp_pdma.c: In function 'mmp_pdma_dma_xlate':
  mmp_pdma.c:909:31: error: 'candidate' undeclared
  mmp_pdma.c:912:3: error: label 'retry' used but not defined
  mmp_pdma.c:901:24: warning: unused variable 'c' [-Wunused-variable]

This removes the extraneous lines.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
drivers/dma/mmp_pdma.c

index c1b7c3ac7676222f5e02785970e44ec6cdb210ea..b439679f4126e98dcd6484607971276b2baf512f 100644 (file)
@@ -899,19 +899,11 @@ static struct dma_chan *mmp_pdma_dma_xlate(struct of_phandle_args *dma_spec,
 {
        struct mmp_pdma_device *d = ofdma->of_dma_data;
        struct dma_chan *chan;
-       struct mmp_pdma_chan *c;
 
        chan = dma_get_any_slave_channel(&d->device);
        if (!chan)
                return NULL;
 
-       /* dma_get_slave_channel will return NULL if we lost a race between
-        * the lookup and the reservation */
-       chan = dma_get_slave_channel(candidate);
-
-       if (!chan)
-               goto retry;
-
        to_mmp_pdma_chan(chan)->drcmr = dma_spec->args[0];
 
        return chan;