From: Cristian Birsan <cristian.bir...@microchip.com>

Use dev_err() to display EP configuration error to avoid silent failure.

Signed-off-by: Cristian Birsan <cristian.bir...@microchip.com>
Acked-by: Nicolas Ferre <nicolas.fe...@microchip.com>
---
 drivers/usb/gadget/udc/atmel_usba_udc.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/usb/gadget/udc/atmel_usba_udc.c 
b/drivers/usb/gadget/udc/atmel_usba_udc.c
index 8bc0b52a..3ccc341 100644
--- a/drivers/usb/gadget/udc/atmel_usba_udc.c
+++ b/drivers/usb/gadget/udc/atmel_usba_udc.c
@@ -1851,8 +1851,8 @@ static irqreturn_t usba_udc_irq(int irq, void *devid)
                 * but it's clearly harmless...
                 */
                if (!(usba_ep_readl(ep0, CFG) & USBA_EPT_MAPPED))
-                       dev_dbg(&udc->pdev->dev,
-                                "ODD: EP0 configuration is invalid!\n");
+                       dev_err(&udc->pdev->dev,
+                               "ODD: EP0 configuration is invalid!\n");
 
                /* Preallocate other endpoints */
                n = fifo_mode ? udc->num_ep : udc->configured_ep;
@@ -1860,8 +1860,8 @@ static irqreturn_t usba_udc_irq(int irq, void *devid)
                        ep = &udc->usba_ep[i];
                        usba_ep_writel(ep, CFG, ep->ept_cfg);
                        if (!(usba_ep_readl(ep, CFG) & USBA_EPT_MAPPED))
-                               dev_dbg(&udc->pdev->dev,
-                                "ODD: EP%d configuration is invalid!\n", i);
+                               dev_err(&udc->pdev->dev,
+                                       "ODD: EP%d configuration is 
invalid!\n", i);
                }
        }
 
-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to