On Wed, Jul 01, 2015 at 12:12:05AM +0300, Sergei Shtylyov wrote:
> On 07/01/2015 12:10 AM, Felipe Balbi wrote:
>
> >>>that's an error condition, not a debugging message.
>
> >>>Let's promote it appropriately.
>
> >>>Signed-off-by: Felipe Balbi <balbi@xxxxxx>
> >>>---
> >>> drivers/usb/dwc3/dwc3-keystone.c | 2 +-
> >>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> >>>diff --git a/drivers/usb/dwc3/dwc3-keystone.c b/drivers/usb/dwc3/dwc3-keystone.c
> >>>index fe3b9335a74e..c1ca2a0a56ef 100644
> >>>--- a/drivers/usb/dwc3/dwc3-keystone.c
> >>>+++ b/drivers/usb/dwc3/dwc3-keystone.c
> >>>@@ -115,7 +115,7 @@ static int kdwc3_probe(struct platform_device *pdev)
> >>>
> >>> error = clk_prepare_enable(kdwc->clk);
> >>> if (error < 0) {
> >>>- dev_dbg(kdwc->dev, "unable to enable usb clock, err %d\n",
> >>>+ dev_err(kdwc->dev, "unable to enable usb clock, err %d\n",
>
> >> Perhaps time to s/err/error/?
>
> >what do you mean ? There's a dev_error now ? Can't seem to find it with
> >grep.
>
> I meant "err" in the message. :-)
oh, haha, I was completely blind to that. I'll fix it in my branch
(testing/next) Thanks.
--
balbi
Attachment:
signature.asc
Description: Digital signature
