csr);
 }
 
-static int dma_channel_program(struct dma_channel * pChannel,
+static int dma_channel_program(struct dma_channel *pChannel,
                                u16 packet_sz, u8 mode,
                                dma_addr_t dma_addr, u32 len)
 {
                if (pImplChannel->transmit) {
 
                        csr = musb_readw(mbase,
-                               MUSB_EP_OFFSET(pImplChannel->epnum,MUSB_TXCSR));
+                               MUSB_EP_OFFSET(pImplChannel->epnum, MUSB_TXCSR));
                        csr &= ~(MUSB_TXCSR_AUTOSET |
                                 MUSB_TXCSR_DMAENAB |
                                 MUSB_TXCSR_DMAMODE);
                        musb_writew(mbase,
-                               MUSB_EP_OFFSET(pImplChannel->epnum,MUSB_TXCSR),
+                               MUSB_EP_OFFSET(pImplChannel->epnum, MUSB_TXCSR),
                                csr);
-               }
-               else {
+               } else {
                        csr = musb_readw(mbase,
-                               MUSB_EP_OFFSET(pImplChannel->epnum,MUSB_RXCSR));
+                               MUSB_EP_OFFSET(pImplChannel->epnum, MUSB_RXCSR));
                        csr &= ~(MUSB_RXCSR_AUTOCLEAR |
                                 MUSB_RXCSR_DMAENAB |
                                 MUSB_RXCSR_DMAMODE);
                        musb_writew(mbase,
-                               MUSB_EP_OFFSET(pImplChannel->epnum,MUSB_RXCSR),
+                               MUSB_EP_OFFSET(pImplChannel->epnum, MUSB_RXCSR),
                                csr);
                }
 
                return NULL;
        }
 
-       if (!(controller = kzalloc(sizeof(struct musb_dma_controller),
-                               GFP_KERNEL)))
+       controller = kzalloc(sizeof(struct musb_dma_controller), GFP_KERNEL);
+       if (!controller)
                return NULL;
 
        controller->bChannelCount = MUSB_HSDMA_CHANNELS;