diff options
| author | Vinod Koul <vinod.koul@intel.com> | 2011-07-25 15:52:01 +0200 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2011-07-26 12:03:29 +0200 |
| commit | f32807f1ff7fbfd2d4ec708b1ac8cb75cb92bfef (patch) | |
| tree | 543d0cacbc1a3e1b146037e5f17d62116f19ac0c /drivers/dma | |
| parent | DMA: PL08x: cleanup selection of burst size (diff) | |
| download | linux-f32807f1ff7fbfd2d4ec708b1ac8cb75cb92bfef.tar.xz linux-f32807f1ff7fbfd2d4ec708b1ac8cb75cb92bfef.zip | |
dmaengine: pl08x: handle the rest of enums in pl08x_width
pl08x_width function does not handle rest of enums for DMA_SLAVE_BUSWIDTH_xxxx
which causes gcc to emit below warining
drivers/dma/amba-pl08x.c: In function 'pl08x_width':
drivers/dma/amba-pl08x.c:1119: warning: enumeration value
'DMA_SLAVE_BUSWIDTH_UNDEFINED' not handled in switch
drivers/dma/amba-pl08x.c:1119: warning: enumeration value
'DMA_SLAVE_BUSWIDTH_8_BYTES' not handled in switch
this patch adds a default case which returns error
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
Diffstat (limited to 'drivers/dma')
| -rw-r--r-- | drivers/dma/amba-pl08x.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c index 9aa2bd4452d3..196a7378d332 100644 --- a/drivers/dma/amba-pl08x.c +++ b/drivers/dma/amba-pl08x.c @@ -1123,8 +1123,9 @@ static u32 pl08x_width(enum dma_slave_buswidth width) return PL080_WIDTH_16BIT; case DMA_SLAVE_BUSWIDTH_4_BYTES: return PL080_WIDTH_32BIT; + default: + return ~0; } - return ~0; } static u32 pl08x_burst(u32 maxburst) |