Staging: emxx_udc: Remove useless type conversion
authorBhumika Goyal <bhumirks@gmail.com>
Sat, 8 Oct 2016 09:59:54 +0000 (15:29 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 16 Oct 2016 08:25:07 +0000 (10:25 +0200)
Some type conversions like casting a pointer to a pointer of same type,
casting to the original type using addressof(&) operator etc. are not
needed. Therefore, remove them. Done using coccinelle:

@@
type t;
t *p;
t a;
@@
(
- (t)(a)
+ a
|
- (t *)(p)
+ p
|
- (t *)(&a)
+ &a
)

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/emxx_udc/emxx_udc.c

index c3e2988..5d7dc00 100644 (file)
@@ -460,7 +460,7 @@ static void _nbu2ss_ep_in_end(
                if (length)
                        _nbu2ss_writel(&preg->EP_REGS[num].EP_WRITE, data32);
 
-               data = ((((u32)length) << 5) & EPn_DW) | EPn_DEND;
+               data = (((length) << 5) & EPn_DW) | EPn_DEND;
                _nbu2ss_bitset(&preg->EP_REGS[num].EP_CONTROL, data);
 
                _nbu2ss_bitset(&preg->EP_REGS[num].EP_CONTROL, EPn_AUTO);
@@ -1852,7 +1852,7 @@ static inline void _nbu2ss_ep0_int(struct nbu2ss_udc *udc)
 
        status = _nbu2ss_readl(&udc->p_regs->EP0_STATUS);
        intr = status & EP0_STATUS_RW_BIT;
-       _nbu2ss_writel(&udc->p_regs->EP0_STATUS, ~(u32)intr);
+       _nbu2ss_writel(&udc->p_regs->EP0_STATUS, ~intr);
 
        status &= (SETUP_INT | EP0_IN_INT | EP0_OUT_INT
                        | STG_END_INT | EP0_OUT_NULL_INT);
@@ -2127,7 +2127,7 @@ static inline void _nbu2ss_epn_int(struct nbu2ss_udc *udc, u32 epnum)
        status = _nbu2ss_readl(&udc->p_regs->EP_REGS[num].EP_STATUS);
 
        /* Interrupt Clear */
-       _nbu2ss_writel(&udc->p_regs->EP_REGS[num].EP_STATUS, ~(u32)status);
+       _nbu2ss_writel(&udc->p_regs->EP_REGS[num].EP_STATUS, ~status);
 
        req = list_first_entry_or_null(&ep->queue, struct nbu2ss_req, queue);
        if (!req) {