On Tue, Sep 16, 2003 at 02:24:07PM -0700, David Brownell wrote:
> Greg KH wrote:
> >On Tue, Sep 16, 2003 at 07:53:45PM +0100, Daniel Drake wrote:
> >
> >No, for different reasons, the authors of these files did not want debug
> >messages to be tied to CONFIG_USB_DEBUG.  Traditionally CONFIG_USB_DEBUG
> >only enabled debugging in the USB core code, although it looks like a
> >few other drivers do tie their debugging messages to this setting
> >(konicawc.c, usbnet.c, usbtest.c, and the usb-skeleton.c driver.)
> >
> >So I'm not going to take that part of the patch.
> 
> I'd be interested in seeing some more standard hook for that then;
> maybe a CONFIG_USB_DRIVER_DEBUG to enable logging extra low-volume
> diagnostics when interesting faults are detected?
> 
> We get better troubleshooting information from users when getting
> it is just a build option, no apply-editor-to-source steps needed.

You get even more when you turn it into a load option, like all of the
usb-serial drivers have. "modprobe foo debug=1" is much easier to tell
people to do than to rebuild their distro kernel :)

thanks,

greg k-h


-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to