In a number of places in emxx_udc.c, there are variables which are
assigned to without that value ever being used. Remove these pointless
assignments.

Reported-by: kernel test robot <l...@intel.com>
Fixes: 33aa8d45a4fe ("staging: emxx_udc: Add Emma Mobile USB Gadget driver")
Signed-off-by: Alex Dewar <alex.dewa...@gmail.com>
---
 drivers/staging/emxx_udc/emxx_udc.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/emxx_udc/emxx_udc.c 
b/drivers/staging/emxx_udc/emxx_udc.c
index d0725bc8b48a..4ceaf1ead123 100644
--- a/drivers/staging/emxx_udc/emxx_udc.c
+++ b/drivers/staging/emxx_udc/emxx_udc.c
@@ -793,7 +793,7 @@ static int _nbu2ss_out_dma(struct nbu2ss_udc *udc, struct 
nbu2ss_req *req,
        u32             dmacnt;
        u32             burst = 1;
        u32             data;
-       int             result = -EINVAL;
+       int             result;
        struct fc_regs __iomem *preg = udc->p_regs;
 
        if (req->dma_flag)
@@ -1288,8 +1288,6 @@ static void _nbu2ss_set_endpoint_stall(struct nbu2ss_udc 
*udc,
 
                        _nbu2ss_bitset(&preg->EP_REGS[num].EP_CONTROL, data);
                } else {
-                       /* Clear STALL */
-                       ep->stalled = false;
                        if (ep_adrs & USB_DIR_IN) {
                                _nbu2ss_bitclr(&preg->EP_REGS[num].EP_CONTROL
                                                , EPN_ISTL);
@@ -1304,6 +1302,7 @@ static void _nbu2ss_set_endpoint_stall(struct nbu2ss_udc 
*udc,
                                                , data);
                        }
 
+                       /* Clear STALL */
                        ep->stalled = false;
                        if (ep->halted) {
                                ep->halted = false;
@@ -3073,8 +3072,8 @@ static int nbu2ss_drv_contest_init(struct platform_device 
*pdev,
  */
 static int nbu2ss_drv_probe(struct platform_device *pdev)
 {
-       int     status = -ENODEV;
-       struct nbu2ss_udc       *udc;
+       int status;
+       struct nbu2ss_udc *udc;
        int irq;
        void __iomem *mmio_base;
 
-- 
2.28.0

Reply via email to