diff options
| author | Sudip Mukherjee <sudipm.mukherjee@gmail.com> | 2015-09-22 09:24:28 -0400 |
|---|---|---|
| committer | Felipe Balbi <balbi@ti.com> | 2015-10-01 13:40:20 -0400 |
| commit | 2e1b7d0c691c63c66d5bf5c097243f3b037adf32 (patch) | |
| tree | 3a2648fd4a46cb3897197e6931bb89de32b9cda3 /drivers/usb/gadget | |
| parent | 580693bb3b66ce299f85f71dee60734438b1cd79 (diff) | |
usb: gadget: amd5536udc: use WARN_ON
Use WARN_ON() instead of halting the kernel with BUG_ON() and also fix
the checkpatch warning.
Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb/gadget')
| -rw-r--r-- | drivers/usb/gadget/udc/amd5536udc.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/usb/gadget/udc/amd5536udc.c b/drivers/usb/gadget/udc/amd5536udc.c index 3f85044be196..6223b1be029f 100644 --- a/drivers/usb/gadget/udc/amd5536udc.c +++ b/drivers/usb/gadget/udc/amd5536udc.c | |||
| @@ -3127,7 +3127,8 @@ static void udc_pci_remove(struct pci_dev *pdev) | |||
| 3127 | 3127 | ||
| 3128 | usb_del_gadget_udc(&udc->gadget); | 3128 | usb_del_gadget_udc(&udc->gadget); |
| 3129 | /* gadget driver must not be registered */ | 3129 | /* gadget driver must not be registered */ |
| 3130 | BUG_ON(dev->driver != NULL); | 3130 | if (WARN_ON(dev->driver)) |
| 3131 | return; | ||
| 3131 | 3132 | ||
| 3132 | /* dma pool cleanup */ | 3133 | /* dma pool cleanup */ |
| 3133 | if (dev->data_requests) | 3134 | if (dev->data_requests) |
