The UDC pointer is set on successful probe and will never be NULL when
the driver is later unbound so drop the misleading sanity check (and
confused error message).

Signed-off-by: Johan Hovold <[email protected]>
---
 drivers/usb/gadget/udc/fsl_udc_core.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/drivers/usb/gadget/udc/fsl_udc_core.c 
b/drivers/usb/gadget/udc/fsl_udc_core.c
index c4761933ec86..20392409afb1 100644
--- a/drivers/usb/gadget/udc/fsl_udc_core.c
+++ b/drivers/usb/gadget/udc/fsl_udc_core.c
@@ -2539,12 +2539,6 @@ static void fsl_udc_remove(struct platform_device *pdev)
 
        DECLARE_COMPLETION_ONSTACK(done);
 
-       if (!udc_controller) {
-               dev_err(&pdev->dev,
-                       "Driver still in use but removing anyhow\n");
-               return;
-       }
-
        udc_controller->done = &done;
        usb_del_gadget_udc(&udc_controller->gadget);
 
-- 
2.53.0


Reply via email to