From f639554b8a3a4aac18193a0c98a4639d432c48f0 Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman Date: Thu, 10 May 2012 11:37:18 -0700 Subject: [PATCH] USB: gadget: ci13xx_udc: fix potential crash in irq handler As pointed out by Dan Carpenter, we should not be trying to call dev_err() on a structure that was previously determined to be NULL, that's just foolish and asking for trouble. So just delete the message, it's not going to do anyone any good to have it anyway. This problem was caused by 0f089094cde5: "usb: gadget: ci13xxx: replace home-brewed logging with dev_{err,warn,info}" Reported-by: Dan Carpenter Cc: Alexander Shishkin Signed-off-by: Greg Kroah-Hartman --- drivers/usb/gadget/ci13xxx_udc.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/usb/gadget/ci13xxx_udc.c b/drivers/usb/gadget/ci13xxx_udc.c index 560b539b9def..5bd95b0a23d7 100644 --- a/drivers/usb/gadget/ci13xxx_udc.c +++ b/drivers/usb/gadget/ci13xxx_udc.c @@ -2719,10 +2719,8 @@ static irqreturn_t udc_irq(int irq, void *data) trace(udc ? udc->dev : NULL, ""); - if (udc == NULL) { - dev_err(udc->dev, "ENODEV"); + if (udc == NULL) return IRQ_HANDLED; - } spin_lock(&udc->lock); -- 2.39.5