* [PATCH net-next] net: cdc_ncm: demote "unexpected notification" to debug level
@ 2013-04-08 18:26 Bjørn Mork
[not found] ` <1365445583-15124-1-git-send-email-bjorn-yOkvZcmFvRU@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Bjørn Mork @ 2013-04-08 18:26 UTC (permalink / raw)
To: netdev; +Cc: linux-usb, Bjørn Mork, Alexey Orishko
Receiving unhandled notifications is most certainly not an error
and should not be logged as one. Knowing that the device sends
notifications we don't handle is useful for developers, but there
is very little a user can do about this. The message is therefore
just annoying noise to most users with devices sending unhandled
notifications like e.g. USB_CDC_NOTIFY_RESPONSE_AVAILABLE
Cc: Alexey Orishko <alexey.orishko@stericsson.com>
Signed-off-by: Bjørn Mork <bjorn@mork.no>
---
drivers/net/usb/cdc_ncm.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/net/usb/cdc_ncm.c b/drivers/net/usb/cdc_ncm.c
index 44a989c..67012cb 100644
--- a/drivers/net/usb/cdc_ncm.c
+++ b/drivers/net/usb/cdc_ncm.c
@@ -1124,8 +1124,9 @@ static void cdc_ncm_status(struct usbnet *dev, struct urb *urb)
break;
default:
- dev_err(&dev->udev->dev, "NCM: unexpected "
- "notification 0x%02x!\n", event->bNotificationType);
+ dev_dbg(&dev->udev->dev,
+ "NCM: unexpected notification 0x%02x!\n",
+ event->bNotificationType);
break;
}
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-next] net: cdc_ncm: demote "unexpected notification" to debug level
[not found] ` <1365445583-15124-1-git-send-email-bjorn-yOkvZcmFvRU@public.gmane.org>
@ 2013-04-08 20:58 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2013-04-08 20:58 UTC (permalink / raw)
To: bjorn-yOkvZcmFvRU
Cc: netdev-u79uwXL29TY76Z2rM5mHXA, linux-usb-u79uwXL29TY76Z2rM5mHXA,
alexey.orishko-0IS4wlFg1OjSUeElwK9/Pw
From: Bjørn Mork <bjorn-yOkvZcmFvRU@public.gmane.org>
Date: Mon, 8 Apr 2013 20:26:23 +0200
> Receiving unhandled notifications is most certainly not an error
> and should not be logged as one. Knowing that the device sends
> notifications we don't handle is useful for developers, but there
> is very little a user can do about this. The message is therefore
> just annoying noise to most users with devices sending unhandled
> notifications like e.g. USB_CDC_NOTIFY_RESPONSE_AVAILABLE
>
> Cc: Alexey Orishko <alexey.orishko-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>
> Signed-off-by: Bjørn Mork <bjorn-yOkvZcmFvRU@public.gmane.org>
Applied.
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-04-08 20:58 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-08 18:26 [PATCH net-next] net: cdc_ncm: demote "unexpected notification" to debug level Bjørn Mork
[not found] ` <1365445583-15124-1-git-send-email-bjorn-yOkvZcmFvRU@public.gmane.org>
2013-04-08 20:58 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).