OSDN Git Service

dmaengine: pl08x: handle the rest of enums in pl08x_width
authorVinod Koul <vinod.koul@intel.com>
Mon, 25 Jul 2011 13:52:01 +0000 (19:22 +0530)
committerVinod Koul <vinod.koul@intel.com>
Tue, 26 Jul 2011 10:03:29 +0000 (15:33 +0530)
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>
drivers/dma/amba-pl08x.c

index 9aa2bd4..196a737 100644 (file)
@@ -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)