Re: [linux-usb-devel] Problems with usblp (2.6.0-test5)

[Daniel Drake]

Dave,
Thanks for your informative reply.
David Brownell wrote:

Hard to say precisely what went wrong, the printer driver doesn't
turn on any diagnostics when kernel config says CONFIG_USB_DEBUG.
(Feel like submitting a patch to make it do so?)
Written a patch. I'm new at this kernel development business, do you 
think you could inform me who I should submit the patch to?
I've included it at the bottom of the mail anyway.

Configure usblp debug info.  This is likely a bug in that driver
(printer driver needs attenion), the other stuff is just debug
noise at the moment (other than pointing out nothing strange was
going on).
With USBLP debug info turned on, here's my dmesg output when I try and 
print:
ohci-hcd 0000:00:02.1: urb dfcc8500 path 3 ep2in 93d20000 cc 9 --> 
status -121
drivers/usb/class/usblp.c: usblp_control_msg: rq: 0x00 dir: 1 recip: 1 
value: 0 len: 0x3ff result: 90
drivers/usb/class/usblp.c: usblp0 Device ID string 
[len=90]="MFG:EPSON;CMD:ESCPL2,BDC,D4;MDL:Stylus COLOR 
680;CLS:PRINTER;DES:EPSON Stylus COLOR 680;"
drivers/usb/class/usblp.c: usblp_control_msg: rq: 0x01 dir: 1 recip: 1 
value: 0 len: 0x1 result: 1
usb 3-3: hcd_unlink_urb deccf900 fail -22
usb 3-3: hcd_unlink_urb dfcc8500 fail -22

I had to comment out one line of the debug code because it caused 
compile errors. My knowledge is not (yet) enough to be able to diagnose 
this myself, sorry.
Patch (against 2.6.0-test5) is below:

--- linux-2.6.0-test5/drivers/usb/class/usblp.c 2003-09-09 08:50:01.000000000 +0100
+++ linux/drivers/usb/class/usblp.c     2003-09-14 22:31:40.000000000 +0100
@@ -54,7 +54,13 @@
#include <linux/init.h>
#include <linux/slab.h>
#include <linux/lp.h>
-#undef DEBUG
+
+#ifdef CONFIG_USB_DEBUG
+       #define DEBUG
+#else
+       #undef DEBUG
+#endif
+
#include <linux/usb.h>
/*
@@ -159,7 +165,9 @@
        dbg("usblp=0x%p", usblp);
        dbg("dev=0x%p", usblp->dev);
        dbg("present=%d", usblp->present);
-       dbg("buf=0x%p", usblp->buf);
+       /* causes error:
+        dbg("buf=0x%p", usblp->buf);
+       */      
        dbg("readcount=%d", usblp->readcount);
        dbg("ifnum=%d", usblp->ifnum);
    for (p = USBLP_FIRST_PROTOCOL; p <= USBLP_LAST_PROTOCOL; p++) {
--

Please CC me as I'm not on the list

--
Daniel Drake (dsd)
http://www.reactivated.net


-------------------------------------------------------
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