diff options
author | Mahati Chamarthy <mahati.chamarthy@gmail.com> | 2014-09-23 22:09:23 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-23 20:33:23 -0700 |
commit | a3d822b8e3df89a116ce86df6200da9004438518 (patch) | |
tree | 292802c4de2704f4ca3144a6763eae2170e4b4d2 /drivers/staging/media | |
parent | d8e186cebc94ac06682364b9931f38809ce7b1b5 (diff) | |
download | blackbird-op-linux-a3d822b8e3df89a116ce86df6200da9004438518.tar.gz blackbird-op-linux-a3d822b8e3df89a116ce86df6200da9004438518.zip |
Staging: media: omap24xx: Fix else is not useful warning
This fixes the following checkpatch.pl warning:
WARNING: else is not generally useful after a break or return
Signed-off-by: Mahati Chamarthy <mahati.chamarthy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/media')
-rw-r--r-- | drivers/staging/media/omap24xx/omap24xxcam-dma.c | 40 |
1 files changed, 19 insertions, 21 deletions
diff --git a/drivers/staging/media/omap24xx/omap24xxcam-dma.c b/drivers/staging/media/omap24xx/omap24xxcam-dma.c index 9c00776d6583..afb9e0870b6a 100644 --- a/drivers/staging/media/omap24xx/omap24xxcam-dma.c +++ b/drivers/staging/media/omap24xx/omap24xxcam-dma.c @@ -149,8 +149,8 @@ static void omap24xxcam_dmahw_transfer_chain(void __iomem *base, int dmach, */ omap24xxcam_dmahw_transfer_start(base, dmach); break; - } else - ch = (ch + 1) % NUM_CAMDMA_CHANNELS; + } + ch = (ch + 1) % NUM_CAMDMA_CHANNELS; } } @@ -332,15 +332,14 @@ void omap24xxcam_dma_isr(struct omap24xxcam_dma *dma) spin_unlock(&dma->lock); omap24xxcam_dma_stop(dma, csr); return; - } else { - callback = dma->ch_state[dmach].callback; - arg = dma->ch_state[dmach].arg; - dma->free_dmach++; - if (callback) { - spin_unlock(&dma->lock); - (*callback) (dma, csr, arg); - spin_lock(&dma->lock); - } + } + callback = dma->ch_state[dmach].callback; + arg = dma->ch_state[dmach].arg; + dma->free_dmach++; + if (callback) { + spin_unlock(&dma->lock); + (*callback) (dma, csr, arg); + spin_lock(&dma->lock); } } @@ -475,17 +474,16 @@ void omap24xxcam_sgdma_process(struct omap24xxcam_sgdma *sgdma) /* DMA start failed */ spin_unlock_irqrestore(&sgdma->lock, flags); return; - } else { - unsigned long expires; - /* DMA start was successful */ - sg_state->next_sglist++; - sg_state->bytes_read += len; - sg_state->queued_sglist++; - - /* We start the reset timer */ - expires = jiffies + HZ; - mod_timer(&sgdma->reset_timer, expires); } + unsigned long expires; + /* DMA start was successful */ + sg_state->next_sglist++; + sg_state->bytes_read += len; + sg_state->queued_sglist++; + + /* We start the reset timer */ + expires = jiffies + HZ; + mod_timer(&sgdma->reset_timer, expires); } queued_sgdma--; sgslot = (sgslot + 1) % NUM_SG_DMA; |