]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
staging/emxx_udc: Remove pointless variable assignments
authorAlex Dewar <alex.dewar90@gmail.com>
Wed, 2 Sep 2020 15:46:24 +0000 (16:46 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 2 Sep 2020 15:56:55 +0000 (17:56 +0200)
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 <lkp@intel.com>
Fixes: 33aa8d45a4fe ("staging: emxx_udc: Add Emma Mobile USB Gadget driver")
Signed-off-by: Alex Dewar <alex.dewar90@gmail.com>
Link: https://lore.kernel.org/r/20200902154628.283453-1-alex.dewar90@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/emxx_udc/emxx_udc.c

index d0725bc8b48a43e599b7cf1416f04feed9d3240a..4ceaf1ead123b344ad2f82b255d7ece8ce40176a 100644 (file)
@@ -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;